From 09576cd905da7be5dbb42fe139d8f347fbfe9649 Mon Sep 17 00:00:00 2001
From: Mathieu Lacage <mathieu.lacage@sophia.inria.fr>
Date: Wed, 30 Sep 2009 10:50:03 +0200
Subject: [PATCH] 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.

---
 src/core/test.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/core/test.cc b/src/core/test.cc
index f34c05a86..29e66548d 100644
--- 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)
     {