Skip to content
This repository has been archived by the owner on Jun 30, 2020. It is now read-only.

simple hover for systemverilog variables and parameters #16

Merged
merged 4 commits into from
Mar 24, 2017

Conversation

Yushiao
Copy link
Contributor

@Yushiao Yushiao commented Mar 23, 2017

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
Copy link
Owner

mshr-h commented Mar 24, 2017

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
Copy link
Contributor Author

Yushiao commented Mar 24, 2017

Hi,

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

But typedef, struct, and enum are not working now

@mshr-h
Copy link
Owner

mshr-h commented Mar 24, 2017

Looks good to me! Cheers! 🍻

@mshr-h mshr-h merged commit 36baa0f into mshr-h:master Mar 24, 2017
mshr-h added a commit that referenced this pull request Mar 24, 2017
@Yushiao Yushiao deleted the variable_hover branch March 27, 2017 06:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants