Skip to content

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

@unicornx

Description

@unicornx

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

Sub-issues

Metadata

Metadata

Assignees

Labels

DocThis PR/issue related with documents

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions