scan_int_tokens line/col fields are untested
Los mantenedores suelen responder en 1 día
@facelessdevhack ya está trabajando en esto.
Desde el 15/10/2026.
- #1794 de @facelessdevhack — abierto
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- c
- Área
- testing-qa
Línea de trabajo
Empieza en src/builtins.c en los contadores token_line y token_col dentro de scan_int_tokens, y luego lee el bloque SIT1-SIT12 de tests/test_string_math.eigs para ver el estilo de aserciones existente. Añade aserciones de line y col para un token después de una línea de comentario omitida y para un token con sangría, siguiendo el modelo de las comprobaciones ST3/ST4/ST6/ST7 eliminadas. Está terminado cuando la prueba pasa con el código actual y falla cuando token_col se desplaza deliberadamente en uno.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Found by the blind critic on #1677 round 4. This predates #1677 and is not caused by it.
scan_int_tokens computes each row's line (index 1) and col (index 2) with its own counter loop in src/builtins.c (token_line / token_col). That code is separate from the removed scan_tokens. No test asserts those two fields. tests/test_string_math.eigs SIT1-SIT12 check the text, is_int, the value and the offset slice, but not line or col. The ST1-ST8 line/col checks only ever exercised scan_tokens, and #1677 deleted them along with it.
Done when
tests/test_string_math.eigsasserts line and col for at least one token after a skipped comment line and one indented token, mirroring the old ST3/ST4/ST6/ST7.- A planted off-by-one in
token_colmakes the test fail.
- Lenguaje dominante
- C
- Estrellas
- 3
- Forks
- 7
- Merge medio
- 8 h 41 min
- PR fusionados (30 d)
- 254
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
- Tiene una 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 InauguralSystems/EigenScript
-
eigenscript.h declares 8 observer functions that no longer existPosiblemente ocupada @facelessdevhack la tomó hace 1 día. Abiertoarea:observer enhancement found-by:cold-read good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
InauguralSystems/EigenScript#1781 ·
Los mantenedores suelen responder en 1 día
-
Amalgamation compiles with warnings at -O0 (-Wreturn-type) and -O1 (-Wmaybe-uninitialized)Posiblemente ocupada @facelessdevhack la tomó hoy. Abiertoarea:embed bug found-by:code-review
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
InauguralSystems/EigenScript#1775 ·
Los mantenedores suelen responder en 1 día
-
UB: (int) cast of a user double outside int range in dispatch / vm_index_is_int / flat_index_atAbiertoarea:runtime-vm
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
InauguralSystems/EigenScript#1748 ·
Los mantenedores suelen responder en 1 día
-
area:gfx bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
InauguralSystems/EigenScript#1684 ·
Los mantenedores suelen responder en 1 día
-
area:stdlib bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
InauguralSystems/EigenScript#1679 ·
Los mantenedores suelen responder en 1 día
Todos los issues de InauguralSystems/EigenScript
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
ChrisTitusTech/dwm-titus#397 ·
Los mantenedores suelen responder en 1 día
-
Translate gcc cheat sheet to frAbiertogood first issue help wanted language:fr translation
Dificultad 1/5 1-3 horas Aptitud para principiantes 85/100
mrvi0/cli-cheatsheet#160 ·
-
Test modules use volatile int for cross-thread flags instead of atomicsPosiblemente ocupada @rahimuddin140 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
valkey-io/valkey#4899 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Profiler (Remotery) leaks a socket for every connection that fails the WebSocket handshakePosiblemente ocupada @rthery la tomó hoy. Abiertotriage
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Symlink in release archive attempts traversalPosiblemente ocupada @Tiancheng-Xu la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
redis/hiredis#1388 · 1 comentario ·
Los mantenedores suelen responder en 1 día