fix: use InterlockedExchangeAdd for Windows atomic ops to return old value

Replace InterlockedAdd/InterlockedIncrement/InterlockedDecrement with
InterlockedExchangeAdd which returns the value before the operation,
matching the behavior of __sync_fetch_and_add (GCC) and
atomic_fetch_add (C11).

Also fix ATOMIC_SUB macro parenthesization: -(n) instead of -n.

Co-authored-by: ithewei <26049660+ithewei@users.noreply.github.com>

Author: Copilot

base/hatomic.h

@@ -52,10 +52,10 @@ static inline bool atomic_flag_test_and_set(atomic_flag* p) {
     return InterlockedCompareExchange(&p->_Value, 1, 0);
 }
 
-#define ATOMIC_ADD          InterlockedAdd
-#define ATOMIC_SUB(p, n)    InterlockedAdd(p, -n)
-#define ATOMIC_INC          InterlockedIncrement
-#define ATOMIC_DEC          InterlockedDecrement
+#define ATOMIC_ADD          InterlockedExchangeAdd
+#define ATOMIC_SUB(p, n)    InterlockedExchangeAdd(p, -(n))
+#define ATOMIC_INC(p)       InterlockedExchangeAdd(p, 1)
+#define ATOMIC_DEC(p)       InterlockedExchangeAdd(p, -1)
 
 #elif __GNUC_PREREQ(4, 1)