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

Error when adding ovr folder as a symlink in workspace directory to get syntax highlighting for nw_inc_nui #31

Closed
Liareth opened this issue Jul 30, 2022 · 2 comments
Assignees
Labels
Projects
Milestone

Comments

@Liareth
Copy link
Contributor

Liareth commented Jul 30, 2022

c:\Users\Lia\.vscode\extensions\philippechab.nwscript-ee-language-server-1.4.0\server\out\Tokenizer\Tokenizer.js:50
        return line.slice(token.startIndex, token.endIndex);
                                ^

TypeError: Cannot read properties of undefined (reading 'startIndex')
    at Tokenizer.getRawTokenContent (c:\Users\Lia\.vscode\extensions\philippechab.nwscript-ee-language-server-1.4.0\server\out\Tokenizer\Tokenizer.js:50:33)
    at Tokenizer.getTokenLanguageType (c:\Users\Lia\.vscode\extensions\philippechab.nwscript-ee-language-server-1.4.0\server\out\Tokenizer\Tokenizer.js:56:33)
    at Tokenizer.tokenizeLinesForGlobalScope (c:\Users\Lia\.vscode\extensions\philippechab.nwscript-ee-language-server-1.4.0\server\out\Tokenizer\Tokenizer.js:169:46)
    at Tokenizer.tokenizeContent (c:\Users\Lia\.vscode\extensions\philippechab.nwscript-ee-language-server-1.4.0\server\out\Tokenizer\Tokenizer.js:293:25)
    at generateTokens (c:\Users\Lia\.vscode\extensions\philippechab.nwscript-ee-language-server-1.4.0\server\out\Documents\DocumentsIndexer.js:19:39)

The ovr folder contents are from the latest preview. I binary chopped the contents of the ovr folder until I figured out that simply including nw_inc_nui in your workspace causes this error.

@PhilippeChab PhilippeChab self-assigned this Jul 30, 2022
@PhilippeChab PhilippeChab added this to To do in LSP via automation Jul 30, 2022
@PhilippeChab
Copy link
Owner

Thank you for reporting this!

I have the same error with the current version of the file. I'll look into this.

@PhilippeChab PhilippeChab added this to the Beta milestone Jul 30, 2022
@PhilippeChab
Copy link
Owner

Fixed in 1.4.1.

LSP automation moved this from To do to Done Jul 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
LSP
Done
Development

No branches or pull requests

2 participants