teemperor requested changes to this revision.
teemperor added a comment.

So CommandObjectRaw does support arguments, but they way it works is that you 
need to have a delimiter for the 'raw' part which is `--`. If you have that, 
then you can just use invoke the normal option parsing like 
CommandObjectExpression does. The handcrafted implementation here adds a 
completely new lldb command syntax which has a 'raw' part without the `--` 
which seems inconsistent (not saying that the syntax here is worse or better 
than `script --language foo --`, but it's just inconsistent with the way the 
rest of LLDB works). It's obviously also missing the other quirky things that 
are part of the command syntax (e.g., quoting arguments isn't supported and so 
on).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86996/new/

https://reviews.llvm.org/D86996

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

Reply via email to