[Lldb-commits] [PATCH] D139484: [lldb/test] Fix data racing issue in TestStackCoreScriptedProcess

Alex Langford via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Dec 13 11:51:39 PST 2022


bulbazord accepted this revision.
bulbazord added a comment.
This revision is now accepted and ready to land.

LGTM, small nit



================
Comment at: lldb/test/API/functionalities/scripted_process/main.cpp:21
+                   // returns
+  lock.lock();
+}
----------------
nit: no need to relock, you're done modifying `n`. The lock will get freed right after anyway.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139484/new/

https://reviews.llvm.org/D139484



More information about the lldb-commits mailing list