After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 748208 - Bug #313527 regression - Enum in bitfield is not parsed properly.
Bug #313527 regression - Enum in bitfield is not parsed properly.
Status: RESOLVED FIXED
Product: doxygen
Classification: Other
Component: general
1.8.9.1-GIT
Other Cygwin
: Normal normal
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
Depends on:
Blocks:
 
 
Reported: 2015-04-20 18:53 UTC by Jonathan Duncan
Modified: 2015-06-27 19:02 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Test case that causes error. (476 bytes, text/x-csrc)
2015-04-20 18:53 UTC, Jonathan Duncan
  Details
Broken HTML output (5.21 KB, text/html)
2015-04-20 18:54 UTC, Jonathan Duncan
  Details
patch of proposed fix. (1.01 KB, patch)
2015-04-20 18:58 UTC, Jonathan Duncan
none Details | Review

Description Jonathan Duncan 2015-04-20 18:53:51 UTC
Created attachment 302020 [details]
Test case that causes error.

Bug #313527 which was fixed seems to have regressed during changes to add support for c++11 explicitly typed enums.

The version that reintroduced the bad behaviour was at least "Release-1.8.1.2-20120729".

Reproduction:

    enum testenum {
        ITEM_ONE,
        ITEM_TWO,
    };

    /** testing struct */
    struct testing {
        enum testenumtest1:2; ///< This field does not get parsed properly
        unsigned int test2:30; ///< other field
    };

The documentation for "struct testing" gets mangled as follows:

    unsigned int 	test2:30
 	    < This is the field that does not get parsed properly More...

I have found fix that seems to work, however my knowledge of the doxygen source code is not good so it may not be the best way of doing this.
Comment 1 Jonathan Duncan 2015-04-20 18:54:50 UTC
Created attachment 302021 [details]
Broken HTML output
Comment 2 Jonathan Duncan 2015-04-20 18:58:41 UTC
Created attachment 302022 [details] [review]
patch of proposed fix.

Added a patch with proposed fix.
Comment 3 Jonathan Duncan 2015-04-20 19:00:05 UTC
I've also created a pull request on github if that is preferred.
Comment 4 Jonathan Duncan 2015-04-21 18:45:48 UTC
The first patch is breaks several tests. I'm working on a better solution.
Comment 5 Jonathan Duncan 2015-04-21 20:03:34 UTC
The last comment was made in error. It does not refer to this issue.
Comment 6 Dimitri van Heesch 2015-05-03 07:52:04 UTC
Confirmed. Should be fixed in the next GIT update.
Comment 7 Dimitri van Heesch 2015-06-27 19:02:00 UTC
This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.8.10. 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 (preferably in the form of a self-contained example).