[cmake-commits] martink committed cmCTestBuildHandler.cxx 1.57 1.58

[cmake-commits at cmake.org]

Update of /cvsroot/CMake/CMake/Source/CTest
In directory public:/mounts/ram/cvs-serv28239/CTest

Modified Files:
	cmCTestBuildHandler.cxx 
Log Message:
BUG: even safer checking of return value


Index: cmCTestBuildHandler.cxx
===================================================================
RCS file: /cvsroot/CMake/CMake/Source/CTest/cmCTestBuildHandler.cxx,v
retrieving revision 1.57
retrieving revision 1.58
diff -u -d -r1.57 -r1.58
--- cmCTestBuildHandler.cxx	28 Mar 2007 15:06:34 -0000	1.57
+++ cmCTestBuildHandler.cxx	28 Mar 2007 15:07:36 -0000	1.58
@@ -713,14 +713,15 @@
         this->TotalWarnings ++;
         }
       }
-    else if(result == cmsysProcess_State_Exception)
+    }
+  else if(result == cmsysProcess_State_Exception)
+    {
+    if (retVal)
       {
-      if (retVal)
-        {
-        *retVal = cmsysProcess_GetExitException(cp);
-        cmCTestLog(this->CTest, WARNING, "There was an exception: " << *retVal
-                   << std::endl);
-        }
+      *retVal = cmsysProcess_GetExitException(cp);
+      cmCTestLog(this->CTest, WARNING, "There was an exception: " << *retVal
+                 << std::endl);
+      }
     }
   else if(result == cmsysProcess_State_Expired)
     {