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 712251 - Documentation for enumeration not generated
Documentation for enumeration not generated
Status: RESOLVED FIXED
Product: doxygen
Classification: Other
Component: general
1.8.5
Other All
: Normal major
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
Depends on:
Blocks:
 
 
Reported: 2013-11-13 21:34 UTC by jraphanel
Modified: 2014-04-21 10:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Source and configuration files (26.66 KB, application/zip)
2013-11-25 08:16 UTC, jraphanel
Details

Description jraphanel 2013-11-13 21:34:25 UTC
Hello,

  When SEPARATE_MEMBER_PAGES option is set to Yes with html documentation, the comment about an enumeration stored in a class is not generated properly. In fact, after analysis the current doxygen implementation, it appears that if the same enumeration is not including into a class definition, the documentation is generated properly. After, I tried to check why the documentation was not generated properly if I put the same enumeration included along a class definition. I found in src\types.h the following lines:
MemberListType_enumValMembers          = 31 + MemberListType_detailedLists,

If I understand well this line, it signifies that an enumeration values might have some detailed information that might be used to generated a separated HTML page. From my point of view, only an enumeration might have detailed information and in case of enumeration value it is not the case, so it is the reason why I suggest the following modification:

MemberListType_enumValMembers          = 31 /*+ MemberListType_detailedLists*/,

In fact, I just suggest to remove the usage of MemberListType_detailedLists option in case of enumeration value.

I made some tests for HTML output and I don't see side effect, but I am not sure to have a clear vision about the impact of such modification.
What do you think of this suggestion?

Thanks and Regards,
Jacques
Comment 1 Dimitri van Heesch 2013-11-23 13:54:06 UTC
Can you please attach a self-contained example (source+config file in a tar or zip) that allows me to reproduce the problem?
Comment 2 Dimitri van Heesch 2013-11-23 13:54:29 UTC
Can you please attach a self-contained example (source+config file in a tar or zip) that allows me to reproduce the problem?
Comment 3 jraphanel 2013-11-25 08:16:36 UTC
Created attachment 261385 [details]
Source and configuration files

Please find in attached file the requested files.
enum.h and enum.cfg have been fetch from examples directory and from enum.cfg I just activate 'SEPARATE_MEMBER_PAGES' option.

Regards,
Jacques
Comment 4 jraphanel 2014-01-06 08:37:37 UTC
Dear Dimitri,

  Is it possible to know if can reproduce the reported issue? If yes, do you know when this bug shall be solved?

Thanks and regards,
Jacques
Comment 5 jraphanel 2014-03-06 11:27:11 UTC
Dear Dimitri,

Sorry to disturb you, but I need to know if you think that I am right when I report my bug? In fact, for me it is always a pleasure to use doxygen because it is a very useful tool, but with this bug, I can't use official doxygen version and from my point of view, it is not very good. So at, can you confirm me if I found a bug or not and at least if yes, do you plan to fix it or not based on my comment?

In addition to that, I want to precise you that I don't submit a patch because I am not familiar to git, but I can make an effort to provide an official patch if it can help you.

Thanks and regards,
Jacques
Comment 6 Dimitri van Heesch 2014-04-20 10:57:25 UTC
I Jacques,

Sorry for the slow response so far. 

I have just committed a change that should address the issue.
https://github.com/doxygen/doxygen/commit/1bb36723a522b371810606c2f6504d0374a7b027

It hides the enum values from the member page index (since they don't have an associated page). It is different from your proposal, so please check if it indeed solves your problem.
Comment 7 Dimitri van Heesch 2014-04-21 10:09:31 UTC
This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.8.7. 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 (preferrably in the form of a self-contained example).