clayborg added inline comments.
================ Comment at: lldb/source/API/SBDebugger.cpp:1190 + options.SetAutoHandleEvents(auto_handle_events); + options.SetSpawnThread(spawn_thread); CommandInterpreter &interp = m_opaque_sp->GetCommandInterpreter(); ---------------- I agree this is a good idea to do in the header doc. Also might be worth marking this and other functions RunCommandInterpreter as deprecated and point them to the new way of doing things once that patch is in where everything is in SBCommandInterpreterRunOptions and SBCommandInterpreterRunResults CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79108/new/ https://reviews.llvm.org/D79108 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits