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

Alias and variable template support #461

Merged
merged 1 commit into from
Nov 26, 2019
Merged

Conversation

melton1968
Copy link
Contributor

I need to document several meta-functions and variables that appear as alias and variable templates:

template<class T>
using Fuzzy = std::integral<T>;

template<class T, class U>
constexpr bool Furry = std::same_v<T,U>;

The current master version of breathe was dropping the template<class T> part of the declaration in each case, so I inserted single lines in visit_variable and visit_declaration to prepend the template prefix back to the declaration right before rendering.

I am unsure if this is the appropriate fix, but I did not detect any changes in the resulting breathe documentation pages (save for the template_type_alias specific test I added).

I would be very happy if this functionality could be integrated into the master branch and eventually a release.

cheers,
mark

@vermeeren vermeeren self-assigned this Nov 26, 2019
@vermeeren vermeeren merged commit 8329044 into breathe-doc:master Nov 26, 2019
vermeeren added a commit that referenced this pull request Nov 26, 2019
@vermeeren
Copy link
Collaborator

4.14.0 has been released containing this patch.

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.

2 participants