according to C language lawyers, this construct is undefined. In practice, its gcc definition is not what you want: the variable content is read before calling DoRun, and is ored+stored after calling DoRun.

2009-09-30 10:50:03 +02:00
parent c5d81b203d
commit 09576cd905
1 changed files with 2 additions and 1 deletions
--- a/src/core/test.cc
+++ b/src/core/test.cc
@@ -124,7 +124,8 @@ TestCase::Run (void)
 {
  DoReportStart ();
  DoSetup ();
-  m_error |= DoRun ();
+  bool status = DoRun ();
+  m_error |= status;
  DoTeardown ();
  if (m_error == false)
    {