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



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

Reply via email to