Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Clarify parameters in AudioWorkletProcessor.process() #2016

Closed
hoch opened this issue Aug 5, 2019 · 1 comment · Fixed by #2017
Closed

Clarify parameters in AudioWorkletProcessor.process() #2016

hoch opened this issue Aug 5, 2019 · 1 comment · Fixed by #2017
Assignees
@hoch
Projects
V1

Comments

@hoch
Copy link
Member

hoch commented Aug 5, 2019

Currently the description says:
https://www.w3.org/TR/webaudio/#dom-audioworkletprocessor-process

A map of string keys and associated Float32Arrays. parameters["name"] corresponds to the automation values of the AudioParam named "name".

The term map here can be confusing, because this is not an ES6 Map. A set of key-value pairs, or we could say an "associative array".
@hoch hoch self-assigned this Aug 5, 2019
@hoch hoch added this to Untriaged in V1 via automation Aug 5, 2019
@hoch
Copy link
Member Author

hoch commented Aug 7, 2019 •
edited

Actually, we can simply use this link: https://infra.spec.whatwg.org/#maps

Also perhaps it's better to use the WebIDL Record type.

@hoch hoch mentioned this issue Aug 7, 2019
Merged
@hoch hoch moved this from Untriaged to Ready for Editing in V1 Aug 8, 2019
@hoch hoch closed this as completed in #2017 Aug 8, 2019
V1 automation moved this from Ready for Editing to Done Aug 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hoch hoch

Labels
None yet
Projects
No open projects
V1
  
Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Clarify parameters argument in AudioWorkletProcessor.process() hoch/web-audio-api
1 participant
@hoch