jingham accepted this revision.
jingham added a comment.
This revision is now accepted and ready to land.

Thanks for these cleanups.  In actual fact, the values set for options in the 
constructor for CommandOptions are never used, only the values in 
OptionParsingStarted matter.  But it's still confusing to have them be wrong...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123204

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

Reply via email to