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.

Sign up for GitHub

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

Jump to bottom

Missing/incorrect structure info for function named "string" #105

Closed
jfehrle opened this issue Oct 7, 2018 · 4 comments
Closed

Missing/incorrect structure info for function named "string" #105

jfehrle opened this issue Oct 7, 2018 · 4 comments

Comments

@jfehrle
Copy link
Collaborator

jfehrle commented Oct 7, 2018

plugin version: 0.63

Description

The structure info should show string rather that s (the first parameter of string). This seems specific to this exact function name; a function named string2 works as expected.

image
@giraud
Copy link
Owner

giraud commented Oct 14, 2018

Yes, I know why, but I won't fix it.
Better to not use a keyword to name your functions (even if it's possible)

@giraud giraud closed this as completed Oct 14, 2018
@jfehrle
Copy link
Collaborator Author

jfehrle commented Oct 14, 2018

OK. FWIW: technically it's a predefined type name, not a keyword. "bool", "false", etc. have the same behavior.

@giraud giraud reopened this Nov 10, 2018
@giraud
Copy link
Owner

giraud commented Nov 10, 2018

I'm reconsidering it

giraud added a commit that referenced this issue Nov 16, 2018
@giraud
#105 - allow string in name identifiers
44a0f5b
@giraud
Copy link
Owner

giraud commented Nov 16, 2018

0.67.2

@giraud giraud closed this as completed Nov 16, 2018
@jfehrle jfehrle mentioned this issue Nov 16, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@giraud @jfehrle