aprantl added a comment.
Did the old implementation come with a testcase? Perhaps I'm misunderstanding
the question, but it would probably be best to preserve the old behavior.
================
Comment at: tools/lldb-mi/MICmdCmdExec.cpp:515
+ lldb::SBError error;
+ if (nThreadId != UINT64_MAX) {
+ lldb::SBThread sbThread =
rSessionInfo.GetProcess().GetThreadByID(nThreadId);
----------------
Usually we prefer early exits in LLVM code
(https://www.llvm.org/docs/CodingStandards.html#use-early-exits-and-continue-to-simplify-code)
```
if (!sbThread.IsValid)
return failure;
sbThread.StepInto();
```
https://reviews.llvm.org/D47838
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
