Avoid relying on a non-portable implementation detail for atomic_flag
tests.  Per the standard, the only way to obtain the value of the flag
is via the return value from atomic_flag_test_and_set.

Signed-off-by: Ed Maste <ema...@freebsd.org>
---
 tests/test-atomic.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tests/test-atomic.c b/tests/test-atomic.c
index e9bd6bd..e6df1cd 100644
--- a/tests/test-atomic.c
+++ b/tests/test-atomic.c
@@ -66,9 +66,9 @@ test_atomic_flag(void)
 {
     atomic_flag flag = ATOMIC_FLAG_INIT;
     ovs_assert(atomic_flag_test_and_set(&flag) == false);
-    ovs_assert(flag.b == true);
+    ovs_assert(atomic_flag_test_and_set(&flag) == true);
     atomic_flag_clear(&flag);
-    ovs_assert(flag.b == false);
+    ovs_assert(atomic_flag_test_and_set(&flag) == false);
 }
 
 int
-- 
1.8.3.3

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to