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

Whitespace in front of "@if" removed differently in 1.7.x than 1.6.x (Origin: bugzilla #665313) #4483

Closed
doxygen opened this issue Jul 2, 2018 · 1 comment

Comments

@doxygen
Copy link
Owner

doxygen commented Jul 2, 2018

status RESOLVED severity critical in component general for ---
Reported in version unspecified on platform Other
Assigned to: Dimitri van Heesch

On 2011-12-01 18:03:22 +0000, MikeH wrote:

I have a lot of documentation done using 1.6.3. In attempting to upgrade to 1.7.x, I discovered that both 1.7.3 and 1.7.5 handle whitespace around conditional @if statements and @endif statements differently. The difference can be illustrated by writing text like the following:

some text in front @if clike middle@endif tail text

1.7.5 eats the space before @if and after @endif, mangling the output and producing the following in the formatted output:

some text in frontmiddletail text

Among the consequences is that if "middle" contained a reference to an object like a class name, the result won't be hyperlinked (because it's mangled and Doxygen itself fails to recognize it). Here is an example of where it causes problems. Suppose your code has this comment:

/**

  • Blah blah blah.
  • Use the method @if java Compartment::setId(String id)@else setId()@endif.
    */

The result in the formatted output will be:

Blah blah blah.

Use the methodsetId().

Doxygen 1.6.3 does not behave this way, and leaves the spaces in the final output. I cannot find this change listed in the ChangeLog. I also don't know how to work around this behavior.

If I'm missing something obvious, or an explanation or a workaround, I apologize, and would appreciate it if someone could explain it to me.

Thanks. BTW, Doxygen has been just a fantastic tool for us. We use it to produce the documentation for our open-source library, libSBML.

On 2011-12-10 12:19:46 +0000, Dimitri van Heesch wrote:

This change was made to fix bug # 620924. For inline @if..@endif constructs like in your example this doesn't work. I'll come up with a better solution in the next release.

On 2011-12-10 20:39:26 +0000, Dimitri van Heesch wrote:

This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.7.6.1. Please verify if this is indeed the case. Reopen the
bug if you think it is not fixed and please include any additional information
that you think can be relevant.

On 2011-12-15 02:05:21 +0000, MikeH wrote:

Hi,

Thanks for such a prompt response to this. I've switched to using 1.7.6.1, and have tested this situation, and it looks like one half of the problem is indeed solved: it doesn't eat the space in front of @if. However, it's still eating the space after @endif. I used the following input file to test it:

Then I ran Doxygen with ENABLED_SECTIONS containing "cpp". The result in the formatted output was:

a bd

a bd

abd

a bd

a cd

a cd

acd

a cd

From this, it appears that the whitespace after @endif is being removed. This is still a bit problematic when the construct is used inside a paragraph.

Thank you very much for your help so far!

Best regards,
MH

@doxygen doxygen closed this as completed Jul 2, 2018
@elsbrock
Copy link

elsbrock commented Jul 2, 2018

:(

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

No branches or pull requests

2 participants