opus - Oct 2009 - API Change

Hi everyone,

I've just changed the API for CELT, but at least there's a good reason
for that. It's now possible to use the same mode data for both mono and
stereo. So here's the change:

- The celt_mode_create() function has a "channels" parameter

- The celt_encode_create() and celt_decoder_create() functions now have
an additional "channels" parameter.

- I also added an error parameter celt_encode_create() and
celt_decoder_create() to return any error that occurs.

Sorry about the inconvenience. You'll see, it's worth it. Please report
any breakage.

Cheers,

	Jean-Marc

Jean-Marc Valin a ?crit :
> - The celt_mode_create() function has a "channels" parameter
Oops, I meant to say that "The celt_mode_create() function *no longer*
has a channels parameter".

	Jean-Marc

Hi all,

I have to say that I'm completely in favor of this change. In my app I 
use both mono and stereo streams, and it is rather a bother to have to 
create my modes in pairs.

Cheers,

John

celt-dev-request at xiph.org wrote:
>
> Message: 3
> Date: Thu, 15 Oct 2009 22:00:06 -0400
> From: Jean-Marc Valin <jean-marc.valin at usherbrooke.ca>
> Subject: [CELT-dev] API Change
> To: celt-dev <celt-dev at xiph.org>
> Message-ID: <4AD7D3A6.8040802 at usherbrooke.ca>
> Content-Type: text/plain; charset=ISO-8859-1
>
> Hi everyone,
>
> I've just changed the API for CELT, but at least there's a good
reason
> for that. It's now possible to use the same mode data for both mono and
> stereo. So here's the change:
>
> - The celt_mode_create() function has a "channels" parameter
>
> - The celt_encode_create() and celt_decoder_create() functions now have
> an additional "channels" parameter.
>
> - I also added an error parameter celt_encode_create() and
> celt_decoder_create() to return any error that occurs.
>
> Sorry about the inconvenience. You'll see, it's worth it. Please
report
> any breakage.
>
> Cheers,
>
> 	Jean-Marc
>
>
>   
>

Jean-Marc Valin wrote:
> Jean-Marc Valin a ?crit :
>> - The celt_mode_create() function has a "channels" parameter
> 
> Oops, I meant to say that "The celt_mode_create() function *no longer*
> has a channels parameter".
> 
/** Creates a new mode struct. This will be passed to an encoder or 
    decoder. The mode MUST NOT BE DESTROYED until the encoders and 
    decoders that use it are destroyed as well.
 @param Fs Sampling rate (32000 to 96000 Hz)
 @param channels Number of channels
 @param frame_size Number of samples (per channel) to encode in each 
                   packet (even values; 64 - 512)
 @param error Returned error code (if NULL, no error will be returned)
 @return A newly created mode
*/
EXPORT CELTMode *celt_mode_create(celt_int32 Fs, int frame_size, int *error);



---> the "channels" parameter is still in the doxygen comment ...
should
     be removed I guess...


/alfred