summaryrefslogtreecommitdiffstats
path: root/Tools
diff options
context:
space:
mode:
authorNick Coghlan <ncoghlan@gmail.com>2012-11-04 14:19:18 (GMT)
committerNick Coghlan <ncoghlan@gmail.com>2012-11-04 14:19:18 (GMT)
commit10d6ddeef19a69ab9a73e64a3a9a82c50a4160a2 (patch)
tree3c6d41e21074d93518346c84a7976b131b79e02f /Tools
parentaab9c2b2ead7b786947c367a9cc5d9c921b9ea99 (diff)
downloadcpython-10d6ddeef19a69ab9a73e64a3a9a82c50a4160a2.zip
cpython-10d6ddeef19a69ab9a73e64a3a9a82c50a4160a2.tar.gz
cpython-10d6ddeef19a69ab9a73e64a3a9a82c50a4160a2.tar.bz2
Issue #5765: Also check the compiler when finding the recursion limit
Diffstat (limited to 'Tools')
-rwxr-xr-xTools/scripts/find_recursionlimit.py7
1 files changed, 7 insertions, 0 deletions
diff --git a/Tools/scripts/find_recursionlimit.py b/Tools/scripts/find_recursionlimit.py
index 7a86603..c42de7f 100755
--- a/Tools/scripts/find_recursionlimit.py
+++ b/Tools/scripts/find_recursionlimit.py
@@ -89,6 +89,12 @@ def test_cpickle(_cache={}):
_pickle.Pickler(io.BytesIO(), protocol=-1).dump(l)
_cache[n] = l
+def test_compiler_recursion():
+ # The compiler uses a scaling factor to support additional levels
+ # of recursion. This is a sanity check of that scaling to ensure
+ # it still throws RuntimeError even at higher recursion limits
+ compile("()" * (10 * sys.getrecursionlimit()), "<single>", "single")
+
def check_limit(n, test_func_name):
sys.setrecursionlimit(n)
if test_func_name.startswith("test_"):
@@ -117,5 +123,6 @@ if __name__ == '__main__':
check_limit(limit, "test_getattr")
check_limit(limit, "test_getitem")
check_limit(limit, "test_cpickle")
+ check_limit(limit, "test_compiler_recursion")
print("Limit of %d is fine" % limit)
limit = limit + 100