更新pre-commit使用说明:style问题全部处理完毕,记录knownConditionTrueFalse误报

This commit is contained in:
2026-09-01 14:01:08 +08:00
parent 6307c3f635
commit 6744c5d3bd
+9 -8
View File
@@ -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(闰年)的分支,实际逻辑正确,暂不处理。
#### 日常速度