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 646533 - [patch] sorting of overloaded member functions
[patch] sorting of overloaded member functions
Status: RESOLVED FIXED
Product: doxygen
Classification: Other
Component: general
1.7.4-SVN
Other Linux
: Normal normal
: ---
Assigned To: Dimitri van Heesch
Dimitri van Heesch
Depends on:
Blocks:
 
 
Reported: 2011-04-02 14:30 UTC by Marcin Wojdyr
Modified: 2011-06-06 11:52 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch (778 bytes, patch)
2011-04-02 14:30 UTC, Marcin Wojdyr 		none Details | Review

Description Marcin Wojdyr 2011-04-02 14:30:54 UTC 
Created attachment 184945 [details] [review]
patch

I've noticed that with SORT_MEMBER_DOCS=YES the order of overloads is undefined, due to unstable sort used to sort member list.
IMHO it would be much better to have the order of overloads the same as in the input file, because it would allow using @overload with SORT_MEMBER_DOCS=YES (@overload writes something about "the above function", so it is confusing if the order changes).

I've made a quick fix (attached) that adds sorting by getDefLine() if it happens that the names are the same. I'm not familiar with doxygen code, so you may find a better solution.
Comment 1 Dimitri van Heesch 2011-04-03 10:22:42 UTC 
Thanks, looks like a neat solution, I'll include the patch in the next subversion update.
Comment 2 Marcin Wojdyr 2011-06-06 11:52:45 UTC 
Apparently it has been done. Thanks! So I'm closing this ticket.