Do we still get crashes because of the usage of atexit(), or did you simply not revert the changes to dotest.py?
With the exception of those, the rest of your patch LGTM > On Aug 8, 2014, at 11:19 AM, Zachary Turner <[email protected]> wrote: > > Got rid of the Release() method and wrapped the destructor-based decrefs > inside of Py_IsInitialized() checks. > > http://reviews.llvm.org/D4826 > > Files: > include/lldb/Interpreter/PythonDataObjects.h > include/lldb/Interpreter/ScriptInterpreterPython.h > source/Interpreter/ScriptInterpreterPython.cpp > test/dotest.py > <D4826.12310.patch>_______________________________________________ > lldb-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits - Enrico 📩 egranata@.com ☎️ 27683
_______________________________________________ lldb-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
