jingham added a comment.

Looks good to me too.  The fact that you were mostly eliminating redirections 
from CommandInterpreter -> Debugger is good evidence this was wrongly 
structured initially.

I'm a little curious that we do:

  m_debugger.GetCommandInterpreter().GetPythonCommandsFromIOHandler

to read "break command add -s python" command text.  Seems like the script 
interpreter IO handler should do this otherwise the Command Interpreter has to 
know the right way to do input for every script interpreter.  But I think 
that's a change for another patch.

This one seems pure goodness.


Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D61172



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

Reply via email to