Farid Zaripov wrote:
From: Martin Sebor [mailto:[EMAIL PROTECTED]
Sent: Ср, 02.01.2008 20:41
To: [EMAIL PROTECTED]
Subject: Re: [PATCH] _mutex.h
+extern "C" long __cdecl _InterlockedIncrement (long volatile*);
+extern "C" long __cdecl _InterlockedDecrement (long volatile*);
+extern "C" long __cdecl _InterlockedExchange (long volatile*, long);
Shouldn't the type of the argument be _RWSTD_INTERLOCKED_T* instead?
No. _RWSTD_INTERLOCKED_T #defined only for 32-bit compiler, while intrinsic
functions are available in 32 and 64-bit compilers.
Why isn't it defined for the 64-bit compiler? (Because the config
test fails?) It seems that it should be defined regardless, wouldn't
you say? Even if we need to hardcode it _config.h or _defs.h...
Martin
If not, our convention is to put the cv-qualifier(s) before the type,
not after it (i.e., const T* or volatile T*, rather than T const* or
T volatile*).
Farid.
