Repository navigation
Commit f906522
authored
Introduced in GH-23469 (bpo-26131, "Deprecate usage of load_module()")
to render an object's qualified name inside the load_module()
deprecation warnings.
Orphaned by gh-142205 (GH-97850, "Remove all uses and definitions of
load_module() from importlib"), which deleted the warning-building call
sites f"{_object_name(spec.loader)}.exec_module() not found; ..." and
left the helper with no caller.
A word-boundary search across Lib, Modules, Python, Objects and Include
finds zero references outside its own definition, and a GitHub code
search finds no downstream importers. The frozen importlib was
regenerated; importlib._bootstrap._object_name no longer exists at
runtime and the full test_importlib suite passes.
1 parent d4a8066 commit f906522
2 files changed
Lines changed: 4 additions & 6 deletions
File tree
- Lib/importlib
- Misc/NEWS.d/next/Library
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -20,12 +20,6 @@ | |||
| 20 | 20 | # reference any injected objects! This includes not only global code but also | |
| 21 | 21 | # anything specified at the class level. | |
| 22 | 22 | ||
| 23 | - def _object_name(obj): | ||
| 24 | - try: | ||
| 25 | - return obj.__qualname__ | ||
| 26 | - except AttributeError: | ||
| 27 | - return type(obj).__qualname__ | ||
| 28 | - | ||
| 29 | 23 | # Bootstrap-related code ###################################################### | |
| 30 | 24 | ||
| 31 | 25 | # Modules injected manually by _setup() | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -0,0 +1,4 @@ | |||
| 1 | + Remove the private, undocumented function | ||
| 2 | + ``importlib._bootstrap._object_name()``. It had no caller after | ||
| 3 | + ``load_module()`` and its deprecation warnings were removed from | ||
| 4 | + :mod:`importlib`. | ||
0 commit comments