tfiala added inline comments.
================
Comment at: source/Core/Debugger.cpp:426
@@ +425,3 @@
+ for (const auto& debugger: debuggers)
+ debugger->Clear();
+
----------------
ovyalov wrote:
> tfiala wrote:
> > Wouldn't Clear() be considered a mutating function? So a const debugger
> > ref seems like maybe it should be non-const?
> >
> > i.e.
> > for (auto& debugger: debuggers) {
> > debugger->Clear();
> > }
> >
> Clear is mutable, but debugger has DebuggerSP type here - so, const is
> applicable only to shared_ptr, not to Debugger instance itself.
Ah gotcha.
http://reviews.llvm.org/D12683
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
