Fix more compile warnings on gcc 8.1

chen3feng · May 30, 2018 · 9ea9602 · 9ea9602
1 parent 5f38b62
commit 9ea9602
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/hash/BUILD b/hash/BUILD
@@ -34,6 +34,7 @@ cc_library(
         ':murmur',
         '//thirdparty/stringencoders:stringencoders'
     ],
+    extra_cppflags = ['-Wno-implicit-fallthrough'],
 )
 
 cc_library(
@@ -52,6 +53,7 @@ cc_library(
         'murmur/MurmurHashAligned2.cpp',
         'murmur/MurmurHashNeutral2.cpp',
     ],
+    extra_cppflags = ['-Wno-implicit-fallthrough'],
 )
 
 cc_test(

diff --git a/system/threading/semaphore.cpp b/system/threading/semaphore.cpp
@@ -15,7 +15,7 @@ namespace internal {
 
 bool SemaphoreOps::TryAcquire(sem_t* sem)
 {
-    if (TOFT_EINTR_IGNORED(sem_trywait(sem) == 0))
+    if (TOFT_EINTR_IGNORED(sem_trywait(sem)) == 0)
         return true;
     int error = errno;
     if (error == EAGAIN)