Message 294255 - Python tracker

➜

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	<[email protected]>
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.

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:46	ncoghlan	set	recipients: + ncoghlan, cjw296, michael.foord, takluyver, yselivanov, The Compiler, pstch
2017-05-23 15:28:46	ncoghlan	set	messageid: <[email protected]>
2017-05-23 15:28:46	ncoghlan	link	issue25532 messages
2017-05-23 15:28:46	ncoghlan	create