labath added a subscriber: labath.
labath added a comment.

Does this fix an existing test or is a new issue? If it's new (it sounds like 
it is, as I don't see any test failures), could you also add a test for this. 
It shouldn't be too difficult to write one... Would something like this:

  dbg.SetAsync(true)
  process.Continue()
  lldbutil.expect_state_changes(self,dbg.GetListener(), [eStateRunning])
  target.BreakpointCreateBySourceRegex("// some code which will not get 
executed by the inferior")
  do_some_assertions_on_the_breakpoint()
  while dbg.GetListener().WaitForEvent(2, event):
    if SBProcess.GetStateFromEvent(event) == eStateStopped and 
SBProcess.GetRestartedFromEvent(state):
      continue
    if SBProcess.GetStateFromEvent(event) == eStateRunning:
      continue
    self.fail("Setting a breakpoint generated an unexpected event: 
%s"%SBDebugger.StateAsCString(SBProcess.GetStateFromEvent(event)));

do the trick?


http://reviews.llvm.org/D18886



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

Reply via email to