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.

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

(AudioBufferSimpl): AudioBuffer simplification #80

Closed
olivierthereaux opened this issue Sep 11, 2013 · 4 comments
Closed

(AudioBufferSimpl): AudioBuffer simplification #80

olivierthereaux opened this issue Sep 11, 2013 · 4 comments
Milestone

Comments

@olivierthereaux
Copy link
Contributor

Originally reported on W3C Bugzilla ISSUE-17341 Tue, 05 Jun 2012 11:34:05 GMT
Reported by Philip Jägenstedt
Assigned to

Audio-ISSUE-48 (AudioBufferSimpl): AudioBuffer simplification [Web Audio API]

http://www.w3.org/2011/audio/track/issues/48

Raised by: Philip Jägenstedt
On product: Web Audio API

The duration attribute fills no purpose, as it can be calculated trivially from sampleRate and length.

In fact, the interface can be reduced to:

interface AudioBuffer {
    readonly attribute float sampleRate;
    Float32Array[] data;
}

The number of channels is data.length and the number of samples is data[0].length.

The name data was picked for consistency with the ImageData interface from the 2D canvas context.

@olivierthereaux
Copy link
Contributor Author

Original comment by Jer Noble on W3C Bugzilla. Thu, 18 Jul 2013 15:32:50 GMT

We should hold off on this issue until the current discussions concerning "data races" are resolved.

@olivierthereaux
Copy link
Contributor Author

Original comment by Olivier Thereaux on W3C Bugzilla. Thu, 18 Jul 2013 15:41:26 GMT

(In reply to comment #1)

We should hold off on this issue until the current discussions concerning
"data races" are resolved.

Agreed. I'm adding a bugzilla entry to track/record the discussions on the race condition issue, and will add appropriate dependency link.

@cwilso
Copy link
Contributor

cwilso commented Oct 22, 2014

Do we want to remove this parameter? (My vote is no, but it is not a strong position.)

@cwilso cwilso added the Needs Discussion The issue needs more discussion before it can be fixed. label Oct 30, 2014
@cwilso cwilso added this to the Web Audio Last Call 1 milestone Oct 30, 2014
@cwilso
Copy link
Contributor

cwilso commented Dec 4, 2014

Resolved by WG on 12/4/14: we should leave this as is, for compat reasons.

@cwilso cwilso closed this as completed Dec 4, 2014
@cwilso cwilso removed the Needs Discussion The issue needs more discussion before it can be fixed. label Dec 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants