Consider cleanup of type for iterator variables bounded by asdl_seq_LEN

Offen
#99,652 1 Kommentar 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
35/100
Issue-Typ
Refactoring
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Tech-Stack
c
Bereich
backend

Rechercherichtung

Beginnen Sie damit, das Makro asdl_seq_LEN zu lokalisieren und seine Verwendungen zu überprüfen, bei denen Iteratorvariablen mit seinem Ergebnis verglichen werden. Identifizieren Sie die betroffenen Variablen vom Typ int oder Py_ssize_t und prüfen Sie anschließend die relevanten vorhandenen Tests, nachdem Sie die Typen im gesamten Codebestand konsistent gemacht haben.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

type-refactor

Feature or enhancement

It looks like the asdl_seq_LEN macro's "return type" is Py_ssize_t. Throughout the code, the type of the variable compared against a value "returned" by the macro are variously

int or
Py_ssize_t

Pitch

I am not sure, but it seems like it would be good to be consistent throughout the code base. If this is something that seems reasonable or valuable, I would be happy to take on the task!

Previous discussion

No previous discussion, however I searched prior issues and see nothing noting something similar.

Vorherrschende Sprache
Python
Sterne
77.2k
Forks
36k
Ø Merge
1 T. 9 Std.
Gemergte PRs (30 T.)
558

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/cpython

Alle Issues in python/cpython

Neue Issues direkt in Ihr Postfach

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