Skip to content

[Bug] audio 的 API doxygen 注释中的问题 #10065

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

Closed
unicornx opened this issue Mar 3, 2025 · 6 comments
Closed

[Bug] audio 的 API doxygen 注释中的问题 #10065

unicornx opened this issue Mar 3, 2025 · 6 comments
Assignees
Labels
Doc This PR/issue related with documents

Comments

@unicornx
Copy link
Contributor

unicornx commented Mar 3, 2025

RT-Thread Version

master

Hardware Type/Architectures

N/A

Develop Toolchain

Other

Describe the bug

#10058 合入后针对 audio 新增的注释部分我发现一些问题,请检查。

  1. documentation/device/adudio/audio.md 这个文件的路径有问题,[RFC][doxygen]Doxygen comment standard processing #10058 的代码 pr 时没有基于最新的 master,因为自从 75722c4 "doxygen: reorganize directory structure" 合入后,device 的文档路径已经移到 documentation/5.device 下去了。另外还要注意的是 ”adudio“ 这个目录名字拼写错误。

  2. components/drivers/include/drivers/dev_audio.h 中的 Data Structures 的成员都缺乏注释

  3. enum 部分代码上应该给 enum 定义一个名字为好,否则在 html 上只显示一个 enum (doxygen 认为其是一个 anonymous enum),不过这个涉及代码的改动,请检查加上是否会有副作用。

  4. rt_audio_ops 缺失了。作为一个需要 user 定义的 callback 函数的集合,需要对回调函数的行为做简单的描述,这对 bsp 使用还是很重要的。

  5. 一个改进建议,对于 marco 我看你为了分组新建了许多 group,这个没有问题。只是在对组的 brief 描述上我建议改进一下。譬如对 audio_dsp,注释是 ”Support Dsp(input/output) Units controls“ 感觉说的不是很清楚,这些 marco 是不是每个对应一种类型(type)? 这个在注释上感觉不出来。类似的还有 audio_mixer 和 audio_codec_cmd。

  6. html 网页的 Modules 部分多了一项出来,见下图:

    Image

  7. rt_audio_samplerate_to_speed 看上去是一个无用函数,感觉可以删除,但需要另外一个 pr 并让 maintaner (熊大?)确认一下。

Other additional context

No response

@unicornx unicornx added the Doc This PR/issue related with documents label Mar 3, 2025
@unicornx
Copy link
Contributor Author

unicornx commented Mar 3, 2025

@1078249029 请检查,如果接受请认领这个 issue。

@unicornx unicornx changed the title [Bug] audio 的 API doxygen 注释需要改进 [Bug] audio 的 API doxygen 注释中的问题 Mar 3, 2025
@1078249029
Copy link
Contributor

1078249029 commented Mar 3, 2025

ok,这些改好后还是提交RFC么?

@unicornx
Copy link
Contributor Author

unicornx commented Mar 4, 2025

ok,这些改好后还是提交RFC么?

RFC 的意思是当改动比较大,希望提出来给社区作为一种草案讨论的意思,也就是说当你提交的 pr 不希望合并而只是给大家参考看看的时候的一种说法,引用 https://kernelnewbies.org/PatchTipsAndTricks

Another common Subject prefix is "RFC". This stands for "Request for Comment", which tells maintainers should review your patch thoroughly, and provide feedback. RFC is typically used when sending feature patches for the first time, or anytime the patch is more than just a simple bug fix.

@unicornx
Copy link
Contributor Author

unicornx commented Mar 6, 2025

#10073 解决了所列问题的 1/2/4/5/6
还有 3 和 7 有待解决。

@1078249029
Copy link
Contributor

3 已经在 #10094 中解决了,bsp编译没有问题,还有什么需要注意的么?7 被汇总在 #10095 等待解决

@unicornx
Copy link
Contributor Author

3 最终在 #10101 中解决。7 归入 #10095, 本 issue 可以关闭了。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Doc This PR/issue related with documents
Projects
None yet
Development

No branches or pull requests

2 participants