codefmt shouldn't call FormatRanges with empty a:ranges
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- vim
- Ambito
- tooling
Direzione di ricerca
Inizia leggendo il contesto nell’issue #48, poi segui le chiamate di codefmt a FormatRanges e analizza la gestione di un a:ranges vuoto da parte del formatter clang-format. Il lavoro è completo quando l’interfaccia del formatter tratta i ranges vuoti come impossibili e i controlli non necessari vengono rimossi, senza lasciare callers che li passino.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
As noted in #48, we seem to have some cases where a formatter's FormatRanges() will be invoked with an empty a:ranges. Either that, or we're unnecessarily checking for emptiness in the clang-format formatter.
Unless I'm missing something, this doesn't seem like it's useful; the interface with formatters should say that this is impossible, and we should remove the checks we have.
- Lingua principale
- Vim Script
- Stelle
- 1.1k
- Fork
- 102
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
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 google/vim-codefmt
-
Write a codefmt FAQApertaenhancement
Difficoltà 1/5 1-3 ore Idoneità per principianti 68/100
google/vim-codefmt#53 ·
-
Add meson formatApertanew formatter
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
google/vim-codefmt#243 ·
-
LCF for lua formattingAperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 52/100
google/vim-codefmt#240 · 1 commento ·
-
Add `:CodefmtInfo` command to report diagnostics to help users check config / diagnose problemsApertaenhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
google/vim-codefmt#234 · 2 commenti ·
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 45/100
google/vim-codefmt#222 · 3 commenti ·
Tutte le issue di google/vim-codefmt
Issue simili
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
Vector35/community-plugins#376 ·
-
bad links in rfc5890.htmlAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
ietf-tools/rfc2html#81 ·
-
refactor
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
tomnewport/memprot-topo#55 ·
-
Difficoltà 2/5 Mezza giornata Idoneità per principianti 72/100
kristoff-it/ziggy#85 · 1 commento ·
-
feature
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
The-PR-Agent/pr-agent#3984 ·
I maintainer di solito rispondono entro 1 giorno