Fix assert in EGLSyncControlTest

`ust`, `ust2`, `msc`, and `msc2` are unsigned, so their difference is still unsigned. The `>0` assert could never fail. Fix this by simplifying the test to compare the pairs of values directly. Fixes: 5d9f5df0 ("Implementation of eglGetSyncValuesCHROMIUM extension.") Fixes: ee218f27 ("Re-land eglGetSyncValuesCHROMIUM extension.") Signed-off-by: Eric Engestrom <eric.engestrom@imgtec.com> Change-Id: I0b19d786720f545c846640d48c30a83a1d001396 Reviewed-on: https://chromium-review.googlesource.com/500189Reviewed-by: Jamie Madill <jmadill@chromium.org> Commit-Queue: Jamie Madill <jmadill@chromium.org>

Fix assert in EGLSyncControlTest
82830ede · Eric Engestrom · Commit Bot · 8d835fba · 82830ede
Commit 82830ede authored May 09, 2017 by Eric Engestrom Committed by Commit Bot May 11, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

EGLSyncControlTest.cpp src/tests/egl_tests/EGLSyncControlTest.cpp +2 -2

No files found.
--- a/src/tests/egl_tests/EGLSyncControlTest.cpp
+++ b/src/tests/egl_tests/EGLSyncControlTest.cpp
@@ -260,6 +260,6 @@ TEST_F(EGLSyncControlTest, DISABLED_SyncValuesTest)
    // sbc2 should be 2. msc2 and ust2 should be greater than previous msc and ust values.
    ASSERT_EQ(2ull, sbc2);
-    ASSERT_GT((ust2 - ust), 0ull);
+    ASSERT_GT(ust2, ust);
-    ASSERT_GT((msc2 - msc), 0ull);
+    ASSERT_GT(msc2, msc);
 }