eigenlsp: scopes past MAX_FN_SCOPES (256) are silently dropped; rename then edits shadowed locals
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 68/100
Línea de trabajo
Start in build_scopes in src/eigenlsp.c and trace how the scope count is returned and consumed by rename resolution. Add a regression test with more than 256 scopes, then run the rename path and execute the renamed program. Done means scope handling no longer silently loses late scopes and a late same-named parameter remains untouched.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Found by /code-review on PR #1330. The limit itself predates that PR, but #1330 makes it bite sooner.
build_scopes in src/eigenlsp.c stops recording scopes after MAX_FN_SCOPES (256), with no warning; the returned count is simply lower. #1330 gives each lambda a scope too, so lambdas now use slots from the same budget as define and for.
In a file with ~200 defines and ~60 early lambdas, the later defines get no scope. Renaming a global x then also rewrites a later define g(x): signature and body, and a later local x resolves to the global. That is a silent wrong edit.
Done when
- The scope table grows as needed (no fixed cap), or rename refuses with an error when the limit is reached rather than guessing.
- A test: a file with more than 256 scopes, where a rename of a global leaves a late
define's same-named parameter untouched (checked by running the renamed program).
- 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 3 días. 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ó hace 3 días. 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
-
scan_int_tokens line/col fields are untestedPosiblemente ocupada @facelessdevhack la tomó hace 3 días. Abiertoarea:stdlib found-by:critic kind:gate-defect
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
InauguralSystems/EigenScript#1693 ·
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
Todos los issues de InauguralSystems/EigenScript
Issues similares
-
Weekly fuzz run failedAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
kubernetes-sigs/security-profiles-operator#3597 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
AFLplusplus/AFLplusplus#2903 ·
-
Clean up stale and inaccurate comments in the platform screen and input filesPosiblemente ocupada @RichStephens la tomó hoy. Abierto
Dificultad 2/5 Medio día Aptitud para principiantes 88/100
FujiNetWIFI/fujinet-config#289 ·
Los mantenedores suelen responder en 1 día
-
[raylib] update to 6.2Abiertocategory:port-update
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
nasa/cFS-GroundSystem#272 ·