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

EventDispatcher::removeEventListener failed to remove listener somtime. #4069

Merged
merged 2 commits into from Nov 2, 2013

Conversation

Projects
None yet
2 participants
@ledyba
Copy link
Contributor

ledyba commented Nov 2, 2013

_toAddedListener list is not searched in EventDispatcher::removeEventListener,
so this function often failed to remove event listener.

@dumganhar dumganhar merged commit 83467e4 into cocos2d:develop Nov 2, 2013

1 check failed

default The Travis CI build failed
Details
@dumganhar

This comment has been minimized.

Copy link

dumganhar commented Nov 2, 2013

Fixed at #4072 . Thanks.

@ledyba ledyba deleted the ledyba:fix_event_listener branch Nov 2, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment