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

[patch] sorting of overloaded member functions (Origin: bugzilla #646533) #4229

Closed
doxygen opened this Issue Jul 2, 2018 · 0 comments

Comments

Projects
None yet
1 participant
@doxygen
Owner

doxygen commented Jul 2, 2018

status RESOLVED severity normal in component general for ---
Reported in version 1.7.4-SVN on platform Other
Assigned to: Dimitri van Heesch

Original attachment names and IDs:

On 2011-04-02 14:30:54 +0000, Marcin Wojdyr wrote:

Created attachment 184945
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.

On 2011-04-03 10:22:42 +0000, Dimitri van Heesch wrote:

Thanks, looks like a neat solution, I'll include the patch in the next subversion update.

On 2011-06-06 11:52:45 +0000, Marcin Wojdyr wrote:

Apparently it has been done. Thanks! So I'm closing this ticket.

@doxygen doxygen closed this Jul 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment