[issue3850] find_recursion_limit.py is broken

2008-09-13 Thread Antoine Pitrou
Antoine Pitrou <[EMAIL PROTECTED]> added the comment: Bug fixed and module comments updated in r66457. -- resolution: -> fixed status: open -> closed ___ Python tracker <[EMAIL PROTECTED]> __

[issue3850] find_recursion_limit.py is broken

2008-09-13 Thread Antoine Pitrou
Antoine Pitrou <[EMAIL PROTECTED]> added the comment: > On my Macbook, it segfaults; perhaps that docstring applied only > to MacOS 9? Well, I just tried under Windows and the interpreter neither segfaults nor prints a MemoryError, it aborts silently. I guess the comment is either out of date o

[issue3850] find_recursion_limit.py is broken

2008-09-12 Thread A.M. Kuchling
A.M. Kuchling <[EMAIL PROTECTED]> added the comment: The patch seems fine to me. The docstring at the top of the file says: It ends when Python causes a segmentation fault because the limit is too high. On platforms like Mac and Windows, it should exit with a MemoryError. On my Macbook, it se

[issue3850] find_recursion_limit.py is broken

2008-09-12 Thread Antoine Pitrou
Antoine Pitrou <[EMAIL PROTECTED]> added the comment: It's not my code, it's the interpreter's code which uses PyDict_GetItem() all over the place, and the aim of find_recursion_limit.py is precisely to test the function call paths inside the interpreter. ___

[issue3850] find_recursion_limit.py is broken

2008-09-12 Thread Alexandre Vassalotti
Alexandre Vassalotti <[EMAIL PROTECTED]> added the comment: Could you use PyDict_GetItemWithError() to avoid this? -- nosy: +alexandre.vassalotti ___ Python tracker <[EMAIL PROTECTED]> ___

[issue3850] find_recursion_limit.py is broken

2008-09-12 Thread Antoine Pitrou
Antoine Pitrou <[EMAIL PROTECTED]> added the comment: Here is a patch. -- keywords: +needs review, patch Added file: http://bugs.python.org/file11480/find_recursion_limit.patch ___ Python tracker <[EMAIL PROTECTED]>

[issue3850] find_recursion_limit.py is broken

2008-09-12 Thread Antoine Pitrou
New submission from Antoine Pitrou <[EMAIL PROTECTED]>: find_recursion_limit.py in trunk is broken: it fails with an AttributeError while a RuntimeError is expected. This has appeared due to the recent changes in recursion limit handling, but the problem is deeper. As I explained on the ML, funct