codefmt shouldn't call FormatRanges with empty a:ranges
- 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
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