DavidCai1111
changed the title
events: make sure the listener added by EventEmitter#once will only be called once
events: pass the original listener added by EventEmitter#once to the removeListener handler
Closed
4 tasks
jasnell pushed a commit that referenced this pull request
Apr 29, 2016When removing a `once` listener, the listener being passed to the `removeListener` callback is the wrapper. This unwraps the listener so that `removeListener` is passed the actual listener. PR-URL: #6394 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Closed
cjihrig
added
the
semver-major
label
Apr 29, 2016
cjihrig
removed
the
semver-major
label
Apr 29, 2016Fishrock123 pushed a commit that referenced this pull request
May 4, 2016When removing a `once` listener, the listener being passed to the `removeListener` callback is the wrapper. This unwraps the listener so that `removeListener` is passed the actual listener. PR-URL: #6394 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
This was referenced
May 4, 2016Closed
Merged
joelostrowski pushed a commit to joelostrowski/node that referenced this pull request
May 4, 2016When removing a `once` listener, the listener being passed to the `removeListener` callback is the wrapper. This unwraps the listener so that `removeListener` is passed the actual listener. PR-URL: nodejs#6394 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Closed
Closed
4 tasks
Merged
3 tasks
MylesBorins pushed a commit that referenced this pull request
Nov 22, 2016When removing a `once` listener, the listener being passed to the `removeListener` callback is the wrapper. This unwraps the listener so that `removeListener` is passed the actual listener. PR-URL: #6394 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
Merged