| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
Initial commit | 6 个月前 | |
[feature] added clang-tidy files Co-authored-by: wiyr0<wangyixian3@huawei.com> # message auto-generated for no-merge-commit merge: !168 merge support_clang_tidy into master [feature] added clang-tidy files Created-by: wiyr0 Commit-by: wiyr0 Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因：** 增加clang-tidy配置文件，做一些静态语法检查，安全类型检查等 https://gitcode.com/Ascend/msot/issues/54 - **修改方案：** 在pre-commit配置文件里，增加.clang-tidy配置; 然后修复扫描出的问题 - **修改内容：** 1. 新增 .clang-tidy：定义 88 行检查规则（Clang 静态分析、Google 编码规范、bugprone、modernize、cppcoreguidelines 等），WarningsAsErrors: '*'。 2. 修改 .pre-commit-config.yaml：新增 clang-tidy 本地 hook（限定 csrc/ 下的 C/C++ 文件）；统一行尾格式。 3. Rule of Five：补全特殊成员函数（约 30 个类），满足 cppcoreguidelines-special-member-functions——析构、拷贝/移动构造、拷贝/移动赋值必须成对声明。 4. 虚析构补全 + final + override, 满足 modernize-use-override；防止基类无虚析构导致未定义行为；标记不可继承类。 5. 满足 cppcoreguidelines-pro-type-cstyle-cast, hal_helper.cpp：3 处 (FuncType) dlsym(...) → reinterpret_cast<FuncType>(dlsym(...))。 6. 消除冗余 std::move 7. 满足 cppcoreguidelines-lambda-capture-default，避免不必要的 this 捕获。 8. UT 修复（因 Rule of Five 导致拷贝/移动被 delete） - [ ] **涉及代码双合**(贴上另一个PR链接)： ---- ### 2. 功能验证 - [x] **功能自验截图**（请确保不体现个人信息） - [x] **冒烟是否通过** ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [x] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** ---- ### 4. 资料修改自检 - **资料修改：** 不涉及 ---- See merge request: Ascend/msopprof!168 | 4 小时前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 6 个月前 | ||
| 4 小时前 |