-
Notifications
You must be signed in to change notification settings - Fork 66
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
Use EventTarget instead EventEmitter #85
Comments
If you'd like to take some features under your control - feel free. There's plenty of work tbd, you have great experience with it. I want to focus on dsp layer - it needs careful thought. |
I gathered some further information. I discovered that Should we switch from |
Nah, totally fine dropping old stuff. Not planning to do any backwards-compat. v1 should be fresh new, with ESM and good architecture. |
I'm happy to hear that. :-) |
As of recently Node.js supports the same (or similar)
EventTarget
as the browser. Since the Web Audio spec is also referring to theEventTarget
in various places it might be worth considering to replace the currentEventEmitter
usage.https://nodejs.org/dist/latest-v16.x/docs/api/events.html#eventtarget-and-event-api
The text was updated successfully, but these errors were encountered: