Message 294282 - 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	yselivanov
Recipients	The Compiler, cjw296, michael.foord, ncoghlan, pstch, takluyver, yselivanov
Date	2017-05-23.20:18:16
SpamBayes Score	-1.0
Marked as misclassified	Yes
Message-id	<1495570696.98.0.690318822325.issue25532@psf.upfronthosting.co.za>
In-reply-to

Content
> I'd be fine with replacing the sys.getrecursionlimit() call with a module level constant like "_UNWRAP_LIMIT = 500". TBH I'm OK either way. `sys.getrecursionlimit()` is 1000 (at least on my machine), which might be a bit too much for this specific use-case. It's also unlikely that someone will set recursion limit to less than 100 (or too many things would break), so we are probably safe here. So I'm +0.5 on using _UNWRAP_LIMIT; if you feel the same way too we should do that, otherwise let's leave it as is.

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

TBH I'm OK either way.

`sys.getrecursionlimit()` is 1000 (at least on my machine), which might be a bit too much for this specific use-case.

It's also unlikely that someone will set recursion limit to less than 100 (or too many things would break), so we are probably safe here.

So I'm +0.5 on using _UNWRAP_LIMIT; if you feel the same way too we should do that, otherwise let's leave it as is.

History
Date	User	Action	Args
2017-05-23 20:18:17	yselivanov	set	recipients: + yselivanov, ncoghlan, cjw296, michael.foord, takluyver, The Compiler, pstch
2017-05-23 20:18:16	yselivanov	set	messageid: <1495570696.98.0.690318822325.issue25532@psf.upfronthosting.co.za>
2017-05-23 20:18:16	yselivanov	link	issue25532 messages
2017-05-23 20:18:16	yselivanov	create