-
Notifications
You must be signed in to change notification settings - Fork 23
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
Trailing . removed from token #35
Comments
Take a look at the formal spec for identifier:
In short an identifier can't end with a period. The lexer reads |
Ahhh - this likely explains why the Grammar treats I could fix this on the grammar side by treating Any suggestions on how to resolve? I guess we could solve this within the intellisense code itself... if we find the last char is |
As far as I know there's no situation where an identifier followed by a dot isn't intended to be an identifier. I see two main options:
Adding an additional flag would more tightly couple the parser's code with the VSCode extension, which is something I'd like to avoid if possible. That said if it's not easy to implement the fix in the Intellisense code and/or it requires a hacky solution then I'd prefer a flag over code separation ideology. |
Input text:
When I retrieve the lexer state, I'd expect a single token containing
Access.
, but it looks like the trailing . is removed. The resulting LineTokens are equal to:The text was updated successfully, but these errors were encountered: