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

C++ params with namespaces have the namespace as docstring name #17

Closed
rmatpd opened this issue Jul 18, 2019 · 3 comments
Closed

C++ params with namespaces have the namespace as docstring name #17

rmatpd opened this issue Jul 18, 2019 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@rmatpd
Copy link

rmatpd commented Jul 18, 2019

The following is what was generated from a function with namespaces:

/**
 * TODO
 *
 * @param std TODO
 * @return TODO
 */
std::string trimLine(const std::string &line);

the "@param std TODO" is incorrect, and should instead be "@param string TODO"

@rmatpd rmatpd added the bug Something isn't working label Jul 18, 2019
@kkoomen
Copy link
Owner

kkoomen commented Jul 19, 2019

It should be @param line TODO. You said it should be @param string TODO but this is the type hint, but it should be the var name.

@kkoomen
Copy link
Owner

kkoomen commented Jul 19, 2019

This has been fixed in v1.4.6.

Feel free to submit any new issues if you experience any unwanted behaviour in the future. Thanks for your contribution.

@kkoomen kkoomen closed this as completed Jul 19, 2019
@rmatpd
Copy link
Author

rmatpd commented Jul 19, 2019

You're correct about my incorrect param name. Thanks for the quick fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants