New REPL: `python -q` outputs unnecessary blank space on empty terminal
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 77.2k
- Forks
- 35.9k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
Bug report
Bug description:
If I start the new REPL on a clear terminal, I get an unnecessary blank space.
clear; python -q
Whereas this does not happen if I run the old ("basic") REPL:
clear; PYTHON_BASIC_REPL=1 python -q
Analysis
I've tracked this down to these 2 lines of code: https://github.com/python/cpython/blob/3.13/Lib/_pyrepl/unix_console.py#L241-L242
self.__move(0, len(self.screen) - 1)
self.__write("\n")
I don't totally understand this code, but it certainly smells like there's a bug here when dealing with an empty screen (for example, if len(self.screen) == 0, are we trying to move the cursor to index -1?)
I'm not sure what the fix is here, though. I don't really understand this code: why are we moving up to a line just to move the cursor down a line afterwards? why can't we jump straight to where we want to be? I tried digging through the git history for answers, but it looks like this incantation dates back all the way to this initial commit to pypi/pyrepl. I'm not sure how to dig any further back in time.
CPython versions tested on:
3.13, 3.14, CPython main branch
Operating systems tested on:
Linux
I'm using Alacritty, in case that's relevant.
Linked PRs
- gh-131907
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, das Leerzeichen mit clear; python -q nachzustellen und es mit PYTHON_BASIC_REPL=1 python -q zu vergleichen. Lies Lib/_pyrepl/unix_console.py rund um die Zeilen 241-242, einschließlich der Aufrufe __move und __write, und sieh dir den verlinkten PR gh-131907 an. Fertig ist die Aufgabe, wenn die neue REPL auf einem leeren Terminal kein unnötiges Leerzeichen mehr ausgibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- cli
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 25/100