[Lldb-commits] [PATCH] D32366: Set "success" status correctly

2017-08-28 Thread John Lindal via Phabricator via lldb-commits
jafl added a comment. Absolutely! https://reviews.llvm.org/D32366 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D32366: Set "success" status correctly

2017-08-28 Thread John Lindal via Phabricator via lldb-commits
jafl updated this revision to Diff 112922. jafl added a comment. Added lldbassert https://reviews.llvm.org/D32366 Files: source/Commands/CommandObjectRegister.cpp source/Interpreter/CommandInterpreter.cpp Index: source/Interpreter/CommandInterpreter.cpp ===

[Lldb-commits] [PATCH] D32366: Set "success" status correctly

2017-08-28 Thread John Lindal via Phabricator via lldb-commits
jafl added a comment. I will try to ensure that the lldbassert gets tested, but right now, none of the tests work on my machine. I emailed lldb-dev for help. Repository: rL LLVM https://reviews.llvm.org/D32366 ___ lldb-commits mailing list lldb

[Lldb-commits] [PATCH] D32366: Set "success" status correctly

2017-05-24 Thread John Lindal via Phabricator via lldb-commits
jafl added a comment. Thanks for the suggestions. I will get around to this - just swamped right now! Repository: rL LLVM https://reviews.llvm.org/D32366 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mai

[Lldb-commits] [PATCH] D32366: Set "success" status correctly

2017-04-21 Thread John Lindal via Phabricator via lldb-commits
jafl created this revision. jafl added a project: LLDB. In CommandObjectRegisterRead.DoExecute(), set the status to "success" when appropriate. Repository: rL LLVM https://reviews.llvm.org/D32366 Files: source/Commands/CommandObjectRegister.cpp Index: source/Commands/CommandObjectRegist

[Lldb-commits] [PATCH] D22831: [LLDB] Documentation for SBAddress class

2017-04-04 Thread John Lindal via Phabricator via lldb-commits
jafl updated this revision to Diff 94112. jafl added a comment. Merged from master and ran clang-format Repository: rL LLVM https://reviews.llvm.org/D22831 Files: include/lldb/API/SBAddress.h Index: include/lldb/API/SBAddress.h =