Simplify assert macro for clang compatibility

mmore500 · Oct 14, 2022 · d707c83 · d707c83
1 parent 48c716d
commit d707c83
Show file tree

Hide file tree

Showing 6 changed files with 13 additions and 10 deletions.
diff --git a/include/netuit/arrange/AdjacencyFileTopologyFactory.hpp b/include/netuit/arrange/AdjacencyFileTopologyFactory.hpp
@@ -15,7 +15,7 @@ namespace netuit {
 inline Topology make_adjacency_file_topology(const std::string& filename) {
   std::ifstream file(filename);
 
-  uitsl_always_assert(file);
+  uitsl_always_assert(file, "");
 
   return dynamic_cast<std::istream&>(file);
 }

diff --git a/include/uitsl/algorithm/clamp_cast.hpp b/include/uitsl/algorithm/clamp_cast.hpp
@@ -35,7 +35,7 @@ To clamp_cast( const From from ) {
   #ifdef __EMSCRIPTEN__
     // necessary to keep nan sanitization from getting optimized away
     // and (I assume?) implicitly taken care of via the static_cast
-    uitsl_always_assert( !std::isnan( clamped ) );
+    uitsl_always_assert( !std::isnan( clamped ), "" );
   #endif
 
   return static_cast<To>( clamped );

diff --git a/include/uitsl/debug/uitsl_always_assert.hpp b/include/uitsl/debug/uitsl_always_assert.hpp
@@ -30,11 +30,14 @@ do {                                                                          \
     std::cout << "LINE: " << __LINE__ << '\n';                                \
     std::cout << '\n';                                                        \
     std::cout << "MESSAGE:\n";                                                \
+    std::cout << uitsl_ns::SetSeparator(" << ") << __VA_ARGS__;;              \
     std::cout << UITSL_STRINGIFY(__VA_ARGS__) << '\n';                        \
-    std::cout << uitsl_ns::SetSeparator(" << ") __VA_OPT__(<<) __VA_ARGS__;   \
     std::cout << '\n';                                                        \
     std::abort();                                                             \
   }                                                                           \
 } while(0)
 
+// clang doesn't support __VA_OPT__ yet
+// std::cout << uitsl_ns::SetSeparator(" << ") __VA_OPT__(<<) __VA_ARGS__;   \
+
 #endif // #ifndef UITSL_DEBUG_UITSL_ALWAYS_ASSERT_HPP_INCLUDE
diff --git a/include/uitsl/fetch/fetch_native.hpp b/include/uitsl/fetch/fetch_native.hpp
@@ -53,7 +53,7 @@ inline std::filesystem::path fetch_native( const std::string& url ) {
     [&curl_handle](){ curl_easy_cleanup( curl_handle ); }
   );
 
-  uitsl_always_assert( curl_handle );
+  uitsl_always_assert( curl_handle, "" );
 
   uitsl::err_verify(
     curl_easy_setopt( curl_handle, CURLOPT_URL, url.c_str())

diff --git a/tests/uitsl/debug/uitsl_always_assert.cpp b/tests/uitsl/debug/uitsl_always_assert.cpp
@@ -6,9 +6,9 @@ TEST_CASE("uitsl_always_assert", "[nproc:1]") {
 
   // uitsl_always_assert(false);
 
-  uitsl_always_assert(true);
-
-  uitsl_always_assert(true, );
+  // uitsl_always_assert(true);
+  //
+  // uitsl_always_assert(true, );
 
   uitsl_always_assert(true, "");
 

diff --git a/tests/uitsl/debug/uitsl_assert.cpp b/tests/uitsl/debug/uitsl_assert.cpp
@@ -7,9 +7,9 @@
 TEST_CASE("uitsl_assert", "[nproc:1]") {
 
 
-  uitsl_assert(true);
-
-  uitsl_assert(true, );
+  // uitsl_assert(true);
+  //
+  // uitsl_assert(true, );
 
   uitsl_assert(true, "");