labath accepted this revision. labath added inline comments. This revision is now accepted and ready to land.
================ Comment at: lldb/source/Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.cpp:1229-1230 + Locker locker(this, + ScriptInterpreterPythonImpl::Locker::AcquireLock | + ScriptInterpreterPythonImpl::Locker::InitSession | + (options.GetSetLLDBGlobals() ---------------- The `ScriptInterpreterPythonImpl` qualification on the `Locker` class is probably overkill. (I know it was present before this patch, but it's overkill there too :P ). CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59976/new/ https://reviews.llvm.org/D59976 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
