[Speex-dev] Please test upcoming release
jim.crichton at comcast.net
Wed Aug 23 07:04:26 PDT 2006
The trouble is that one cannot identify the type difference without reading
the nb_celp.c source file, and this parameter is treated differently than
other, similar parameters. It would seem like the most intuitive approach
would be to match the interface to the structure.
This is a trivial point, of course. I have attached a patch to the two TI
interface files, which adds the configuration for the filter, sized as
spx_int32_t. I also included the modified files (I know that you have had
some trouble with patches that I have sent before).
----- Original Message -----
From: "Jean-Marc Valin" <jean-marc.valin at usherbrooke.ca>
To: "Jim Crichton" <jim.crichton at comcast.net>
Cc: <speex-dev at xiph.org>
Sent: Tuesday, August 22, 2006 8:12 PM
Subject: Re: [Speex-dev] Please test upcoming release
> Hi Jim,
> Actually, I don't see anything wrong with the internal structure having
> a different type than the interface, as long both types are big enough
> to hold the possible values (in this case 0 and 1). Though, as you
> pointed out, testenc needs to be fixed to use spx_int32_t instead of
> int. I'll change that.
> Jim Crichton a écrit :
>> st->highpass_enabled is typed inconsistently. It is "int" in the
>> structure, but "spx_int32_t" in the SET, GET cases in
>> nb(sb)_encoder(decoder)_ctl. The parameter is "int" in testenc.c, and
>> in the TI version ti\testenc-TI-C5x.c. This breaks for 16 bit platforms,
>> where "int" is 16 bits.
>> The types in the state structure and the _ctl routines really should
>> match. For now, I have changed testenc-TI-C5x.c to pass a 32-bit
>> parameter, which works with no change elsewhere. I can send a patch for
>> Another inconsistency is that GET_COMPLEXITY returns a spx_int32_t, but
>> the field st->complexity is an int.
>> - Jim
>> Speex-dev mailing list
>> Speex-dev at xiph.org
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 7074 bytes
Desc: not available
Url : http://lists.xiph.org/pipermail/speex-dev/attachments/20060823/0f01afe5/ti_11791_patch.bin
More information about the Speex-dev