Pablo Galindo Salgado <pablog...@gmail.com> added the comment:

> We could also adjust Parser/asdl_c.py to decref right after a failed 
> _PyArena_AddPyObject() call, instead of goto failure and Py_XDECREF. I'm not 
> sure it's worth it though.

I normally recommend having the least amount of exist paths possible, makes 
reasoning easily normally at the cost of some extra NULL checks for the XDECREF 
in failure scenarios

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<https://bugs.python.org/issue43779>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to