google / google/vim-codefmt

codefmt shouldn't call FormatRanges with empty a:ranges

Abierto
#50 1 comentario 0 reacciones 0 asignados Ver en GitHub
bug
Lenguaje dominante
Vim Script
Estrellas
1.1k
Forks
102
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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.

Guía de contribución

Abrir la guía de contribución

Línea de trabajo

Empieza leyendo el contexto de issue #48, luego sigue las llamadas de codefmt a FormatRanges e inspecciona cómo el formateador clang-format gestiona un a:ranges vacío. Se considera terminado cuando la interfaz del formateador trata los ranges vacíos como imposibles y se eliminan las comprobaciones innecesarias sin dejar callers que los pasen.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
vim
Área
tooling
Tipo de issue
Refactorización
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.