Re: [Lldb-commits] [PATCH] D15582: Improve error handling for `f s #`

2016-09-19 Thread Eugene Zelenko via lldb-commits
Eugene.Zelenko added a subscriber: Eugene.Zelenko. Eugene.Zelenko closed this revision. Eugene.Zelenko added a comment. Committed in r256251. Please specify Differential revision in commit message. https://reviews.llvm.org/D15582 ___ lldb-commits ma

Re: [Lldb-commits] [PATCH] D15582: Improve error handling for `f s #`

2015-12-17 Thread Ed Maste via lldb-commits
emaste accepted this revision. emaste added a comment. This revision is now accepted and ready to land. LGTM Or if you want show the set of arguments - e.g. ```expected frame-index, got 's 1'``` That might clarify the "too many arguments." http://reviews.llvm.org/D15582 _

[Lldb-commits] [PATCH] D15582: Improve error handling for `f s #`

2015-12-16 Thread Adrian McCarthy via lldb-commits
amccarth created this revision. amccarth added reviewers: emaste, jasonmolenda. amccarth added a subscriber: lldb-commits. When there were too many arguments given for the `frame select` command, the error message wasn't particularly helpful and then lldb would try to select frame -1, which resu