Take the LoadContext of findStackAnalyzerSection by const reference
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
打开 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 小时 9 分钟
- 30 天内合并 PR
- 81
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
CoreTrace/coretrace 的其他 Issue
-
build severity/minor
难度 2/5 1-3 小时 新手友好度 84/100
CoreTrace/coretrace#176 · 1 条评论 ·
维护者通常 1 天内回复
-
build good first issue severity/minor
难度 2/5 1-3 小时 新手友好度 85/100
维护者通常 1 天内回复
-
test: add unit tests for ctrace::RunDirectory可能已有人在做 @tecnolgd 于 2 天前认领。 未关闭good first issue test
难度 2/5 1-3 小时 新手友好度 88/100
CoreTrace/coretrace#164 · 2 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
-
Release a macOS disk image (.dmg) for Apple silicon可能已有人在做 @SizzleUnrlsd 今天认领。 未关闭build enhancement
CoreTrace/coretrace#198 · 已指派 1 人 ·
维护者通常 1 天内回复
-
Builds without a build type are unoptimized: the README's commands and CI's main job可能已有人在做 @SizzleUnrlsd 于 1 天前认领。 未关闭build severity/minor
CoreTrace/coretrace#192 · 已指派 1 人 ·
维护者通常 1 天内回复
查看 CoreTrace/coretrace 的全部 Issue
相似的 Issue
-
bot-triaged module: correctness (silent) module: mps module: NaNs and Infs triage review
难度 2/5 半天 新手友好度 62/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
SCIInstitute/SCIRun#2842 ·
维护者通常 3 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复
-
agent:Windows bug code-review LOW testing
难度 2/5 1-3 小时 新手友好度 63/100
维护者通常 1 天内回复