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

EXPAND_ONLY_PREDEF neglected within function macro expansions (Origin: bugzilla #559489) #3165

Closed
doxygen opened this issue Jul 2, 2018 · 4 comments
Labels

Comments

@doxygen
Copy link
Owner

doxygen commented Jul 2, 2018

status NEW severity normal in component general for ---
Reported in version 1.5.5 on platform Other
Assigned to: Dimitri van Heesch

Original attachment names and IDs:

On 2008-11-05 19:23:14 +0000, Martin Baeuml wrote:

If a macro A is used inside another macro B, A is expanded even if
EXPAND_ONLY_PREDEF = YES and A is not in the predefined list.

Example (see also attached files):

#define AUTO 1
#define DEFAULT(x) = x
int test_function(int p DEFAULT(AUTO));

expected documentation:

 int test_function(int p = AUTO); 

wrongly generated documentation:

 int test_function(int p = 1); 

with doxygen settings
MACRO_EXPANSION    = YES
EXPAND_ONLY_PREDEF = YES
EXPAND_AS_DEFINED  = DEFAULT

On 2008-11-05 19:24:52 +0000, Martin Baeuml wrote:

Created attachment 122050
test case

On 2008-11-05 19:25:13 +0000, Martin Baeuml wrote:

Created attachment 122051
Doxyfile for test case
albert-github added a commit to albert-github/doxygen that referenced this issue May 16, 2022
A  relates issue is:
EXPAND_ONLY_PREDEF neglected within function macro expansions (Origin: bugzilla 559489) doxygen#3165

During substitution we should also test whether or not the restrictions about `EXPAND_AS_DEFINED` and `EXPAND_ONLY_PREDEF `` are fulfilled.
@albert-github
Copy link
Collaborator

I've just extended the proposed patch, pull request #9341, for this case.

@albert-github
Copy link
Collaborator

With the fix 387f508 a number of issues regarding the EXPAND_AS_DEFINED have been solved but this issue is still present.

doxygen added a commit that referenced this issue Aug 26, 2022
@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 Aug 26, 2022
@albert-github
Copy link
Collaborator

Code has been integrated in master on GitHub (please don't close the issue as this will be done at the moment of an official release).

@doxygen
Copy link
Owner Author

doxygen commented Aug 26, 2022

This issue was previously marked 'fixed but not released',
which means it should be fixed in doxygen version 1.9.5.
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 Aug 26, 2022
@doxygen doxygen closed this as completed Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants