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