Skip to content
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

fix Doxygen warnings #590

Merged
merged 2 commits into from
May 16, 2024
Merged

fix Doxygen warnings #590

merged 2 commits into from
May 16, 2024

Conversation

arnopo
Copy link
Collaborator

@arnopo arnopo commented May 7, 2024

Fix warnings reported by Doxygen on the open-amp lib

arnopo added 2 commits May 7, 2024 10:35
Remove warning reported by Doxygen on doc generation:
- rewrite enum documentation,
- replace @ by \ref for references,
- fix param names

Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
The doxygen build report issue for function descriptions that provide
reference to some function parameters using @:

...
lib/include/openamp/rpmsg.h:160: warning: Found unknown command '\data'
lib/include/openamp/rpmsg.h:160: warning: Found unknown command '\len'
...

Fix the warning by replacing with back-ticks that will be interpreted as
code by Markdown(Markdown support is enabled in the Doxygen configuration).

Signed-off-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
@arnopo arnopo merged commit 7257dea into OpenAMP:main May 16, 2024
3 checks passed
@arnopo arnopo added this to the Release V2024.05 milestone May 20, 2024
@arnopo arnopo deleted the doxygen branch May 20, 2024 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants