Add support for class/struct/union specific location #203
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hey,
Nice project!
I encountered a wrong #include text while generating the documentation (the hyperref is good).
Doxygen allows the user to specify a specific include path and your script did not extract the correct information from the xml.
Fortunately this info is present inside the
<includes>
tag.This is somewhat related to #137. Because the path in
<location file="xx"/>
is not stripped using the STRIP_FROM_INC_PATH option (otherwise we would have a problem referencing the files).That is, there is still an issue for the path displayed in the text for functions, enums, typedefs, variables and
defines which are not part of a class/struct or union.
I believe that this was a custom addition of yours and Doxygen doesn't provide the tag for these unfortunately.
A fix to the latter would involve something as provided in #139, but where the paths in STRIP_FROM_INC_PATH are also stripped of STRIP_FROM_PATH beforehand to mimic Doxygen behavior (because the paths in
<location file="xx"/>
are already stripped of STRIP_FROM_PATH).