[issue13546] sys.setrecursionlimit() crashes IDLE

Amaury Forgeot d'Arc added the comment: 'e' is "used" in the comment above: "isinstance(e, ValueError) used to fail"... I agree to use the modern 'as' syntax. -- ___ Python tracker

[issue13546] sys.setrecursionlimit() crashes IDLE

Mark Dickinson added the comment: Style nit: how about 'except ValueError as e' instead of 'except ValueError, e' (Actually, why do we need e?) -- nosy: +mark.dickinson ___ Python tracker

[issue13546] sys.setrecursionlimit() crashes IDLE

Amaury Forgeot d'Arc added the comment: This will be fixed in the next 2.7 release, thanks for the report! -- resolution: -> fixed stage: patch review -> committed/rejected status: open -> closed ___ Python tracker

[issue13546] sys.setrecursionlimit() crashes IDLE

Roundup Robot added the comment: New changeset 57de1ad15c54 by Amaury Forgeot d'Arc in branch '2.7': Issue #13546: Fixed an overflow issue that could crash the intepreter when http://hg.python.org/cpython/rev/57de1ad15c54 -- nosy: +python-dev ___ Pyt

[issue13546] sys.setrecursionlimit() crashes IDLE

Ned Deily added the comment: LGTM -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/ma

[issue13546] sys.setrecursionlimit() crashes IDLE

Changes by Amaury Forgeot d'Arc : -- assignee: -> amaury.forgeotdarc stage: needs patch -> patch review ___ Python tracker ___ ___ Py

[issue13546] sys.setrecursionlimit() crashes IDLE

Amaury Forgeot d'Arc added the comment: Here is a patch, with test. -- keywords: +patch Added file: http://bugs.python.org/file23868/issue13546.patch ___ Python tracker ___

[issue13546] sys.setrecursionlimit() crashes IDLE

Amaury Forgeot d'Arc added the comment: Tested on Linux: Python 2.7.2+ (2.7:16c4137a413c+, Dec 4 2011, 22:56:38) [GCC 4.4.3] on linux2 >>> import sys >>> sys.setrecursionlimit((1<<31)-1) >>> import xx Exception RuntimeError: 'maximum recursion depth exceeded in __subclasscheck__' in ignored

[issue13546] sys.setrecursionlimit() crashes IDLE

Ji Han added the comment: @amaury.forgeotdarc It says 'python.exe has stopped working'. Details: Problem Event Name: APPCRASH Application Name: python.exe Application Version: 0.0.0.0 Application Timestamp:4df4b010 Fault Module Name:MSVCR90.dll Fault Module Versi

[issue13546] sys.setrecursionlimit() crashes IDLE

Ned Deily added the comment: But after setting it, IDLE is going to be executing code. In the 64-bit Windows Python 2.7.2 that I have available, there were exceptions reported in the command line interpreter (not IDLE) after changing the recursion limit and then executing some code. Perhaps

[issue13546] sys.setrecursionlimit() crashes IDLE

Amaury Forgeot d'Arc added the comment: hanji, could you start IDLE from a command prompt and try again: c:\python27\python.exe -m idlelib.idle Do you see additional error messages? -- ___ Python tracker

[issue13546] sys.setrecursionlimit() crashes IDLE

Amaury Forgeot d'Arc added the comment: On the other hand, there is no reason for the recursion limit to be actually reached, just by setting it! Is there a hidden infinite recursion somewhere? -- nosy: +amaury.forgeotdarc ___ Python tracker

[issue13546] sys.setrecursionlimit() crashes IDLE

Ned Deily added the comment: Trying to set the recursion limit to a large number defeats its purpose. As documented in the Standard Library Reference: The highest possible limit is platform-dependent. A user may need to set the limit higher when she has a program that requires deep recursion

[issue13546] sys.setrecursionlimit() crashes IDLE

New submission from Ji Han : The following code snippet will crash IDLE: >>> import sys >>> sys.setrecursionlimit((1<<31)-1) The crash happens immediately and is consistently reproducible (environment: Windows 7 SP1 64-bit, Python 2.7.2 for Windows X86_64). -- components: None message