bulbazord added inline comments.
================
Comment at: lldb/test/API/functionalities/scripted_process/main.cpp:18
+ std::unique_lock<std::mutex> lock(mutex);
+ cv.notify_one();
+ n = foo(n);
----------------
Why do you need this initial notification?
================
Comment at: lldb/test/API/functionalities/scripted_process/main.cpp:25-27
+ while (baz(n, mutex, cv, done) != 42) {
+ ;
+ }
----------------
Why does this need to be in a while loop? The condition_variable's `wait`
function should guarantee that `baz` only ever returns `42`, no? Unless I'm
missing something.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139484/new/
https://reviews.llvm.org/D139484
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits