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

Doxygen link generation does not find concepts unless complete path is specified #10395

Closed
higaski opened this issue Nov 6, 2023 · 3 comments

Comments

@higaski
Copy link

higaski commented Nov 6, 2023

For some reason Doxygen seems to make a difference between structs/classes and concepts when it comes to name lookup for link generation.

In the following snippet for example Doxygen is able to create a link to the plain struct but not the concept.

namespace c {

/// \page LinkConcept
/// Link to NoConcept works, but link to Concept does not!

struct NoConcept {};

template<typename T>
concept Concept = requires(T a) {
  { std::hash<T>{}(a) } -> std::convertible_to<std::size_t>;
};

}  // namespace c

doxygen_concepts_issue

The only way I found to get this to work is to explicitly link it using the complete path \ref c:Concept

I'm currently on version 1.9.8. For convenience, here's the main.cpp containing the code and the doxyfile:
doxygen_concepts_issue.zip

doxygen added a commit that referenced this issue Nov 11, 2023
@doxygen
Copy link
Owner

doxygen commented Nov 11, 2023

@higaski Please verify if the referenced commit fixes the problem for you. Do not close this issue, this will be done automatically when the next official release is made available.

@higaski
Copy link
Author

higaski commented Nov 14, 2023

Looks good, thanks for the fast fix!

@albert-github albert-github added the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Nov 14, 2023
@doxygen
Copy link
Owner

doxygen commented Dec 25, 2023

This issue was previously marked 'fixed but not released',
which means it should be fixed in doxygen version 1.10.0.
Please verify if this is indeed the case. Reopen the
issue if you think it is not fixed and please include any additional information
that you think can be relevant (preferably in the form of a self-contained example).

@doxygen doxygen removed the fixed but not released Bug is fixed in github, but still needs to make its way to an official release label Dec 25, 2023
@doxygen doxygen closed this as completed Dec 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants