➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author ncoghlan
Recipients The Compiler, cjw296, michael.foord, ncoghlan, pstch, takluyver, yselivanov
Date 2017-05-23.15:28:46
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1495553326.6.0.935472055379.issue25532@psf.upfronthosting.co.za>
In-reply-to
Content 
Ah, sorry - I merged the PR before seeing the discussion about the limit over here.

I'd be fine with replacing the sys.getrecursionlimit() call with a module level constant like "_UNWRAP_LIMIT = 500".

If someone desperately needed a higher limit for some reason, they could still poke around and change that from outside the module, even if doing so wasn't officially supported.

That wouldn't need a new NEWS entry, since it would just be an amendment to the existing patch.
History
Date User Action Args
2017-05-23 15:28:46ncoghlansetrecipients: + ncoghlan, cjw296, michael.foord, takluyver, yselivanov, The Compiler, pstch
2017-05-23 15:28:46ncoghlansetmessageid: <1495553326.6.0.935472055379.issue25532@psf.upfronthosting.co.za>
2017-05-23 15:28:46ncoghlanlinkissue25532 messages
2017-05-23 15:28:46ncoghlancreate