[Lldb-commits] [PATCH] D59976: [Python] Remove Python include from ScriptInterpreterPython.h

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Mar 29 13:03:12 PDT 2019


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





More information about the lldb-commits mailing list