contextlib: Use a list instead of deque for exit callbacks.

dpgeorge · dpgeorge · commit 469b81b567b9 · 2024-06-17T11:18:48.000+10:00
Since deque was removed from this repository the built-in one needs to be
used, and that doesn't have unbounded growth.  So use a list instead, which
is adequate becasue contextlib only needs append and pop, not double ended
behaviour (the previous pure-Python implementation of deque that was used
here anyway used a list as its storage container).

Also tweak the excessive-nesting test so it uses less memory and can run on
the unix port.

Signed-off-by: Damien George &lt;damien@micropython.org&gt;
diff --git a/python-stdlib/contextlib/contextlib.py b/python-stdlib/contextlib/contextlib.py
@@ -85,13 +85,13 @@ class ExitStack(object):
     """
 
     def __init__(self):
-        self._exit_callbacks = deque()
+        self._exit_callbacks = []
 
     def pop_all(self):
         """Preserve the context stack by transferring it to a new instance"""
         new_stack = type(self)()
         new_stack._exit_callbacks = self._exit_callbacks
-        self._exit_callbacks = deque()
+        self._exit_callbacks = []
         return new_stack
 
     def _push_cm_exit(self, cm, cm_exit):
diff --git a/python-stdlib/contextlib/manifest.py b/python-stdlib/contextlib/manifest.py
@@ -1,4 +1,4 @@
-metadata(description="Port of contextlib for micropython", version="3.4.3")
+metadata(description="Port of contextlib for micropython", version="3.4.4")
 
 require("ucontextlib")
 require("collections")
diff --git a/python-stdlib/contextlib/tests.py b/python-stdlib/contextlib/tests.py
@@ -399,7 +399,7 @@ def test_exit_exception_chaining_suppress(self):
     def test_excessive_nesting(self):
         # The original implementation would die with RecursionError here
         with ExitStack() as stack:
-            for i in range(10000):
+            for i in range(5000):
                 stack.callback(int)
 
     def test_instance_bypass(self):

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-metadata(description="Port of contextlib for micropython", version="3.4.3")`
	`1`	`+metadata(description="Port of contextlib for micropython", version="3.4.4")`
`2`	`2`
`3`	`3`	`require("ucontextlib")`
`4`	`4`	`require("collections")`