[issue27514] SystemError when compiling deeply nested for loops

2016-07-14 Thread Roundup Robot
Roundup Robot added the comment: New changeset e6e7c8368c70 by Benjamin Peterson in branch '3.5': make too many nested blocks be a SyntaxError instead of a SystemError (closes #27514) https://hg.python.org/cpython/rev/e6e7c8368c70 New changeset 345ec7455b75 by Benjamin Peterson in branch

[issue27514] SystemError when compiling deeply nested for loops

2016-07-14 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: LGTM. -- stage: needs patch -> commit review ___ Python tracker ___ ___

[issue27514] SystemError when compiling deeply nested for loops

2016-07-14 Thread Ammar Askar
Ammar Askar added the comment: Amended patch with the comment fixed to refer to just the issue numbers according to advice from bitdancer. -- Added file: http://bugs.python.org/file43726/nested_blocks.diff2 ___ Python tracker

[issue27514] SystemError when compiling deeply nested for loops

2016-07-14 Thread Ammar Askar
Ammar Askar added the comment: This patch changes it to a SyntaxError instead of a SystemError. -- keywords: +patch nosy: +ammar2 Added file: http://bugs.python.org/file43725/nested_blocks.diff ___ Python tracker

[issue27514] SystemError when compiling deeply nested for loops

2016-07-14 Thread Nick Coghlan
Nick Coghlan added the comment: +1 for Serhiy's recommendation. It's OK for the compiler to say "Don't do that", but the correct error is SyntaxError, even when it's a limitation later in the code generation pipeline (rather than failing to comply with the language grammar) --

[issue27514] SystemError when compiling deeply nested for loops

2016-07-14 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: There is a static limit for the number of statically nested blocks. Getting rid of this limit looks not easy. But SystemError is not an exception that should be raised. SystemError is for errors that can't be occurred in normal case. It should only be

[issue27514] SystemError when compiling deeply nested for loops

2016-07-14 Thread ppperry
New submission from ppperry: The following code: for a in range(26): for b in range(26): for c in range(26): for d in range(26): for e in range(26): for f in range(26): for g in range(26): for h in range(26): for i in range(26): for j in