PEP 667: correct/clarify handling of PyEval_GetLocals · Issue #245 · python/steering-council (original) (raw)

While writing the docs for python/cpython#74929 and reviewing python/cpython#118934, I discovered a discrepancy in the way PEP 667 described the updates affecting PyEval_GetLocals (what the text described didn't match the proposed implementation given).

After discussing it with @gaogaotiantian in python/cpython#118934 (comment) I'm happy that the discrepancy should be resolved in favour of the proposed implementation, but I'd also like the SC to ratify that since we're arguably changing a user-facing detail of an accepted PEP.

python/peps#3809 updates the PEP text to resolve the inconsistency (since it's referenced from the Python 3.13 What's New and leaving it as is might be confusing for users of PyEval_GetLocals)

Edit: PEP PR moved to python/peps#3845