Try to fix name mangling issues with GCC and static call operators

Author: trueqbit

tests/CMakeLists.txt

@@ -55,10 +55,7 @@ if(SQLITE_ORM_OMITS_CODECVT)
     target_compile_definitions(unit_tests PRIVATE SQLITE_ORM_OMITS_CODECVT=1)
 endif()
 
-if(CMAKE_CXX_COMPILER_ID MATCHES "GNU")
-    # GCC would complain about name mangling errors (with type traits for locally defined lambdas) if the ABI version is not set to 0
-    add_compile_options(-fabi-version=0)
-elseif(MSVC)
+if(MSVC)
     target_compile_options(unit_tests PUBLIC
         # multi-processor compilation
         /MP

tests/static_tests/function_static_tests.cpp

@@ -315,29 +315,31 @@ TEST_CASE("function static") {
             }
 #endif
         };
+#ifdef SQLITE_ORM_STATIC_CALL_OPERATOR_SUPPORTED
+        // note: this static lambda lives up here because of GCC 13.2 and mangling issues
+        constexpr auto lambda_static_dummy = [](unsigned long errcode) static {
+            return errcode != 0;
+        };
+#endif
 
         SECTION("detect overloaded call operator") {
-            constexpr auto lambda = [](unsigned long errcode) {
-                return errcode != 0;
-            };
+            constexpr auto lambda = [](unsigned long) {};
             using lambda_type = std::remove_const_t<decltype(lambda)>;
 
             STATIC_REQUIRE(
-                polyfill::is_detected_v<internal::overloaded_callop_t, bool(unsigned long) const, lambda_type>);
+                polyfill::is_detected_v<internal::overloaded_callop_t, void(unsigned long) const, lambda_type>);
             STATIC_REQUIRE_FALSE(
-                polyfill::is_detected_v<internal::overloaded_static_callop_t, bool(unsigned long) const, lambda_type>);
+                polyfill::is_detected_v<internal::overloaded_static_callop_t, void(unsigned long) const, lambda_type>);
         }
 #ifdef SQLITE_ORM_STATIC_CALL_OPERATOR_SUPPORTED
         SECTION("detect overloaded static call operator") {
-            constexpr auto lambda = [](unsigned long errcode) static {
-                return errcode != 0;
-            };
+            constexpr auto lambda = [](unsigned long) static {};
             using lambda_type = std::remove_const_t<decltype(lambda)>;
 
             STATIC_REQUIRE_FALSE(
-                polyfill::is_detected_v<internal::overloaded_callop_t, bool(unsigned long), lambda_type>);
+                polyfill::is_detected_v<internal::overloaded_callop_t, void(unsigned long), lambda_type>);
             STATIC_REQUIRE(
-                polyfill::is_detected_v<internal::overloaded_static_callop_t, bool(unsigned long), lambda_type>);
+                polyfill::is_detected_v<internal::overloaded_static_callop_t, void(unsigned long), lambda_type>);
         }
 #endif
         SECTION("freestanding function") {

tests/user_defined_functions.cpp

@@ -561,6 +561,12 @@ TEST_CASE("generalized scalar udf") {
         }
 #endif
     };
+#ifdef SQLITE_ORM_STATIC_CALL_OPERATOR_SUPPORTED
+    // note: this static lambda lives up here because of GCC 13.2 and mangling issues
+    constexpr auto lambda_static_dummy = [](unsigned long errcode) static {
+        return errcode != 0;
+    };
+#endif
 
     auto storage = make_storage("");
     storage.sync_schema();