aprantl added a comment.

This patch is adding new overloads to SBAPI calls that don't return an SBError, 
such as:

  // Old:
  void StepOutOfFrame(SBFrame &frame);
  // New:
  void StepOutOfFrame(SBFrame &frame, SBError &error);

I wonder if it would be easier to use and more consistent with the rest of the 
API if we instead added an overload that returns an SBError, like this:

  // New:
  SBError StepOutOfFrameWithError(SBFrame &frame);
  // Alternative names that are just as ugly.
  SBError StepOutOfFrameE(SBFrame &frame);
  SBError StepOutOfFrame2(SBFrame &frame);

@clayborg, @jingham: What do you think?


https://reviews.llvm.org/D47991



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

Reply via email to