python / python/mypy

Platform check is not remembered after `return`

Offen
#14,940 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

feature topic-type-narrowing
Vorherrschende Sprache
Python
Sterne
20.6k
Forks
3.3k
PR-Merge-Kennzahlen
PR-Kennzahlen ausstehend

Beschreibung

Bug Report

A platform check that is working fine in if and else is not working after if ... return.

To Reproduce

import sys

def works() -> None:
    if sys.platform != "win32":
        return
    else:
        from winreg import REG_SZ

def fails() -> None:
    if sys.platform != "win32":
        return
    from winreg import REG_SZ

https://mypy-play.net/?mypy=latest&python=3.11&gist=eddad091a6082bd85c5c7524839726b1

Expected Behavior

No error.

Actual Behavior

main.py:12: error: Module "winreg" has no attribute "REG_SZ" [attr-defined]
Found 1 error in 1 file (checked 1 source file)

Your Environment

  • Failure is on Linux
  • Mypy version used: 1.1.1
  • Mypy command-line flags: none
  • Mypy configuration options from mypy.ini (and other config files): none
  • Python version used: 3.11.2

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit dem bereitgestellten Python-Reproducer und dem mypy-play-Link und verfolge dann, wie Plattformprüfungen und vorzeitige Rückgaben die Typverengung beeinflussen. Vergleiche die Fälle if/else und if ... return; abgeschlossen ist die Aufgabe, wenn die zweite Form keinen falschen Attributfehler für winreg.REG_SZ mehr meldet.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
compilers
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.