开源改变世界

增加默认 HAL_NAME_LEN 当前 47 个字符 #615

推推 grbl 3年前 (2023-01-29) 163次浏览
打开
lorenzncode 打开了这个问题 2019 年 6 月 24 日 · 3条评论
打开

增加默认 HAL_NAME_LEN 当前 47 个字符#615

lorenzncode 打开了这个问题 2019 年 6 月 24 日 · 3条评论

注释

增加默认 HAL_NAME_LEN 当前 47 个字符 #615
贡献者

这是一个增强请求,而不是错误。

我正在以编程方式构建信号名称并使用 Python hal.connect()。我遇到了名称太长的错误,例如:
HAL: ERROR: signal name ‘scopeinterface.microscope.rapid-scope-zero-link2’ is too long

在这里,我正在构建由两个不同的组件名称和一个引脚名称组成的名称,它最终超过了最大限制。

该限制似乎在 hal.h 中定义:HAL_NAME_LEN

如果可能,请求是将默认值增加到 128。

增加默认 HAL_NAME_LEN 当前 47 个字符 #615
合作者

这是可能的,但对信号名称强制缩短限制也许有一些优势。
它们应该是人类可读的。是否有人认为很长的名字会更难区分?

此外,存在 128 个字符的 HAL 信号名称在文本编辑器中换行、不适合 halshow 窗口等的风险。

增加默认 HAL_NAME_LEN 当前 47 个字符 #615
合作者

我也会将它增加到 128。尤其是在较大的 hal 文件上可能会使事情更容易阅读。需要更多更改,并增加 hal 版本号。
最后更新是几年前:5735855

增加默认 HAL_NAME_LEN 当前 47 个字符 #615
合作者

我想这是消除 gcc-8 中 snprintf 警告的一种方法。
虽然我更愿意以正确的方式来做,即对 snprintf 返回值做一些事情并警告用户 pin-name(等)截断。

免费注册 在 GitHub 上加入此对话。已有帐户? 登录评论
标签
还没有
项目

还没有

发展

没有分支机构或拉取请求

3人参加
增加默认 HAL_NAME_LEN 当前 47 个字符 #615增加默认 HAL_NAME_LEN 当前 47 个字符 #615增加默认 HAL_NAME_LEN 当前 47 个字符 #615

喜欢 (0)