开源改变世界

编码风格 #427

推推 grbl 3年前 (2023-02-05) 198次浏览
关闭
ledvinap 打开了这个问题 2013 年 3 月 19 日 · 7条评论
关闭

编码风格#427

ledvinap 打开了这个问题 2013 年 3 月 19 日 · 7条评论

评论

编码风格 #427

您好,
可以为 Marlin 推荐一些编码风格吗?
当前源代码不一致,这使得它难以阅读,甚至更难编写新代码。
从一些主流项目中选择编码风格应该很容易,随着时间的推移,Marlin 会逐渐融入其中。

也许 ErickZalm 或 daid(作为主要贡献者)可以选择一个?任何事情都可能比当前状态更好……

编码风格 #427
贡献者

在 2013 年 3 月 19 日凌晨 1:49,ledvinap 写道:

作为主要贡献者)
gnu 括号,双空格缩进。不要在任何
提交中更改 eol,至少不要与代码更改一起更改。

代码块中的#ifdefs 旨在不破坏可读性。

那将是我一年前尝试 top have uniform style 时使用的样式。

我认为在非分层项目中强制执行单一样式
效果不够好,因为关于它的讨论没有解决方案。

编码风格 #427

好的,我试着坚持这一点。

另一个问题是某些文件中的混合行尾。似乎错误的行尾进入了拉取请求 – 大部分文件是 LF,有些行是 CRLF。这使得做出合理的拉取请求变得非常困难。

实施https://help.github.com/articles/dealing-with-line-endings(.gitattributes可以让新贡献者省去一些麻烦)并规范化所有文件会有很大帮助。

编码风格 #427
贡献者

讨论过很多次。太多有自己风格的贡献者(或有时缺乏)……