[Python-checkins] gh-95913: make the new internal classes pdb.ModuleTarget/ScriptTarget private (GH-96053)
iritkatriel
webhook-mailer at python.org
Thu Aug 18 06:16:44 EDT 2022
https://github.com/python/cpython/commit/1a720c600391f4076097f2ab667e9521c8e521fa
commit: 1a720c600391f4076097f2ab667e9521c8e521fa
branch: main
author: Irit Katriel <1055913+iritkatriel at users.noreply.github.com>
committer: iritkatriel <1055913+iritkatriel at users.noreply.github.com>
date: 2022-08-18T11:16:07+01:00
summary:
gh-95913: make the new internal classes pdb.ModuleTarget/ScriptTarget private (GH-96053)
files:
M Lib/pdb.py
M Lib/test/test_pyclbr.py
diff --git a/Lib/pdb.py b/Lib/pdb.py
index e6ed814acbe..b0acb1f571d 100755
--- a/Lib/pdb.py
+++ b/Lib/pdb.py
@@ -131,7 +131,7 @@ def __repr__(self):
return self
-class ScriptTarget(str):
+class _ScriptTarget(str):
def __new__(cls, val):
# Mutate self to be the "real path".
res = super().__new__(cls, os.path.realpath(val))
@@ -167,7 +167,7 @@ def code(self):
return f"exec(compile({fp.read()!r}, {self!r}, 'exec'))"
-class ModuleTarget(str):
+class _ModuleTarget(str):
def check(self):
try:
self._details
@@ -1625,7 +1625,7 @@ def lookupmodule(self, filename):
return fullname
return None
- def _run(self, target: Union[ModuleTarget, ScriptTarget]):
+ def _run(self, target: Union[_ModuleTarget, _ScriptTarget]):
# When bdb sets tracing, a number of call and line events happen
# BEFORE debugger even reaches user's code (and the exact sequence of
# events depends on python version). Take special measures to
@@ -1789,7 +1789,7 @@ def main():
commands = [optarg for opt, optarg in opts if opt in ['-c', '--command']]
module_indicated = any(opt in ['-m'] for opt, optarg in opts)
- cls = ModuleTarget if module_indicated else ScriptTarget
+ cls = _ModuleTarget if module_indicated else _ScriptTarget
target = cls(args[0])
target.check()
diff --git a/Lib/test/test_pyclbr.py b/Lib/test/test_pyclbr.py
index b2de4e8397d..23453e34015 100644
--- a/Lib/test/test_pyclbr.py
+++ b/Lib/test/test_pyclbr.py
@@ -229,7 +229,7 @@ def test_others(self):
cm(
'pdb',
# pyclbr does not handle elegantly `typing` or properties
- ignore=('Union', 'ModuleTarget', 'ScriptTarget'),
+ ignore=('Union', '_ModuleTarget', '_ScriptTarget'),
)
cm('pydoc', ignore=('input', 'output',)) # properties
More information about the Python-checkins
mailing list