codefmt shouldn't call FormatRanges with empty a:ranges
- Vorherrschende Sprache
- Vim Script
- Sterne
- 1.1k
- Forks
- 102
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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.
Beitragsleitfaden
Rechercherichtung
Beginne damit, den Kontext in Issue #48 zu lesen, verfolge dann die Aufrufe von codefmt an FormatRanges und untersuche die Behandlung eines leeren a:ranges durch den clang-format-Formatter. Erledigt ist die Aufgabe, wenn die Formatter-Schnittstelle leere Ranges als unmöglich behandelt und die unnötigen Prüfungen entfernt wurden, ohne Aufrufer zu hinterlassen, die diese übergeben.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- vim
- Bereich
- tooling
- Issue-Typ
- Refactoring
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100