Harness header hygiene: trailing semicolons, name collisions, argument capture
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 50/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Área
- embedded-iot, testing-qa
Línea de trabajo
Start by inspecting the listed harness/tsffs-*.h and harness/tsffs.h macro definitions, then check harness/test.sh:25-26 and the MSVC C++ branch. Compare tsffs-gcc-x86.h with tsffs-gcc-x86_64.h and the riscv32/riscv64 headers; verify the reported issues with focused compile cases, including if (x) HARNESS_STOP(); else ... and an argument named value. Done means the listed header and script issues are corrected and the harness checks pass.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Where
- all
harness/tsffs-*.handharness/tsffs.h
Problem
- All macro bodies end in
} while (0);, breakingif (x) HARNESS_STOP(); else .... - x86 headers define
__cpuid(collides with<cpuid.h>/<intrin.h>); arm headers define__stringify(collides withlinux/stringify.h). - The local
valuein x86 macros captures a user argument namedvalue:HARNESS_START_WITH_MAXIMUM_SIZE(buf, value)passes the magic number. tsffs-gcc-x86.hduplicatestsffs-gcc-x86_64.h; riscv32/riscv64 differ only in whitespace.harness/test.sh:25-26missing/inrmpaths; MSVC C++ branch usessize_twith only<cstdint>.
Impact
Compile errors or silent miscompiles in user harnesses.
Suggested fix
Drop the trailing ;, prefix internal helpers with __tsffs_, rename the macro local, dedupe identical headers.
Related
#394, #379 (arm32 index)
Status
Reported by static code review at 79972db; not yet re-verified line by line. Found by reading the code; not reproduced under SIMICS.
- Lenguaje dominante
- Rust
- Estrellas
- 335
- Forks
- 26
- Merge medio
- 1 d 50 min
- PR fusionados (30 d)
- 12
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de intel/tsffs
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 86/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
Todos los issues de intel/tsffs
Issues similares
-
bug UI
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
wardian-app/Wardian#1746 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
jlcodes99/cockpit-tools#3125 ·
Los mantenedores suelen responder en 2 días
-
bug user-sprint
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Vondor/KassenSturz#967 ·
Los mantenedores suelen responder en 1 día
-
area/ci
Dificultad 1/5 Menos de una hora Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
[BUG] Incremental sync warns "no stored highest_uid" on every round for folders with no new mailAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
rustmailer/bichon#411 ·