Suppress clang false positive warning

ad-freiburg · Feb 27, 2025 · df1f21e · df1f21e
1 parent 474f936
commit df1f21e
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/engine/Union.cpp b/src/engine/Union.cpp
@@ -7,7 +7,7 @@
 
 #include "engine/CallFixedSize.h"
 #include "util/ChunkedForLoop.h"
-#include "util/TransparentFunctors.h"
+#include "util/CompilerWarnings.h"
 
 const size_t Union::NO_COLUMN = std::numeric_limits<size_t>::max();
 
@@ -498,6 +498,9 @@ Result::Generator Union::computeResultKeepOrder(
                          ? Range{std::array{Wrapper{result2->idTable(),
                                                     result2->localVocab()}}}
                          : Range{std::move(result2->idTables())};
+
+  // Clang falsely reports that the this capture is unused here.
+  DISABLE_UNUSED_LAMBDA_CAPTURE_WARNINGS
   return std::visit(
       [this, requestLaziness, &result1, &result2](auto leftRange,
                                                   auto rightRange) {
@@ -511,4 +514,5 @@ Result::Generator Union::computeResultKeepOrder(
             });
       },
       std::move(leftRange), std::move(rightRange));
+  ENABLE_UNUSED_LAMBDA_CAPTURE_WARNINGS
 }
diff --git a/src/util/CompilerWarnings.h b/src/util/CompilerWarnings.h
@@ -18,4 +18,14 @@
 #define ENABLE_UNINITIALIZED_WARNINGS
 #endif
 
+#if defined(__clang__) && (__clang_major__ >= 16 && __clang_major__ <= 18)
+#define DISABLE_UNUSED_LAMBDA_CAPTURE_WARNINGS \
+  _Pragma("clang diagnostic push")             \
+      _Pragma("clang diagnostic ignored \"-Wunused-lambda-capture\"")
+#define ENABLE_UNUSED_LAMBDA_CAPTURE_WARNINGS _Pragma("clang diagnostic pop")
+#else
+#define DISABLE_UNUSED_LAMBDA_CAPTURE_WARNINGS
+#define ENABLE_UNUSED_LAMBDA_CAPTURE_WARNINGS
+#endif
+
 #endif  // QLEVER_COMPILERWARNINGS_H