[ros-diffs] [hyperion] 37704: ASSERT really needs to be void, despite what DDK headers might say. Fixes following warning: lib\rtl\srw.c: In function 'RtlAcquireSRWLockShared': lib\rtl\srw.c:498: error: statement with no effect lib\rtl\srw.c: In function 'RtlAcquireSRWLockExclusive': lib\rtl\srw.c:637: error: statement with no effect lib\rtl\srw.c:668: error: statement with no effect lib\rtl\srw.c:696: error: statement with no effect See issue #3922 for more details.

hyperion at svn.reactos.org hyperion at svn.reactos.org
Fri Nov 28 15:33:22 CET 2008


Author: hyperion
Date: Fri Nov 28 08:33:21 2008
New Revision: 37704

URL: http://svn.reactos.org/svn/reactos?rev=37704&view=rev
Log:
ASSERT really needs to be void, despite what DDK headers might say. Fixes following warning:

lib\rtl\srw.c: In function 'RtlAcquireSRWLockShared':
lib\rtl\srw.c:498: error: statement with no effect
lib\rtl\srw.c: In function 'RtlAcquireSRWLockExclusive':
lib\rtl\srw.c:637: error: statement with no effect
lib\rtl\srw.c:668: error: statement with no effect
lib\rtl\srw.c:696: error: statement with no effect
See issue #3922 for more details.

Modified:
    trunk/reactos/include/ndk/rtlfuncs.h

Modified: trunk/reactos/include/ndk/rtlfuncs.h
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/ndk/rtlfuncs.h?rev=37704&r1=37703&r2=37704&view=diff
==============================================================================
--- trunk/reactos/include/ndk/rtlfuncs.h [iso-8859-1] (original)
+++ trunk/reactos/include/ndk/rtlfuncs.h [iso-8859-1] Fri Nov 28 08:33:21 2008
@@ -195,14 +195,14 @@
 #if DBG
 
 #define ASSERT( exp ) \
-    ((!(exp)) ? \
+    ((void)((!(exp)) ? \
         (RtlAssert( #exp, __FILE__, __LINE__, NULL ),FALSE) : \
-        TRUE)
+        TRUE))
 
 #define ASSERTMSG( msg, exp ) \
-    ((!(exp)) ? \
+    ((void)((!(exp)) ? \
         (RtlAssert( #exp, __FILE__, __LINE__, msg ),FALSE) : \
-        TRUE)
+        TRUE))
 
 #else
 
@@ -670,7 +670,7 @@
     IN HANDLE HeapHandle,
     IN PVOID HeapEntry
 );
-    
+
 #define RtlGetProcessHeap() (NtCurrentPeb()->ProcessHeap)
 
 //



More information about the Ros-diffs mailing list