Repository navigation
TypeError: EventEmitter, setMaxListeners, Symbol #9003
Description
- Version: 6.7.0
- Platform: Win 10 @ 64-bit
Expected Behavior
(node:3712) Warning: Possible EventEmitter memory leak detected.
2 Symbol("X") listeners added. Use emitter.setMaxListeners() to increase limitCurrent Behavior
TypeError: Cannot convert a Symbol value to a stringSteps to Reproduce
"use strict";
const EventEmitter = require("events");
const ev = new EventEmitter(),
sym = Symbol("X"); // "X";
ev
.setMaxListeners(1)
.on(sym, console.log)
.on(sym, console.log)
.emit(sym, 200);Stack trace
events.js:260
`${existing.length} ${type} listeners added. ` +
^
TypeError: Cannot convert a Symbol value to a string
at _addListener (events.js:260:51)
at EventEmitter.addListener (events.js:275:10)
at Object.<anonymous> (J:\main.js:13:6)
at Module._compile (module.js:556:32)
at Object.Module._extensions..js (module.js:565:10)
at Module.load (module.js:473:32)
at tryModuleLoad (module.js:432:12)
at Function.Module._load (module.js:424:3)
at Module.runMain (module.js:590:10)
at run (bootstrap_node.js:394:7)
Activity
events,test: fix TypeError in EventEmitter warning
events,test: fix TypeError in EventEmitter warning
polomsky commentedon Apr 16, 2017
@jasnell It is fixed in master branch but still not fixed in LTS (6.x)
jasnell commentedon Apr 16, 2017
Ok. Will have to investigate why that didn't get pulled into a release.
Ping... @nodejs/lts
gibfahn commentedon Apr 16, 2017
@jasnell looks like it needs a backport: #9021 (comment)
events,test: fix TypeError in EventEmitter warning
events,test: fix TypeError in EventEmitter warning
events,test: fix TypeError in EventEmitter warning
Trott commentedon Aug 2, 2017
It looks like the backport loanded in April. Closing. Feel free to comment or re-open if I'm wrong and this is still a thing.
events,test: fix TypeError in EventEmitter warning