[ros-diffs] [hyperion] 37319: modified include/reactos/libs/pseh/pseh2.h Kill the last few remaining "'x' may be used uninitialized in this function" warnings. I'd love to know the opinion of GCC developers on what does this hack do to their compiler and why

hyperion at svn.reactos.org hyperion at svn.reactos.org
Wed Nov 12 22:15:46 CET 2008


Author: hyperion
Date: Wed Nov 12 15:15:45 2008
New Revision: 37319

URL: http://svn.reactos.org/svn/reactos?rev=37319&view=rev
Log:
modified   include/reactos/libs/pseh/pseh2.h
   Kill the last few remaining "'x' may be used uninitialized in this function" warnings. I'd love to know the opinion of GCC developers on what does this hack do to their compiler and why

Modified:
    trunk/reactos/include/reactos/libs/pseh/pseh2.h

Modified: trunk/reactos/include/reactos/libs/pseh/pseh2.h
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/reactos/libs/pseh/pseh2.h?rev=37319&r1=37318&r2=37319&view=diff
==============================================================================
--- trunk/reactos/include/reactos/libs/pseh/pseh2.h [iso-8859-1] (original)
+++ trunk/reactos/include/reactos/libs/pseh/pseh2.h [iso-8859-1] Wed Nov 12 15:15:45 2008
@@ -234,8 +234,6 @@
 	__SEH_BEGIN_SCOPE \
 	{ \
 		__SEH_SCOPE_LOCALS; \
- \
-		void _SEHJumpToHandler() { goto _SEHBeginExcept; } \
 \
 		__SEH_BEGIN_TRY \
 		{
@@ -327,6 +325,7 @@
 		{ \
 			{ \
 				_SEHFrame_t * const _SEHFrameP = _SEHTopTryLevel ? &_SEHFrame : _SEHCurFrameP; \
+				(void)_SEHFrameP; \
 				__SEH_BARRIER;
 
 #define _SEH2_END \
@@ -334,6 +333,12 @@
 			} \
 		} \
 		_SEHEndExcept: __SEH_SIDE_EFFECT; \
+ \
+		void _SEHJumpToHandler() \
+		{ \
+			if(_SEHFrameP == (_SEHTopTryLevel ? &_SEHFrame : _SEHCurFrameP) && _SEHPrevTryLevelP == _SEHTryLevelP) \
+				goto _SEHBeginExcept; \
+		} \
 	} \
 	__SEH_END_SCOPE;
 



More information about the Ros-diffs mailing list