Take the LoadContext of findStackAnalyzerSection by const reference
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- Menos de una hora
- Aptitud para principiantes
- 90/100
Línea de trabajo
Abre src/App/ToolConfig.cpp en findStackAnalyzerSection (alrededor de la línea 1074); el issue identifica el cambio del parámetro y explica por qué los manejadores ApplyFn deben permanecer sin cambios. Cambie solo ese parámetro y, a continuación, ejecute la compilación de CMake indicada, ctest y ./scripts/format-check.sh. La tarea estará terminada cuando la compilación y las pruebas pasen y no se hayan realizado otros cambios en el código.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
findStackAnalyzerSection (src/App/ToolConfig.cpp:1074) takes LoadContext& ctx, but only calls ctx.deprecated(...), which is a const member function. The initial static analysis flagged it twice: cppcheck's constParameterReference and the stack analyzer's ConstParameterNotModified.Reference (docs/quality/static-analysis-initial.md).
To do: change the parameter to const LoadContext& ctx.
Leave the configuration handlers (ApplyFn) as they are. They also take LoadContext& and are flagged too, but they write the configuration through ctx.config. config is a reference, so a const LoadContext& would still compile, but it would hide that these handlers modify the configuration.
Check:
cmake --build build --parallel 4
ctest --test-dir build --output-on-failure
./scripts/format-check.sh
Acceptance
findStackAnalyzerSectiontakesconst LoadContext&; nothing else changes.- The build (warnings are errors) and
ctestpass.
From the technical debt register (docs/quality/technical-debt.md, D11).
- Lenguaje dominante
- C++
- Estrellas
- 5
- Forks
- 4
- Merge medio
- 1 h 10 min
- PR fusionados (30 d)
- 79
Preparar el entorno
- Sin Dockerfile ni 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 CoreTrace/coretrace
-
build severity/minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
CoreTrace/coretrace#176 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
build good first issue severity/minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
test: add unit tests for ctrace::RunDirectoryPosiblemente ocupada @tecnolgd la tomó hace 1 día. Abiertogood first issue test
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
CoreTrace/coretrace#164 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Builds without a build type are unoptimized: the README's commands and CI's main jobPosiblemente ocupada @SizzleUnrlsd la tomó hoy. Abiertobuild severity/minor
CoreTrace/coretrace#192 · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
bug severity/minor
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
Los mantenedores suelen responder en 1 día
Todos los issues de CoreTrace/coretrace
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
FriendsofECCE/ECCE#336 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
bitcoin-core/gui-qml#1030 ·
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
tesseract-robotics/tesseract#1384 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
NVIDIA/nccl-extensions#26 ·