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

big max script file is really slow to navigate #9

Open
elpie89 opened this issue Jun 22, 2021 · 5 comments
Open

big max script file is really slow to navigate #9

elpie89 opened this issue Jun 22, 2021 · 5 comments
Labels
enhancement New feature or request help wanted Extra attention is needed wontfix This will not be worked on

Comments

@elpie89
Copy link

elpie89 commented Jun 22, 2021

I have a max script file with more than 20K lines of codes
I cannot use this plugin with that code as vscode is slowed down and almost impossible to navigate
Disabling the plugin everything works well
is there a way to customize/improve performances? maybe adding memory for big file size etc?

@HAG87
Copy link
Owner

HAG87 commented Jun 26, 2021

Yes, I know; limitations of writing the parser in JS. Not much that can be done, either:

  • Disabling the parser for long files (losing the outliner and code navigation) or maybe a simpler method to get code symbols, without structure.
  • Modularize your code in shorter scripts,
  • Write a compiled parser in ANTLR4

You're welcome to collaborate if you can do some improvements to the code.

@HAG87 HAG87 added this to the Optimization, fixes milestone Jun 26, 2021
@HAG87 HAG87 added enhancement New feature or request wontfix This will not be worked on labels Jun 26, 2021
@HAG87 HAG87 pinned this issue Jun 26, 2021
@Trebor777
Copy link

About antlr4 , something to get you started
https://gitlab.com/MrPingouin/mxs-linter

@HAG87
Copy link
Owner

HAG87 commented Oct 7, 2021

@Trebor777 Could you contribute to the extension? I think unary minus syntax will be very problematic to resolve, also, once the grammar is done, a replacement for the post-processors needs to be done.

@Trebor777
Copy link

@HAG87 If only I had the time ^^' ... Sadly no. not yet anyway. Since mentionned it I only took 5min to look for it to see if there was anything and thought it might help you.
Awesome work on the current extension btw :) thanks a lot.

@HAG87
Copy link
Owner

HAG87 commented Oct 11, 2021 via email

@HAG87 HAG87 added the help wanted Extra attention is needed label Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants