Take the LoadContext of findStackAnalyzerSection by const reference
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Idoneità per principianti
- 90/100
Direzione di ricerca
Apri src/App/ToolConfig.cpp in corrispondenza di findStackAnalyzerSection (intorno alla riga 1074); l’issue identifica la modifica del parametro e spiega perché i gestori ApplyFn devono rimanere invariati. Modifica solo quel parametro, poi esegui la build CMake indicata, ctest e ./scripts/format-check.sh. Il lavoro è completato quando la build e i test hanno esito positivo e non vengono apportate altre modifiche al codice.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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).
- Lingua principale
- C++
- Stelle
- 5
- Fork
- 4
- Merge medio
- 1h 2m
- PR unite (30g)
- 74
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di CoreTrace/coretrace
-
documentation severity/minor
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
build severity/minor
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
I maintainer di solito rispondono entro 1 giorno
-
build good first issue severity/minor
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno
-
test: add unit tests for ctrace::RunDirectoryForse già presa @tecnolgd l’ha presa 1 giorno fa. Apertagood first issue test
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
CoreTrace/coretrace#164 · 2 commenti · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
build severity/minor
Difficoltà 3/5 1-2 giorni Idoneità per principianti 69/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di CoreTrace/coretrace
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno
-
Ctrl-C can abort a node: the state timer asks for the node names after shutdownForse già presa @fmrico l’ha presa oggi. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
fmrico/cascade_lifecycle#24 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
Deltakit/deltakit-stim#52 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno