Most changes to Pythonrequire a NEWS entry.

Please add it using theblurb_it web app or theblurb command-line tool.

All commit authors signed the Contributor License Agreement.

Most changes to Pythonrequire a NEWS entry.

Please add it using theblurb_it web app or theblurb command-line tool.

@iritkatriel I am wondering if you could be an appropriate reviewer for this. Could you please take a look?

Thanks for adding the test. Before merging this I would need to spend some time to understand why this happened and to convince myself that this is the right solution. If someone else reviews it before I get around to it that's obviously fine.

@iritkatriel Thank you for your quick response.

I have investigated the issue further and discovered that the problematicobj.f_globals is obj.f_locals check was introduced in5ed2b5a to work around the limitation ofinspect.getsourcelines, as the latter did not check that the given frame is a module-level frame.

However,inspect.getsourcelines was independently corrected in91cb298 using the sameobject.f_code.co_name == "<module>" technique.

So I believe that we should consider replacing the custompdb.getsourcelines function with directinspect.getsourcelines invocations. After making this change, the updatedtest_pdb still passes.@gaogaotiantian WDYT?

Yeah I think that’s the way to go. The original customized pdb version was to avoid module level frame issue and it would be nice to not to keep multiple identical code pieces around. The functionality seems to be the same. I’ll make the changes to use inspect version directly and remove the pdb version.

As@ortem said, the reason we had a separategetsourcelines in pdb was because theinspect version did not deal with the module level frame. It does now so we can just switch back toinspect version and kept one less duplicated code piece. I did include the regression test for the issue.

Thanks@gaogaotiantian for the PR, and@iritkatriel for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒⛏🤖

Sorry,@gaogaotiantian and@iritkatriel, I could not cleanly backport this to3.10 due to a conflict.
Please backport usingcherry_picker on command line.
cherry_picker 5d677c556f03a34d1c2d86e4cc96025870c20c12 3.10

GH-102632 is a backport of this pull request to the3.11 branch.

GH-102633 is a backport of this pull request to the3.10 branch.