[Lldb-commits] [PATCH] D61090: [SBHostOS} Remove getting the python script interpreter path

Greg Clayton via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu Apr 25 10:50:56 PDT 2019


clayborg added a comment.

I agree with all that was said above.

My reason for leaving it in SBHostOS is due to SBCommandInterpreter not being the right place, since we don't have SBScriptInprereter in the API, and we might as well leave it in SBHostOS until we are going to do it right by having a static call on SBScriptInterpreter that takes a enumeration from lldb-enumerations.h with a list of script interpreter languages that could be supported. Right now we just have python, so no need to add that functionality yet.


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

https://reviews.llvm.org/D61090





More information about the lldb-commits mailing list