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

ParameterHints within Doxygen comment - that uses colon after param #5836

Closed
heartacker opened this issue Jul 26, 2020 · 5 comments
Closed
Labels
bug Feature: Doc comments An issue related to code comments shown in hover, completion, and signature help. fixed Check the Milestone for the release in which the fix is or will be available. Language Service
Milestone

Comments

@heartacker
Copy link

hi, c_cpp has support Doxygen comment.
thus, it would be great if Parameter Hints support them.
c++

@heartacker heartacker changed the title ParameterHints within Doxygen coomment ParameterHints within Doxygen comment Jul 26, 2020
@michelleangela
Copy link
Contributor

@heartacker

The extension supports parameter hints. The name of the parameter is used as the key to find its corresponding comment from the Doxygen comment structure. It looks like from the screenshot, the name of the param in the comment has : and the actual parameter does not. The Doxygen parser will not be able to match E4E2x: with the actual parameter name E4E2x. Could you try removing the : from the name of the parameter comment and see if that displays the comment for the parameter hint?

Example of parameter hint:

// @brief this is brief comment
// @param GPIOx this is comment for parameter GPIOx
// @param E4E2x this is comment for parameter E4E2x
void myFunction(int GPIOx, int E4E2x)
{
}

image

@michelleangela michelleangela added Feature: Doc comments An issue related to code comments shown in hover, completion, and signature help. Language Service more info needed The issue report is not actionable in its current state labels Jul 27, 2020
@heartacker
Copy link
Author

doxygen/doxygen#7928 (comment)

The hint works within the standard format.
BTW, doxygen tolerates colons.

@michelleangela
Copy link
Contributor

Thanks for confirming that the colon after the parameter is supported by Doxygen. I'll mark this as a bug to fix.

@michelleangela michelleangela added bug and removed more info needed The issue report is not actionable in its current state labels Aug 3, 2020
@michelleangela michelleangela changed the title ParameterHints within Doxygen comment ParameterHints within Doxygen comment - that uses colon after param Aug 3, 2020
@heartacker
Copy link
Author

Thanks for confirming that the colon after the parameter is supported by Doxygen. I'll mark this as a bug to fix.

if colon follow parameter and also have space, that is accept. like @para arg: args for method
but using colon instead of space is wrong format, no need to be Compatible .@para arg:args for method, beacuse doxygen will identify arg:args as the para name.

@michelleangela michelleangela added this to the 0.30.0 milestone Aug 4, 2020
@michelleangela michelleangela added the fixed Check the Milestone for the release in which the fix is or will be available. label Aug 19, 2020
@sean-mcmanus
Copy link
Contributor

@github-actions github-actions bot locked and limited conversation to collaborators Oct 9, 2020
@michelleangela michelleangela removed their assignment Feb 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Feature: Doc comments An issue related to code comments shown in hover, completion, and signature help. fixed Check the Milestone for the release in which the fix is or will be available. Language Service
Projects
None yet
Development

No branches or pull requests

3 participants