vscode-terosHDL icon indicating copy to clipboard operation
vscode-terosHDL copied to clipboard

comment of 1st constant or signal in Architecture is ignored for documentation

Open thklumpp opened this issue 3 years ago • 9 comments

In this case:

...
ARCHITECTURE rtl OF AXI4_Error_Sync IS
  --! This comment will be ignored for documentation
  constant sync_ffs_c    : integer := 2;
...

thklumpp avatar Jan 14 '22 15:01 thklumpp

Your issues are very useful!! I will fix it, but I will take me a time... since I need to update the VHDL parser, and that is very hard.

qarlosalberto avatar Jan 14 '22 15:01 qarlosalberto

Ok. And thanks a lot for this great extension! Is there a workaround so far? A dummy comment or so?

thklumpp avatar Jan 14 '22 15:01 thklumpp

You need a VHDL statement. E.g:

image

qarlosalberto avatar Jan 14 '22 15:01 qarlosalberto

And you can select "only commented" if you want to delete de dummy signal/type/constant:

image

image

qarlosalberto avatar Jan 14 '22 15:01 qarlosalberto

For process you can create a dummy assign:

image

qarlosalberto avatar Jan 14 '22 15:01 qarlosalberto

In general you can see that you need a dummy statement (declaration, assign...) before the real first comment.

qarlosalberto avatar Jan 14 '22 15:01 qarlosalberto

Very helpful, thanks!

thklumpp avatar Jan 14 '22 15:01 thklumpp

Hi. I justed wanted to ask if there is a rough date for this fix. Or also, if this could be end of the year. Or already next month. It is only, that I can decide how to proceed to use TerosHDL or not - as the dummy assignments are not a nice option for me. Anything helps. Thanks.

thklumpp avatar Mar 02 '22 09:03 thklumpp

It will be fixed this year, but not in the following months. Maybe at the end.

El mié., 2 mar. 2022 10:19, thklumpp @.***> escribió:

Hi. I justed wanted to ask if there is a rough date for this fix. Or also, if this could be end of the year. Or already next month. It is only, that I can decide how to proceed to use TerosHDL or not - as the dummy assignments are not a nice option for me. Anything helps. Thanks.

— Reply to this email directly, view it on GitHub https://github.com/TerosTechnology/vscode-terosHDL/issues/291#issuecomment-1056653298, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABNN4R5LSD7ADQEQTS7VX2TU54XDLANCNFSM5L7BIAQA . You are receiving this because you commented.Message ID: @.***>

qarlosalberto avatar Mar 02 '22 09:03 qarlosalberto

Fixed on v5.0.0

qarlosalberto avatar Jun 26 '23 22:06 qarlosalberto