Skip to content

Commit 0905767

Browse files
committed
[libc++] Address remaining comments from #130145
I've accidentally merged the PR before addressing all comments. This patch fixes the remaining ones.
1 parent 163871c commit 0905767

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

libcxx/test/std/thread/futures/futures.async/wait_on_destruct.pass.cpp

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,15 @@
1515
// Make sure that the `future` destructor keeps the data alive until the thread finished. This test fails by triggering
1616
// TSan. It may not be observable by normal means.
1717

18+
// See https://github.com/llvm/llvm-project/pull/125433#issuecomment-2703618927 for more details.
19+
1820
#include <atomic>
1921
#include <future>
2022
#include <mutex>
2123

2224
std::mutex mux;
2325

24-
int main() {
26+
int main(int, char**) {
2527
using namespace std::chrono_literals;
2628
std::unique_lock lock(mux);
2729
std::atomic<bool> in_async = false;
@@ -34,4 +36,6 @@ int main() {
3436
});
3537
in_async.wait(true);
3638
lock.unlock();
39+
40+
return 0;
3741
}

0 commit comments

Comments
 (0)