diff options
author | Nick Coghlan <ncoghlan@gmail.com> | 2012-11-04 14:19:18 (GMT) |
---|---|---|
committer | Nick Coghlan <ncoghlan@gmail.com> | 2012-11-04 14:19:18 (GMT) |
commit | 10d6ddeef19a69ab9a73e64a3a9a82c50a4160a2 (patch) | |
tree | 3c6d41e21074d93518346c84a7976b131b79e02f | |
parent | aab9c2b2ead7b786947c367a9cc5d9c921b9ea99 (diff) | |
download | cpython-10d6ddeef19a69ab9a73e64a3a9a82c50a4160a2.zip cpython-10d6ddeef19a69ab9a73e64a3a9a82c50a4160a2.tar.gz cpython-10d6ddeef19a69ab9a73e64a3a9a82c50a4160a2.tar.bz2 |
Issue #5765: Also check the compiler when finding the recursion limit
-rwxr-xr-x | Tools/scripts/find_recursionlimit.py | 7 |
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 |