ovyalov added inline comments.

================
Comment at: source/Core/Debugger.cpp:426
@@ +425,3 @@
+    for (const auto& debugger: debuggers)
+        debugger->Clear();
+
----------------
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.


http://reviews.llvm.org/D12683



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

Reply via email to