ki.stfu added a comment. In https://reviews.llvm.org/D29256#662167, @krytarowski wrote:
> @ki.stfu do you still agree with this patch? Please follow my comments and then we can go ahead. ================ Comment at: tools/lldb-mi/MIDriver.cpp:512 CMICmnThreadMgrStd::Instance().GetErrorDescription().c_str()); - SetErrorDescriptionn(errMsg); + SetErrorDescriptionn(errMsg.c_str()); return MIstatus::failure; ---------------- `SetErrorDescription` accepts a const reference to `CMIUtilString` so it suits better than `SetErrorDescriptionn` (double `n`). ``` SetErrorDescription(errMsg); ``` //Oh, certainly it should be renamed// Repository: rL LLVM https://reviews.llvm.org/D29256 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits