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

Case insensitive go to definition #118

Open
nfrancque opened this issue Jan 16, 2019 · 9 comments
Open

Case insensitive go to definition #118

nfrancque opened this issue Jan 16, 2019 · 9 comments
Assignees
Labels
bug ST3 Dependent Requires bug or enhancement fix to ST3 to implement.

Comments

@nfrancque
Copy link

VHDL is case insensitive. Remove case requirements for going to definition of signal or entity on hover.

@Remillard
Copy link
Owner

Remillard commented Jan 17, 2019 via email

@nfrancque
Copy link
Author

Thank you!

@Remillard
Copy link
Owner

Unfortunately I am not certain I can do this. I have checked and duplicated your findings with the following:

entity foobar is
	
end entity foobar; 

architecture rtl of foobar is
	signal MYSIGNAL : std_logic; 
begin
	MYSIGNAL <= 5; 
	mysignal <= 4; 
	
end architecture rtl;

In this instance if you hover over MYSIGNAL it will show the definition, and if you hover over mysignal it won't. Both words are scoped identically and the defining line is scoped correctly. However Sublime Text is doing the match, not me, and ST seems to be doing a case sensitive search. Currently documentation does not indicate that there is a setting I can put in there for case insensitivity for matching, but there might be something undocumented. I'll check with the experts and see if there's anything I can do.

@Remillard
Copy link
Owner

Okay from a quick chat on Discord with other package developers, it turns out that this is a Sublime Text limitation currently. There's a link there to the issue on the editor bug page. So unfortunately there's nothing I can do about this one right now. Best suggestion is to adopt a good coding style (like chained_lowercase_words and stick to it. I know that is not always possible if handed code done by someone else though.

@Remillard
Copy link
Owner

Maybe I need a new label for ST bug.

@Remillard
Copy link
Owner

This is now an enhancement request for Sublime Text here: sublimehq/sublime_text#2571

@Remillard Remillard added ST3 Dependent Requires bug or enhancement fix to ST3 to implement. and removed wontfix labels Jan 17, 2019
@nfrancque
Copy link
Author

Thanks for submitting that. Appreciate the responsiveness.

@Remillard
Copy link
Owner

Remillard commented Jan 18, 2019 via email

@nfrancque
Copy link
Author

My specific use case is entities being named in all caps but instantiated in lower case. Not a big deal, but would be nice to have.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug ST3 Dependent Requires bug or enhancement fix to ST3 to implement.
Projects
None yet
Development

No branches or pull requests

2 participants