Re: AudioBuffer channel read/write APIs

I think in copyToChannel, start should indicate the offset in the channel
data living inside the AudioBuffer to start writing into, as opposed to an
offset into the source argument, since source can be a view into another
ArrayBuffer if needed.

With that correction, this seems fine to me.

Cheers,

--
Ehsan
<http://ehsanakhgari.org/>


On Fri, Sep 6, 2013 at 9:50 AM, Marcus Geelnard <mage@opera.com> wrote:

>  2013-09-06 04:34, Robert O'Callahan skrev:
>
>   This is what I proposed:
>
> partial interface AudioBuffer {
>   void copyChannelDataTo(long channelNumber, unsigned long start, unsigned
> long length, Float32Array destination);
> }
>
>  I agree with Jer that it makes sense to have a corresponding method to
> write into channel buffers.
>
>
> Me too.
>
>
>
>  We could call it copyChannelData>From but i think that might get a little
> confusing. We can also make 'length' optional. So here's my revised
> proposal:
>
> partial interface AudioBuffer {
>    void copyFromChannel(Float32Array destination, long channelNumber,
> unsigned long start, optional unsigned long length);
>   void copyToChannel(Float32Array source, long channelNumber, unsigned
> long start, optional unsigned long length);
> }
>
>  In both methods, 'length' defaults to the array's length. An exception
> is thrown if start+length is greater than the ArrayBuffer's length or if
> the array length is less than 'length'. I put the array first so that the
> parameters identifying the channel and the channel data range are together.
>
>
> This sounds perfectly OK to me.
>
> /Marcus
>
>
>
>  I'd like to get this implemented in Gecko ASAP, so please bikeshed in a
> timely manner :-).
>
>  Rob
>  --
> Jtehsauts  tshaei dS,o n" Wohfy  Mdaon  yhoaus  eanuttehrotraiitny  eovni
> le atrhtohu gthot sf oirng iyvoeu rs ihnesa.r"t sS?o  Whhei csha iids  teoa
> stiheer :p atroa lsyazye,d  'mYaonu,r  "sGients  uapr,e  tfaokreg iyvoeunr,
> 'm aotr  atnod  sgaoy ,h o'mGee.t"  uTph eann dt hwea lmka'n?  gBoutt  uIp
> waanndt  wyeonut  thoo mken.o w  *
> *
>
>
>
> --
> Marcus Geelnard
> Technical Lead, Mobile Infrastructure
> Opera Software
>
>