Merge pull request #338 from ewasm/sign-conversion

CMake: enable -Wsign-conversion
ewasm · Aug 21, 2018 · 0a7d219 · 0a7d219
2 parents 98b47bf + e999462
commit 0a7d219
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -28,7 +28,7 @@ cable_set_build_type(DEFAULT RelWithDebInfo CONFIGURATION_TYPES Debug;Release;Re
 
 cable_configure_compiler()
 # TODO: fix the warnings instead
-add_compile_options(-Wno-pedantic -Wno-sign-conversion)
+add_compile_options(-Wno-pedantic)
 
 include(ProjectBinaryen)
 

diff --git a/src/eei.cpp b/src/eei.cpp
@@ -671,7 +671,7 @@ namespace hera {
       }
 
       ensureCondition(gas >= 0, ArgumentOutOfRange, "Negative gas supplied.");
-      heraAssert((m_msg.flags & ~EVMC_STATIC) == 0, "Unknown flags not supported.");
+      heraAssert((m_msg.flags & ~uint32_t(EVMC_STATIC)) == 0, "Unknown flags not supported.");
 
       evmc_message call_message;
       call_message.destination = loadUint160(addressOffset);