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

Reply via email to