Small nit
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 35/100
Direzione di ricerca
Individua lo snippet C mostrato nel codice sorgente ed esamina come vengono utilizzati la ricerca della riga e il ciclo delle colonne del cursore. Confronta la forma condizionale proposta con il comportamento esistente, quindi verifica che l’editor continui a gestire correttamente righe, tabulazioni e offset orizzontali dopo la semplificazione.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Hey there, I don't really want to make a pull request here so I thought I'd just make an issue. Sorry if that's not proper etiquette! By the way, love the work.
The following lines of code appear in the source code:
erow *row = (filerow >= E.numrows) ? NULL : &E.row[filerow];
if (row) {
for (j = E.coloff; j < (E.cx+E.coloff); j++) {
if (j < row->size && row->chars[j] == TAB) cx += 7-((cx)%8);
cx++;
}
}
Surely this should be simplified slightly to:
if (filerow < E.numrows) {
erow *row = &E.row[filerow];
for (j = E.coloff; j < (E.cx+E.coloff); j++) {
if (j < row->size && row->chars[j] == TAB) cx += 7-((cx)%8);
cx++;
}
}
It seems a little off to make the conditional based off of the result of another conditional that appears literally one line above.
- Lingua principale
- C
- Stelle
- 9.2k
- Fork
- 1k
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di antirez/kilo
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 68/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 35/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 55/100
-
Multi-line comment highlighting breaks after row deletionForse già presa @davidklassen l’ha presa 271 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
Tutte le issue di antirez/kilo
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
containers/bubblewrap#826 ·
I maintainer di solito rispondono entro 1 giorno
-
feature/themes QA/No release-notes/exclude tests tests/missing-coverage tests/test-quality
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
brave/brave-browser#60971 ·
I maintainer di solito rispondono entro 1 giorno
-
[compiler-rt] Rtsan-aarch64-NoInstTest: error: no such file or directory: 'NOTFOUND' (libatomic)Apertabuild-problem compiler-rt:rtsan platform:linux
Difficoltà 2/5 1-3 ore Idoneità per principianti 63/100
llvm/llvm-project#239039 ·
I maintainer di solito rispondono entro 1 giorno
-
component:obi contribfest go test
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 83/100
open-telemetry/opentelemetry-ebpf-instrumentation#4143 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
checkpoint-restore/criu#3304 ·
I maintainer di solito rispondono entro 1 giorno