[clang-format] Improve declaration alignment and spacing - #11636
Conversation
|
👋 感谢您对 RT-Thread 的贡献!Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范,请在你的仓库中执行以下步骤运行代码格式化工作流(如果格式化CI运行失败)。 🛠 操作步骤 | Steps
完成后,提交将自动更新至 如有问题欢迎联系我们,再次感谢您的贡献!💐 |
|
这一块得快点先收敛一下,然后格式化一下仓库 |
|
AlignConsecutiveAssignments.Enabled: false -> AlignConsecutiveAssignments.Enabled: true 对于该参数的修改,我持保留意见,在一些变量对齐的情况下会很不美观,另外还占用额外空字符,个人更倾向保留紧凑语法,相比较于对齐的效果,紧凑型语法会更优: struct rt_mutex *mutex = RT_NULL;
int a = 0;格式化后: struct rt_mutex *mutex = RT_NULL;
int a = 0;格式化语法的修改建议起个议题征集意见,另外内核还有组件建议先不要做全面格式化 |
我看最近主线上的PR都有这个ci格式化检查问题了,我感觉先保留之前的格式化风格吧,先不要这么激进了 |
拉取/合并请求描述:(PR description)
[
为什么提交这份PR (why to submit this PR)
当前
.clang-format未启用连续赋值和连续声明对齐,函数形参中的普通指针、函数指针及标量参数无法形成统一的变量名列。同时,配置最多保留两个连续空行,与 RT-Thread 编码规范中避免连续使用两个以上空行的要求不完全一致。你的解决方案是什么 (what is your solution)
启用连续赋值和连续声明对齐,并让函数指针参与声明对齐;普通函数声明不参与该对齐,避免扩大格式化范围。将最多保留的连续空行从两个调整为一个。
格式化修改效果说明:
AlignConsecutiveAssignments.Enabled: false->AlignConsecutiveAssignments.Enabled: true格式化前:
格式化后:
AlignConsecutiveDeclarations.Enabled: false->AlignConsecutiveDeclarations.Enabled: true同时增加:
格式化前:
格式化后:
MaxEmptyLinesToKeep: 2->MaxEmptyLinesToKeep: 1格式化前(两个连续空行):
格式化后(一个空行):
请提供验证的bsp和config (provide the config and bsp)
BSP: N/A(本次仅修改格式化配置,不涉及 BSP 运行时功能)
.config: N/A
action: 本地使用 clang-format 22.1.5 验证配置解析,并验证上述三组格式化示例。
]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality:
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
#if 0代码,不包含已经被注释了的代码 All redundant code is removed and cleaned up