summaryrefslogtreecommitdiff
path: root/test/buildtool/execution_engine/executor
diff options
context:
space:
mode:
Diffstat (limited to 'test/buildtool/execution_engine/executor')
-rw-r--r--test/buildtool/execution_engine/executor/executor.test.cpp9
1 files changed, 7 insertions, 2 deletions
diff --git a/test/buildtool/execution_engine/executor/executor.test.cpp b/test/buildtool/execution_engine/executor/executor.test.cpp
index 8e187115..6793014b 100644
--- a/test/buildtool/execution_engine/executor/executor.test.cpp
+++ b/test/buildtool/execution_engine/executor/executor.test.cpp
@@ -143,10 +143,15 @@ class TestApi : public IExecutionApi {
bool /*unused*/) noexcept -> bool final {
return false; // not needed by Executor
}
- auto RetrieveToCas(std::vector<Artifact::ObjectInfo> const& /*unused*/,
+ auto RetrieveToCas(std::vector<Artifact::ObjectInfo> const& unused,
gsl::not_null<IExecutionApi*> const& /*unused*/) noexcept
-> bool final {
- return false; // not needed by Executor
+ // Note that a false-positive "free-nonheap-object" warning is thrown by
+ // gcc 12.2 with GNU libstdc++, if the caller passes a temporary vector
+ // that is not used by this function. Therefore, we explicitly use this
+ // vector here to suppress this warning. The actual value returned is
+ // irrelevant for testing though.
+ return unused.empty(); // not needed by Executor
}
auto Upload(BlobContainer const& blobs, bool /*unused*/) noexcept
-> bool final {