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

invalid argument for command iline #8642

Closed
eseiler opened this issue Jul 2, 2021 · 3 comments
Closed

invalid argument for command iline #8642

eseiler opened this issue Jul 2, 2021 · 3 comments
Labels

Comments

@eseiler
Copy link

eseiler commented Jul 2, 2021

Describe the bug
After #8632, I'm getting warnings when using \if.

/*!\brief Some struct
 * \if DEV
 * Some conditional stuff
 * \endif
 */
struct S{};

/*!\brief Some other struct
 * \if DEV Some conditional stuff \endif
 */
struct T{};

This will emit:

/run/shm/example/example.hpp:2: warning: invalid argument for command iline
/run/shm/example/example.hpp:9: warning: invalid argument for command iline
/run/shm/example/example.hpp:2: warning: invalid argument for command iline
/run/shm/example/example.hpp:9: warning: invalid argument for command iline

This is independent of whether DEV is added to ENABLED_SECTIONS or not.
Except for the warning, the behaviour is otherwise the same, i.e. the if is evaluated correctly and the resulting documentation does not change.

Expected behavior
No warning, or, if syntactical error, point to what is wrong.

To Reproduce
example.tar.gz

Version
fc67a82
Any version before #8632 will work.

@albert-github
Copy link
Collaborator

Looks like a mean interaction between the \brief command and the, internal \iline command (the \iline is necessary to get proper line numbers in warnings in case the \if command is used).

Adding an empty line before the \if command is a workaround for the problem.

albert-github added a commit to albert-github/doxygen that referenced this issue Jul 2, 2021
From the string presented to the tokenizer the end spaces are stripped so it is not necessary that a soace is present at the end.
Due to the brief command it is also possible tat a `.` is present at the end.
These cases are handled now.

Furthermore small improvement on warning message.
@albert-github
Copy link
Collaborator

I've just pushed a proposed patch, pull request #8644

doxygen added a commit that referenced this issue Jul 3, 2021
issue #8642 invalid argument for command iline
@albert-github
Copy link
Collaborator

Code has been integrated in master on github, closing issue as it was only present in master.

ralexe pushed a commit to ralexe/doxygen that referenced this issue Jul 3, 2021
From the string presented to the tokenizer the end spaces are stripped so it is not necessary that a soace is present at the end.
Due to the brief command it is also possible tat a `.` is present at the end.
These cases are handled now.

Furthermore small improvement on warning message.
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