| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
fix:clean code Co-authored-by: liu-lu<www.liulu824910939@qq.com> # message auto-generated for no-merge-commit merge: !4869 merge master into master fix:clean code Created-by: liu-lu Commit-by: liu-lu Merged-by: cann-robot Description: # Pull Request ## 描述 本 PR 对 tsd 模块进行头文件依赖清理（clean code），主要变更如下： 1. **替换宽口径头文件**：将 #include "driver/ascend_hal.h" 替换为更精确的子模块头文件 - device_comm_agent.h：替换为 #include "driver/ascend_hal_base.h" - stub_dc.cpp：替换为 #include "driver/ascend_hal_base.h" + #include "driver/ascend_hal_error.h" 2. **移除未使用的 include**： - package_env_info.h：移除未使用的 #include "driver/ascend_hal.h" - package_manager.h：移除未使用的 #include "driver/ascend_hal.h" - package_env_info.cpp：移除未使用的 #include "driver/ascend_hal.h" 和 #include "error_manager.h" 3. **显式声明依赖**： - weak_ascend_hal.h：新增 #include "driver/ascend_hal_base.h" 和 #include "driver/ascend_hal_error.h" ## 变更类型 请选择本次引入的变更类型： <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新（格式化，局部变量） - [x] ♻️ 重构（既不修复错误也不增加功能的代码变动） - [ ] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在当前页面的右侧'关联Issue'部分添加相应Issue链接，并勾选'合并后关闭已关联的 Issue'选项。 --> #965 ## 如何测试 描述测试此变更的步骤和前提条件： 1. 执行全量编译构建，验证头文件替换后编译无报错 2. 运行 tsd 模块相关单元测试，确认功能无回归 3. 验证 device_comm、package_manager 模块加载和初始化流程正常 ## 核对清单 <!-- [x] 表示选中 --> - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签（如：feat:, fix:） - [x] 我已经详细阅读了贡献指南（CONTRIBUTING.md），并遵守了其中的所有规定，包括但不限于commit message的格式、无效commit的合并等 ## 其他信息 本次变更仅涉及头文件 include 调整，不涉及任何功能逻辑变更。目标是降低头文件依赖耦合度，提升编译效率。 See merge request: cann/runtime!4869 | 26 天前 | |
fix:clean code Co-authored-by: liu-lu<www.liulu824910939@qq.com> # message auto-generated for no-merge-commit merge: !4869 merge master into master fix:clean code Created-by: liu-lu Commit-by: liu-lu Merged-by: cann-robot Description: # Pull Request ## 描述 本 PR 对 tsd 模块进行头文件依赖清理（clean code），主要变更如下： 1. **替换宽口径头文件**：将 #include "driver/ascend_hal.h" 替换为更精确的子模块头文件 - device_comm_agent.h：替换为 #include "driver/ascend_hal_base.h" - stub_dc.cpp：替换为 #include "driver/ascend_hal_base.h" + #include "driver/ascend_hal_error.h" 2. **移除未使用的 include**： - package_env_info.h：移除未使用的 #include "driver/ascend_hal.h" - package_manager.h：移除未使用的 #include "driver/ascend_hal.h" - package_env_info.cpp：移除未使用的 #include "driver/ascend_hal.h" 和 #include "error_manager.h" 3. **显式声明依赖**： - weak_ascend_hal.h：新增 #include "driver/ascend_hal_base.h" 和 #include "driver/ascend_hal_error.h" ## 变更类型 请选择本次引入的变更类型： <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新功能 - [ ] 💄 代码风格更新（格式化，局部变量） - [x] ♻️ 重构（既不修复错误也不增加功能的代码变动） - [ ] 📦 构建过程或辅助工具的变动 - [ ] 📝 文档内容更新 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在当前页面的右侧'关联Issue'部分添加相应Issue链接，并勾选'合并后关闭已关联的 Issue'选项。 --> #965 ## 如何测试 描述测试此变更的步骤和前提条件： 1. 执行全量编译构建，验证头文件替换后编译无报错 2. 运行 tsd 模块相关单元测试，确认功能无回归 3. 验证 device_comm、package_manager 模块加载和初始化流程正常 ## 核对清单 <!-- [x] 表示选中 --> - [x] 我的代码遵循了项目的代码风格 - [x] 我已对代码进行了自测 - [x] 我已更新了相关的文档 - [x] 我在标题中使用了合适的类型标签（如：feat:, fix:） - [x] 我已经详细阅读了贡献指南（CONTRIBUTING.md），并遵守了其中的所有规定，包括但不限于commit message的格式、无效commit的合并等 ## 其他信息 本次变更仅涉及头文件 include 调整，不涉及任何功能逻辑变更。目标是降低头文件依赖耦合度，提升编译效率。 See merge request: cann/runtime!4869 | 26 天前 |