Take the LoadContext of findStackAnalyzerSection by const reference
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 1/5
- 見積もり時間
- 1時間未満
- 初心者へのやさしさ
- 90/100
調査の方向性
src/App/ToolConfig.cpp の findStackAnalyzerSection（1074 行付近）を開いてください。issue では、パラメーターの変更点と、ApplyFn ハンドラーを変更せずにおくべき理由が説明されています。そのパラメーターだけを変更し、続いて記載されている CMake ビルド、ctest、./scripts/format-check.sh を実行してください。ビルドとテストが成功し、ほかにコード変更がなければ完了です。
索引モデルが issue の本文から書いたものです。
説明
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).
- 主要言語
- C++
- スター
- 5
- フォーク
- 4
- 平均マージ
- 1時間 2分
- マージ済み PR（30日）
- 74
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
CoreTrace/coretrace のほかの issue
-
documentation severity/minor
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
build severity/minor
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
メンテナーはふだん 1 日以内に返信
-
build good first issue severity/minor
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信
-
test: add unit tests for ctrace::RunDirectory対応中かも @tecnolgd が 1 日前に担当しました。 オープンgood first issue test
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
CoreTrace/coretrace#164 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
build severity/minor
難易度 3/5 1〜2日 初心者へのやさしさ 69/100
メンテナーはふだん 1 日以内に返信
CoreTrace/coretrace の issue をすべて見る
似ている issue
-
ai_p1 comp-message-queues
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
ClickHouse/ClickHouse#125890 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
redis/memtier_benchmark#642 ·
メンテナーはふだん 1 日以内に返信
-
Ctrl-C can abort a node: the state timer asks for the node names after shutdown対応中かも @fmrico が今日担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
fmrico/cascade_lifecycle#24 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100