]> granicus.if.org Git - python/commit
bpo-18533: Avoid RecursionError from repr() of recursive dictview (GH-4823)
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Mon, 26 Feb 2018 14:42:00 +0000 (06:42 -0800)
committerGitHub <noreply@github.com>
Mon, 26 Feb 2018 14:42:00 +0000 (06:42 -0800)
commitfbf7aac36bd1017bc87964b5d17dce0e101ff2d6
tree0b0cf881f870040087cb51cbbf4e614058912394
parent5a07608d0855e4104e4070328155b52010dec4e7
bpo-18533: Avoid RecursionError from repr() of recursive dictview (GH-4823)

dictview_repr(): Use a Py_ReprEnter() / Py_ReprLeave() pair to check
for recursion, and produce "..." if so.

test_recursive_repr(): Check for the string rather than a
RecursionError.  (Test cannot be any tighter as contents are
implementation-dependent.)

test_deeply_nested_repr(): Add new test, replacing the original
test_recursive_repr().  It checks that a RecursionError is raised in
the case of a non-recursive but deeply nested structure.  (Very
similar to what test_repr_deep() in test/test_dict.py does for a
normal dict.)

OrderedDictTests: Add new test case, to test behavior on OrderedDict
instances containing their own values() or items().
(cherry picked from commit d7773d92bd11640a8c950d6c36a9cef1cee36f96)
Lib/test/test_dictviews.py
Lib/test/test_ordered_dict.py
Misc/NEWS.d/next/Core and Builtins/2017-12-13-16-46-23.bpo-18533.Dlk8d7.rst [new file with mode: 0644]
Objects/dictobject.c