[ros-diffs] [cfinck] 33486: Fix the Release build by marking MmPageArraySize as unreferenced. Changing the ASSERT, so that it doesn't spit out an "unused variable" warning on Release builds doesn't work, because then it expects this variable to exist. (which is not the case for some struct elements passed to ASSERT in some other parts of the kernel)

cfinck at svn.reactos.org cfinck at svn.reactos.org
Mon May 12 22:48:49 CEST 2008


Author: cfinck
Date: Mon May 12 15:48:48 2008
New Revision: 33486

URL: http://svn.reactos.org/svn/reactos?rev=33486&view=rev
Log:
Fix the Release build by marking MmPageArraySize as unreferenced.
Changing the ASSERT, so that it doesn't spit out an "unused variable" warning on Release builds doesn't work, because then it expects this variable to exist. (which is not the case for some struct elements passed to ASSERT in some other parts of the kernel)

Modified:
    trunk/reactos/ntoskrnl/include/internal/mm.h

Modified: trunk/reactos/ntoskrnl/include/internal/mm.h
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/include/internal/mm.h?rev=33486&r1=33485&r2=33486&view=diff
==============================================================================
--- trunk/reactos/ntoskrnl/include/internal/mm.h [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/include/internal/mm.h [iso-8859-1] Mon May 12 15:48:48 2008
@@ -986,6 +986,9 @@
     extern PPHYSICAL_PAGE MmPageArray;
     extern ULONG MmPageArraySize;
 
+    /* Mark MmPageArraySize as unreferenced, otherwise it will appear as an unused variable on a Release build */
+    UNREFERENCED_PARAMETER(MmPageArraySize);
+
     /* Make sure the PFN number is valid */
     ASSERT(Pfn <= MmPageArraySize);
 



More information about the Ros-diffs mailing list