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

Possible EventEmitter memory leak #236

Open
hemendpatel opened this issue May 12, 2023 · 0 comments
Open

Possible EventEmitter memory leak #236

hemendpatel opened this issue May 12, 2023 · 0 comments

Comments

@hemendpatel
Copy link

hemendpatel commented May 12, 2023

I'm creating a terminal app and noticed that if we create more than 10 instances of terminal, it is throwing below error. All session are not active at the same time. I have destroyed document and terminal object on close of each session using below code.
image

image

Each time when we create a new terminal, we are subscribing to process exit event but can't find an option to unsubscribe or verify before subscribing to same event again and again.

terminal-kit\lib\Terminal.js:
image

please guide me if I am missing anything here.

do we have to set MaxListeners to Infinity?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant