Skip to content

Conversation

@echoix
Copy link
Owner

@echoix echoix commented Jul 27, 2025

No description provided.

echoix added 13 commits July 26, 2025 17:58
… add missing param tags in doxygen docstrings
Since the UNUSED macros are placed after the parameter names, Doxygen parses the attribute (UNUSED) like if it was the parameter name, and the parameter name as if it was part of the type. Doxygen then can't find the correct parameter in the docstring, and complains that there's a parameter 'UNUSED' not documented, and that an extra parameter is documented but is missing in the function signature.

This also applies to the NOPG_UNUSED and NO_NLS_UNUSED macros.

In the Doxygen config, we can predefine macros with another definition, essentially removing them.
Note that this also means that permalinks built from a has of the signatures will change, but they were wrong before.
Comment on lines 236 to 237
* \param out_list pointer to array of found signature strings or NULL if none found
* \return count of signature strings in the array

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[clang-format] reported by reviewdog 🐶

Suggested change
* \param out_list pointer to array of found signature strings or NULL if none found
* \return count of signature strings in the array
* \param out_list pointer to array of found signature strings or NULL if none
* found \return count of signature strings in the array

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants