Nick Coghlan <ncogh...@gmail.com> added the comment:

Reviewing the PR post-merge, I'm pretty sure this has introduced a 
double-decrement bug due to the original code being hard to read (the error 
cases did the decrement inside the helper function, while the success case did 
it in the calling function).

https://github.com/python/cpython/pull/15593 builds on your PR by removing the 
recursion counter adjustments from inside the helper function, leaving only the 
ones in the calling function.

----------
nosy: +ncoghlan

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

Reply via email to