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

Rapid memory leak when the type definition file contains declare class Widget state: {string: function(string, Widget)} end #756

Closed
coderedart opened this issue Sep 2, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@coderedart
Copy link

declare class Widget
    state: {string: function(string, Widget)}
end

If I copy paste the above text into a type definition file and reload the lsp, luau-lsp starts leaking memory at around 100MB+ per second until the system runs out of ram.

The culprit seems to be {string: function(string, Widget)}. The syntax is probably wrong. If I use (string, Widget) -> nil instead, there's no leak.

But just wanted to report the leak anyway, as someone else might fumble into this issue.

@JohnnyMorganz JohnnyMorganz added the bug Something isn't working label Sep 11, 2024
@JohnnyMorganz
Copy link
Owner

I think this might be a Luau issue which is best to report upstream. If they say it isn't then I'll dig deeper

@JohnnyMorganz
Copy link
Owner

Moved this upstream: luau-lang/luau#1520

@JohnnyMorganz JohnnyMorganz closed this as not planned Won't fix, can't repro, duplicate, stale Nov 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants