labath added inline comments.

================
Comment at: include/lldb/Breakpoint/BreakpointSiteList.h:159-161
+  class Guard : private std::unique_lock<std::recursive_mutex> {
+    using std::unique_lock<std::recursive_mutex>::unique_lock;
+  };
----------------
zturner wrote:
> Err, I meant to just deleted the `Guard` class entirely and return 
> `llvm::Optional<std::unique_lock<std::recursive_mutex>>`
You don't even need the Optional class, unique_lock already has an 
uninitialized state.


https://reviews.llvm.org/D39967



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to