Skip to content
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

does ALIAS work for VHDL code? (Origin: bugzilla #652086) #4289

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

does ALIAS work for VHDL code? (Origin: bugzilla #652086) #4289

doxygen opened this issue Jul 2, 2018 · 0 comments

Comments

@doxygen
Copy link
Owner

doxygen commented Jul 2, 2018

status RESOLVED severity major in component build for ---
Reported in version 1.7.1 on platform Other
Assigned to: Dimitri van Heesch

Original attachment names and IDs:

On 2011-06-07 22:56:01 +0000, David Belohrad wrote:

I have specified an alias in the Doxyfile as follows:

ALIASES += register_definition{3}="\1 booh \2"

then I put comment into the VHDL comment block:

--! @register_definition{0x680014}{32}{Turn Clock Watchdog}

and now, during compilation I get:

/home/belohrad/SVN/fbctdab/branches/06_improving_bpf/stratix_fbct/sources/newstyle/tdclk_watchdog/tdclk_watchdog.vhd:103: warning: Found unknown command
`\register_definition'

and of course HTML does not contain the aliased code.

Does the ALIAS work for VHDL code?

On 2011-06-11 10:00:38 +0000, mk wrote:

I tried the alias example from the manual ALIASES += Bold{1}="\1"
--! signal \Bold{main clk}
clk : in STD_LOGIC;

and it works.

try version 1.7.4

On 2013-10-21 18:24:50 +0000, albert wrote:

As far as I can see the usage of the ALIAS is not conform the documentation and should be like:
@register_definition{0x680014,32,Turn Clock Watchdog}

The remark of mk is just for an ALIAS with a single argument.

I checked it with version 1.8.5.

I think this problem can be set to: resolved, notabug

On 2017-05-30 09:29:02 +0000, JK wrote:

In version 1.8.6 and further it does not work anymore!

: warning: Found unknown command `\register_definition'

Obviously there's a change in that version!

On 2017-05-30 16:54:29 +0000, albert wrote:

  1. it does make sense a lot of sense to add a comment to a closed issue as in most cases this people will filter the issues and not display resolved issues. So either create an new issue or reopen the current issue but in both cases:

attach a (small) self-contained example (source+config file in a tar or zip) that allows us to reproduce the problem?

and also indicate the results with the latest version (1.8.13) of doxygen.

On 2017-05-31 06:30:05 +0000, JK wrote:

Created attachment 352921
ALIAS with VHDL code in Doxygen 1.8.5/1.8.6 and 1.8.13

I've made a simple testcase with a counter (VHDL code):

In Doxygen version 1.8.5 the ALIAS works, but in further versions it does not word ("unknown command").

Just look at the HTML output of each Doxygen version.

On 2017-05-31 19:17:07 +0000, mk wrote:

try register_definition{2}="\1 booh \2"

change

@reg{Counter register,Down counter with n bits}
in @register_definition{Counter register,Down counter with n bits}
in your example

On 2017-06-01 05:36:55 +0000, JK wrote:

Created attachment 352980
Updated example: ALIAS with VHDL code in Doxygen 1.8.5/1.8.6 and 1.8.13

Thanks, but the phrase "reg" or "register_definition" isn't the critical issue.
It's rather the processing of the ALIAS which seems to be "skipped" in version 1.8.6 and above.

I've updated my example to match the previous example and show that it is a bug.

On 2017-06-04 10:45:13 +0000, albert wrote:

I've just pushed a proposed patch to github (pull request 571)

On 2017-06-19 08:33:51 +0000, JK wrote:

For my application and testcase your patch works!
Thanks a lot!

On 2017-08-21 11:39:35 +0000, albert wrote:

Code integrated in master version on github.

On 2017-12-25 18:43:55 +0000, Dimitri van Heesch wrote:

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

@doxygen doxygen closed this as completed Jul 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant