google / google/vim-codefmt

codefmt shouldn't call FormatRanges with empty a:ranges

Aperta
#50 1 commento 0 reazioni 0 assegnatari Vedi su GitHub
bug
Lingua principale
Vim Script
Stelle
1.1k
Fork
102
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

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.

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.