[Lldb-commits] [PATCH] D47991: Improve SBThread's stepping API using SBError parameter.

[Jim Ingham via Phabricator via lldb-commits]

jingham added a comment.

The SB API's tend to take SBError as an in/out parameter for the cases where the function in question naturally returns some other value, and return an SBError when there's no other logical return value.

So in these cases it would be more in line with the general practice to return an SBError.  OTOH, I really don't like the practice of adding "Ex" or other similar function name decorations to get around the fact that C++ doesn't overload on return values.  IMO they make the API harder to remember.  And especially since we really should use the error version, so you will end up with the primary function having this weird tag as part of the name...

In balance I think adding it as an argument is better, but this is one of those cases where I would leave it to the taste of the implementor...

Jim


https://reviews.llvm.org/D47991