deleting a class-scoped variable that shadows a global variable causes mypy to think it's undefined
Offen
Dieses Issue hat noch niemand übernommen.
bug
topic-runtime-semantics
topic-variable-scope
- Vorherrschende Sprache
- Python
- Sterne
- 20.6k
- Forks
- 3.3k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
To Reproduce
from typing import Callable, TypeVar
T = TypeVar("T")
x = 1
def decorate(num: int) -> Callable[[T], T]:
def thru(it: T) -> T:
print("decorating", it, "with", num)
return it
return thru
class y(object):
@decorate(x)
def a(self) -> None:
...
x = 2
@decorate(x)
def b(self) -> None:
...
del x
@decorate(x)
def c(self) -> None:
...
Expected Behavior
This is perfectly valid, it should not cause an error! It works at runtime.
Actual Behavior
$ mypy class_scope.py
class_scope.py:28: error: Trying to read deleted variable 'x'
Found 1 error in 1 file (checked 1 source file)
Your Environment
- Mypy version used: 0.790
- Mypy command-line flags: all defaults
- Mypy configuration options from
mypy.ini(and other config files): no mypy.ini - Python version used: Python 3.8.5
- Operating system and version:
ProductName: Mac OS X
ProductVersion: 10.15.7
BuildVersion: 19H2
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Reproduziere das Problem aus dem Beispiel class_scope.py mit mypy und verwende die gemeldeten Python- und mypy-Versionen als Ausgangspunkt. Verfolge, wie der Checker einen klassenbezogenen Namen nach del x behandelt, und füge anschließend einen Regressionstest hinzu, der zeigt, dass die spätere Verwendung des Decorators keine nicht definierte Variable meldet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- devtools
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100