[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