google / google/vim-codefmt

codefmt shouldn't call FormatRanges with empty a:ranges

Offen
#50 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
bug
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.