[Lldb-commits] [PATCH] D49207: Get rid of the C-string parameter in DoExecute

Raphael Isemann via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Jul 12 10:05:02 PDT 2018


teemperor added a comment.

@jingham I thought the same at first, but giving an error `command.empty()` turns out to be a problem. The command string contains the arguments (but not the command name itself).  So when calling for example `bt` we would hit this error (as there are no args, so `command` is empty). I think this code was written with the assumption that `command` contains the whole command line or something like that.


https://reviews.llvm.org/D49207





More information about the lldb-commits mailing list