-
Notifications
You must be signed in to change notification settings - Fork 5.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[lwp]add comments for lwp system calls. #9934
Conversation
这个 PR 的工作属于 #9263 跟踪的问题的一部分 此外,欢迎关注 |
这个comment要不要统一放到.h中? |
我建议对函数的 comment 跟着函数体走,而不是放在头文件中。好处是头文件中列出函数可以比较紧凑,方便一眼就看出所有的 API。如果夹杂了comment,导致头文件内容很长,反而不容易快速了解模块暴露的 API 列表。 参考其他软件,譬如 linux 也是推荐将函数的说明放在 .c 中 |
个人也是建议放在C文件中,component中其它组件的注释也都是放在c文件中的,因此这样可以保持整体的风格一致。 |
Hi,ALL。后面大家做 doxygen 部分工作时,请关注一下 #9824 这个 issue,我们用这个 issue 来统一规划和同步相关工作,譬如哪些重要哪些可以先做等等。我的建议是如果您以后在提相关 pr 前可以先看一下 9824,或者提交一个 issue 将你想要解决的问题发出来,确定和整体规划不要有冲突后再进行。我现在自愿 lead 并维护 doxygen 相关工作,任何相关疑问可以找我沟通(群内或者 github 9824 上)。 |
拉取/合并请求描述:(PR description)
[
为什么提交这份PR (why to submit this PR)
add comments for lwp system calls.
你的解决方案是什么 (what is your solution)
add comments for lwp system calls.
请提供验证的bsp和config (provide the config and bsp)
BSP:
.config:
action:
]
当前拉取/合并请求的状态 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