diff --git a/docs/pre-commit使用说明.md b/docs/pre-commit使用说明.md index 22e4bd2d..a72a7b28 100644 --- a/docs/pre-commit使用说明.md +++ b/docs/pre-commit使用说明.md @@ -164,17 +164,18 @@ MDK-ARM/ # Keil 工程文件 test/ # 测试代码 ``` -#### style 检查发现的有价值问题(待修复) +#### style 检查发现的问题(已处理完毕) -开启 style 后,以下问题值得修复(符合嵌入式C代码规范): +开启 style 后发现的有价值问题已全部处理: -| 规则 | 问题 | 涉及文件 | 价值 | +| 规则 | 问题 | 涉及文件 | 处理结果 | |---|---|---|---| -| `knownConditionTrueFalse` | 条件恒假(`uint16_t > 65535`) | net_socket.c | **高**:潜在 Bug | -| `compareValueOutOfTypeRangeError` | 同上 | net_socket.c | **高**:潜在 Bug | -| `redundantAssignment` | 重复赋值覆盖旧值 | ch395f.c | **高**:Bug 嫌疑 | -| `unreadVariable` | 变量赋值后未读取 | gd5f2gq5ue.c, time_sync.c, ringbuf.c | **中**:冗余代码 | -| `variableScope` | 变量作用域可缩小 | sys_clock.c, net_socket.c 等 | **低**:规范6.1,可逐步修 | +| `compareValueOutOfTypeRangeError` | `uint16_t > 65535` 恒假 | net_socket.c | ✅ 已修复(改为 `== 0` 检测溢出绕回) | +| `redundantAssignment` | 重复赋值覆盖旧值 | ch395f.c | ✅ 已抑制(HAL 清标志宏读寄存器即目的,误报) | +| `unreadVariable` | 变量赋值后未读取 | gd5f2gq5ue.c, time_sync.c, ringbuf.c | ✅ 已修复(移除冗余初始化) | +| `variableScope` | 变量作用域可缩小 | sys_clock.c, net_socket.c 等 18 处 | ✅ 已修复(声明移入实际使用块) | + +> 注:`sys_clock.c` 的 `knownConditionTrueFalse`(`day_count < diy` 恒假)为 cppcheck 误报——未跟踪 `diy` 可能被赋为 366(闰年)的分支,实际逻辑正确,暂不处理。 #### 日常速度