Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

[fix] newListener emits correct fn when using once #2826

Closed
wants to merge 2 commits into from
Closed

[fix] newListener emits correct fn when using once #2826

wants to merge 2 commits into from

Conversation

fent
Copy link

@fent fent commented Feb 24, 2012

The event emitter once function wraps the listener around a self removing function. In that case the wrapping function gets emitted in newListener event instead of the original listener.

This pull request fixes that.

@koichik
Copy link

koichik commented Feb 25, 2012

@fent Can you sign the CLA?

@fent
Copy link
Author

fent commented Feb 25, 2012

Signed the CLA.

Running make jslint did not show any warnings on patched files.

@koichik koichik closed this in db8940d Feb 25, 2012
@koichik
Copy link

koichik commented Feb 25, 2012

@fent Thansk! merged in db8940d.

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

Successfully merging this pull request may close these issues.

None yet

2 participants