[Lldb-commits] [PATCH] D16247: Don't assume that thread 0 is always the main thread

[Jim Ingham via lldb-commits]

jingham requested changes to this revision.
jingham added a comment.
This revision now requires changes to proceed.

That looks good.  get_one_thread_stopped_at_breakpoint is convenient, but I'd rather not sweep under the rug cases where you expected only one thread to hit your breakpoint but more than one did.  I think it would be better if this API returned None for number of threads != 1, not just threads == 0.  That way you'd get testing for that for free as well.  And then in the cases where it was expected that more than one thread might hit the breakpoint, you should use get_threads_stopped_at_breakpoint.


http://reviews.llvm.org/D16247