search

mshr-h / vscode-systemverilog-support

[deprecated]use mshr-h/vscode-verilog-hdl-support
https://github.com/mshr-h/vscode-verilog-hdl-support
MIT License
22 stars 14 forks source link

simple hover for systemverilog variables and parameters #16

Closed Yushiao closed 7 years ago

Yushiao commented 7 years ago

Hello

I made a feature that show the line where the variable declare when mouse hover to the variable

like pictures below 01 02

If there are any problems, let me know

mshr-h commented 7 years ago

First of all, thank you for sending me your great PR.

I have a few comments below.

  1. Variables defined as inout is also port. (https://www.hdlworks.com/hdl_corner/verilog_ref/items/PortDeclaration.htm)
  2. There are some integer and real types not only int but also byte, long, etc... (https://www.doulos.com/knowhow/sysverilog/tutorial/datatypes/)
Yushiao commented 7 years ago

Hi,

I update the supported datatypes for byte, long, etc...

But typedef, struct, and enum are not working now

mshr-h commented 7 years ago

Looks good to me! Cheers! 🍻