Skip to content
This repository was archived by the owner on Feb 26, 2024. It is now read-only.

fix(event): fix #1110, nodejs EventEmitter should support Symbol eventName - #1113

Merged
mhevery merged 1 commit into
angular:masterfrom
JiaLiPassion:issue-1110
Jan 8, 2019
Merged

fix(event): fix #1110, nodejs EventEmitter should support Symbol eventName#1113
mhevery merged 1 commit into
angular:masterfrom
JiaLiPassion:issue-1110

Conversation

@JiaLiPassion

@JiaLiPassionJiaLiPassion commented Jul 17, 2018

Copy link
Copy Markdown
Collaborator

fix#1110.

In nodejs, EventEmitter should support Symbol as eventName.

constsymbolA=Symbol('A');emitter.addListener(symbolA,listener);emitter.removeListener(symbolA,listener);

@JiaLiPassionJiaLiPassion changed the title fix(event): fix #1110, nodejs EventEmitter should support Symbol eventNameWIP(event): fix #1110, nodejs EventEmitter should support Symbol eventNameJul 17, 2018
@JiaLiPassion
JiaLiPassionforce-pushed the issue-1110 branch 2 times, most recently from 6f017d5 to 3fe873aCompareJuly 18, 2018 01:31
@JiaLiPassionJiaLiPassion changed the title WIP(event): fix #1110, nodejs EventEmitter should support Symbol eventNamefix(event): fix #1110, nodejs EventEmitter should support Symbol eventNameJul 18, 2018
@jreidgreer

jreidgreer commented Aug 7, 2018

Copy link
Copy Markdown

@JiaLiPassion do you happen to know when this will get merged? Thanks!

@jreidgreer

jreidgreer commented Oct 26, 2018

Copy link
Copy Markdown

@JiaLiPassion do you have an update on when this might be merged in?

@JiaLiPassion

JiaLiPassion commented Oct 27, 2018

Copy link
Copy Markdown
CollaboratorAuthor

@jreidgreer, I am not sure, If you want to test, I can create a temp release branch for you.

@Mathyn

Mathyn commented Nov 6, 2018

Copy link
Copy Markdown

Any updates on this? When can we expect this to be released?

@JiaLiPassionJiaLiPassion mentioned this pull request Dec 13, 2018
Merged
@mhevery
mhevery merged commit 96420d6 into angular:masterJan 8, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer
@mheverymheverymhevery approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla: yes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

makeAddListener breaks when the Event Name is a Symbol

5 participants

@JiaLiPassion@jreidgreer@Mathyn@mhevery@googlebot