Look into handling potential OOM errors from recursive implementation of BUILD_LIST_UNPACK

Question

Look into handling potential OOM errors from recursive implementation of BUILD_LIST_UNPACK

brettcannon opened this issue 9 years ago · 2 comments

#141 implemented BUILD_LIST_UNPACK using recursion. While fine, it creates a lot of locals which could trigger a OOM error in crazy edge cases. It wouldn't hurt to avoid that situation if possible.

Answer 1 · 2016-02-22T22:24:16.000Z

There should also be a way to avoid storing the temp locally by simply pushing back on to the stack the object being extended and swapping the order of PyJist_ExtendList().

Answer 2 · 2020-11-11T05:08:26.000Z

Implemented in #237
Dropped the recursive implementation in favour of a stack iterator