Minor optimization for set object, swap bodies
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 77.2k
- Forks
- 35.9k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
Based on this observation:
https://github.com/python/cpython/pull/132290#discussion_r2403137517
set_swap_bodies is only called in two locations in this file. For both cases the second argument b is a newly created temporary which is discarded afterwards. So we could replace part (all?) of these branches with asserts. Part of the other work (e.g. copying back to b) might not be needed for the same reason.
We should investigate this idea and see if it's worth it.
Linked PRs
- gh-148155
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
Beginne damit, set_swap_bodies und seine beiden Aufrufstellen in der referenzierten Datei zu lesen, und sieh dir anschließend die verknüpfte Diskussion in PR 132290 an. Prüfe, ob das temporäre zweite Argument immer verworfen wird und ob das Zurückkopieren unnötig ist. Als erledigt gilt die Feststellung und Dokumentation, ob die vorgeschlagene Vereinfachung das Verhalten beibehält; PR 148155 zeigt, dass daran bereits gearbeitet wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- backend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 20/100