From 6744c5d3bdbfd7fb81bbd82205c8722e6772c3e4 Mon Sep 17 00:00:00 2001 From: Wanderingss <1624155937@qq.com> Date: Tue, 1 Sep 2026 14:01:08 +0800 Subject: [PATCH] =?UTF-8?q?=E6=9B=B4=E6=96=B0pre-commit=E4=BD=BF=E7=94=A8?= =?UTF-8?q?=E8=AF=B4=E6=98=8E=EF=BC=9Astyle=E9=97=AE=E9=A2=98=E5=85=A8?= =?UTF-8?q?=E9=83=A8=E5=A4=84=E7=90=86=E5=AE=8C=E6=AF=95=EF=BC=8C=E8=AE=B0?= =?UTF-8?q?=E5=BD=95knownConditionTrueFalse=E8=AF=AF=E6=8A=A5?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- docs/pre-commit使用说明.md | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) 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(闰年)的分支,实际逻辑正确,暂不处理。 #### 日常速度