-
Notifications
You must be signed in to change notification settings - Fork 139
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
copyTo from planar to interleave should copy / convert all channels #819
Comments
I think that's possibly already written down. The description of an interleaved buffer in 9.3.1. Arrangement of audio buffer says ...
But |
It could be made optional in this case, clearly indicating intent. |
w3c/webcodecs#819 Differential Revision: https://phabricator.services.mozilla.com/D216561 bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1907438 gecko-commit: f909b83620331385e7bb8ef1291b7df7a310f9a5 gecko-reviewers: chunmin
w3c/webcodecs#819 Differential Revision: https://phabricator.services.mozilla.com/D216561 bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1907438 gecko-commit: f909b83620331385e7bb8ef1291b7df7a310f9a5 gecko-reviewers: chunmin
w3c/webcodecs#819 Differential Revision: https://phabricator.services.mozilla.com/D216561 bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1907438 gecko-commit: f909b83620331385e7bb8ef1291b7df7a310f9a5 gecko-reviewers: chunmin
w3c/webcodecs#819 Differential Revision: https://phabricator.services.mozilla.com/D216561 bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1907438 gecko-commit: f909b83620331385e7bb8ef1291b7df7a310f9a5 gecko-reviewers: chunmin
It is useful to copy all frames of an
AudioData
containing planar data to a buffer containing interleaved data, in one go. However,planeIndex
is required incopyTo
AudioDatacopyToOptions
, so one has to write:and this appears to copy all channels in Chrome (and soon, Firefox), despite the
planeIndex:0
bit. I think it's good behaviour and we should clarify this in the spec.The text was updated successfully, but these errors were encountered: