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

Typedef classes are not handled by Omnicomplete #225

Open
ESodikromo opened this issue Feb 15, 2023 · 3 comments
Open

Typedef classes are not handled by Omnicomplete #225

ESodikromo opened this issue Feb 15, 2023 · 3 comments
Assignees

Comments

@ESodikromo
Copy link

ESodikromo commented Feb 15, 2023

Classes generated through typedefs are not interpreted as having an "inherits" key when calling taglist() inside of s:GetInheritanceTags which breaks it. This can be demonstrated by something along the lines of having this

class base_class;
endclass

typedef base_class #(parameter) custom_class;

and evaluating

echom string(taglist('^custom_class$')[0])

after generating a tags file.
Notice the lack of "inherits", which is what s:GetInheritanceTags uses. The verbose messaging simply stop at Searching inheritance of <object>. It leads to failing to find any tags from the base classes.

This ties in with the other issues regarding typedefs. This is common in UVM, so it would be great if there was a solution. However, the taglist() output for typedefs seems rather unhelpful, so there might need to be a more robust typedef handling function.

Hope your family is doing well.

@vhda vhda self-assigned this Feb 21, 2023
@vhda
Copy link
Owner

vhda commented Mar 5, 2023

Have you followed these instructions on running ctags https://github.com/vhda/verilog_systemverilog.vim/blob/master/doc/verilog_systemverilog.txt#L113 ?
That is, are you using Universal Ctags and the arguments --extra=+q --fields=+i -n?

@vhda
Copy link
Owner

vhda commented Mar 5, 2023

I can see that the inheritance is not included in the tags file... I used to maintain that parser in Universal Ctags, but unfortunately I no longer have any free time to do that. Would you mind reporting this limitation there, in the hope that someone can look into this?

@vhda
Copy link
Owner

vhda commented May 10, 2023

Any updates?

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

2 participants