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

Label "abstract" instead of "pure virtual" for Java language (Origin: bugzilla #719489) #5364

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

Comments

Projects
None yet
1 participant
@doxygen
Owner

doxygen commented Jul 2, 2018

status VERIFIED severity normal in component general for ---
Reported in version 1.8.5 on platform Other
Assigned to: Dimitri van Heesch

Original attachment names and IDs:

On 2013-11-28 13:30:06 +0000, Volodymyr wrote:

Created attachment 263021
Example of label

Hello,

I use C++ language and generate html documentation with the help of doxygen. If I have virtual method - doxygen add pretty label "virtual" in right corner of function description. Looks good!
But if I use Java - doxygen add label "virtual" for abstract function too.
If I set OPTIMIZE_OUTPUT_JAVA = YES, it does not help.

I propose for case OPTIMIZE_OUTPUT_JAVA = YES, create label "abstract" instead of "virtual".

On 2013-11-28 13:36:20 +0000, Volodymyr wrote:

Created attachment 263022
Example of label

Example of label

On 2013-11-28 13:43:56 +0000, Volodymyr wrote:

And it would be a good idea to add similar label override for override functions

On 2013-12-01 17:43:20 +0000, albert wrote:

Can you please attach a self-contained example (source+config file in a tar or
zip) that allows reproduction of the problem?

On 2013-12-02 09:17:10 +0000, Volodymyr wrote:

Created attachment 263273
Example of abstract method

Zip archive contains example of source java file, doxygen configuration, html output result and main idea - png result image.

On 2013-12-02 19:12:43 +0000, albert wrote:

I've pushed a proposed patch (pull request 64) to git

On 2013-12-09 07:50:49 +0000, Volodymyr wrote:

Thank you!

On 2013-12-24 18:59:22 +0000, Dimitri van Heesch wrote:

This bug was previously marked ASSIGNED, which means it should be fixed in
doxygen version 1.8.6. 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).

On 2013-12-25 13:37:19 +0000, Volodymyr wrote:

I have verified it. Now it's good.

@doxygen doxygen closed this Jul 19, 2018

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