Crash while hashing a callable type

Offen
#4,601 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
25/100
Issue-Typ
Bug
Klarheit
Muss geklärt werden
Aktivitätsstatus
Veraltet
Tech-Stack
python
Bereich
compilers

Rechercherichtung

Start with the traceback locations in mypy/build.py, especially finish_passes and _patch_indirect_dependencies, then inspect the hash and is_type_obj implementations in mypy/types.py. Reproduce the crash if possible and determine how a callable type reaches hashing with a missing fallback. Done means the crash is prevented and the regression is covered by a test.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

needs discussion refactoring

Some time ago there appeared a crash ending in:

...
File "[somewhere]/mypy/build.py", line 1954, in finish_passes
    self.type_map())
File "[somewhere]/mypy/build.py", line 1964, in _patch_indirect_dependencies
    types = set(type_map.values())
File "[somewhere]/mypy/types.py", line 938, in __hash__
    return hash(tuple(self.items()))
File "[somewhere]/mypy/types.py", line 839, in __hash__
    return hash((self.ret_type, self.is_type_obj(),
File "[somewhere]/mypy/types.py", line 723, in is_type_obj
    return self.fallback.type.is_metaclass()
AttributeError: 'NoneType' object has no attribute 'type'

There is no repro yet. There however may be a point for a wider discussion: currently most types in mypy are hashable (they have both __hash__ and __eq__) while they are also mutable. Recently there was a discussion on Python-Dev list and some people said this is a bad pattern. I think this is OK in our case (we use hashes to cache is_subtype results).

Vorherrschende Sprache
Python
Sterne
20.6k
Forks
3.3k
Ø Merge
1 T. 18 Std.
Gemergte PRs (30 T.)
54

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus python/mypy

Alle Issues in python/mypy

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

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