Mixture of tabs and spaces can make reading code difficult in some configurations (e.g. browsing on GitHub)

Abierto
#5,834 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
38/100
Tipo de issue
Refactorización
Claridad
Bastante claro
Estado de actividad
Tranquilo
Stack tecnológico
c
Área
tooling

Línea de trabajo

Comienza por driver/others/blas_server_omp.c alrededor de las líneas 354-357 e inspecciona las convenciones de indentación existentes para C y los headers del repositorio. Evalúa si una configuración de .editorconfig o una política coherente de tabuladores/espacios encaja con los archivos afectados; se considera terminado cuando el código afectado muestra una anidación inequívoca y la convención elegida está documentada o se aplica de forma coherente.

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

Descripción

Consider how this code, which mixes tabs and spaces for indentation, is displayed by GitHub: https://github.com/OpenMathLib/OpenBLAS/blob/a033631ee8486c245967465c2b43b0a3eb9a1b61/driver/others/blas_server_omp.c#L354-L357

The nesting is confusing; whoever edited that file had tabs set to 8 spaces, but GitHub's embedded display is using a different number of spaces per tab.

And if you click through, indentation looks different, though still confusing! Seems like GitHub uses inconsistent spaces per tab across different parts of its code rendering:

Image

I fixed it locally with a .editorconfig file, which it's at least possible GitHub's code browser supports, and in general has wide-spread editor support:

[*.{c,h}]
tab_width = 8

I don't know if that setting is correct for all files though.

Another option is to standardize on either tabs or spaces and reformat the code to match.

Lenguaje dominante
C
Estrellas
7.6k
Forks
1.7k
Merge medio
1 d 3 h
PR fusionados (30 d)
42

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de OpenMathLib/OpenBLAS

Todos los issues de OpenMathLib/OpenBLAS

Issues similares

Más issues de C

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.