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.

Sign up for GitHub

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

Jump to bottom

MaxListenersExceededWarning in repl #18284

Closed
maximelkin opened this issue Jan 21, 2018 · 3 comments
Closed

MaxListenersExceededWarning in repl #18284

maximelkin opened this issue Jan 21, 2018 · 3 comments
Labels
events Issues and PRs related to the events subsystem / EventEmitter. repl Issues and PRs related to the REPL subsystem.

Comments

@maximelkin
Copy link

maximelkin commented Jan 21, 2018 •
edited
Loading 

$ node
> async function test() {
... await Promise.resolve()<many tab clicks>

causes

(node:9133) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 Runtime.executionContextCreated listeners added. Use emitter.setMaxListeners() to increase limit
@vsemozhetbyt vsemozhetbyt added events Issues and PRs related to the events subsystem / EventEmitter. repl Issues and PRs related to the REPL subsystem. labels Jan 21, 2018
@yasinaydinnet
Copy link

What do you mean by <many tab clicks>?

@sallen450
Copy link
Contributor

I think it means that click tab multi times. I reproduce it on Node v9.4.0. It's ok on Node v8.9.4

@killagu
Copy link
Contributor

killagu commented Feb 20, 2018

@maximelkin It's node repl bug.I will make a PR to fix it.

killagu added a commit to killagu/node that referenced this issue Feb 20, 2018
@killagu
repl: fix tab-complete warning
f6a77fc 
When create a nest repl, will register `Runtime.executionContextCreated`
listener to the inspector session.This patch will fix listener
repeatedly register.

Fixes: nodejs#18284
@killagu killagu mentioned this issue Feb 20, 2018
Closed
3 tasks
addaleax pushed a commit to addaleax/node that referenced this issue Feb 26, 2018
@killagu @addaleax
repl: fix tab-complete warning
f6e2ba4 
When create a nest repl, will register `Runtime.executionContextCreated`
listener to the inspector session.This patch will fix listener
repeatedly register.

PR-URL: nodejs#18881
Fixes: nodejs#18284
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
addaleax pushed a commit that referenced this issue Feb 26, 2018
@killagu @addaleax
repl: fix tab-complete warning
01398b2 
When create a nest repl, will register `Runtime.executionContextCreated`
listener to the inspector session.This patch will fix listener
repeatedly register.

PR-URL: #18881
Fixes: #18284
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
MayaLekova pushed a commit to MayaLekova/node that referenced this issue May 8, 2018
@killagu @MayaLekova
repl: fix tab-complete warning
709532c 
When create a nest repl, will register `Runtime.executionContextCreated`
listener to the inspector session.This patch will fix listener
repeatedly register.

PR-URL: nodejs#18881
Fixes: nodejs#18284
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
events Issues and PRs related to the events subsystem / EventEmitter. repl Issues and PRs related to the REPL subsystem.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

repl: fix tab-complete warning killagu/node
5 participants
@yasinaydinnet @sallen450 @killagu @vsemozhetbyt @maximelkin