[Lldb-commits] [PATCH] Fixup some issues with Python shutdown / finalization
Enrico Granata
egranata at apple.com
Fri Aug 8 13:23:12 PDT 2014
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 <zturner at google.com> 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
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
- Enrico
📩 egranata@.com ☎️ 27683
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20140808/82ffef1c/attachment.html>
More information about the lldb-commits
mailing list