[FFmpeg-devel] [PATCH] RealAudio SIPR @16k decoder (3/4) acelp_vectors

[Michael Niedermayer]

On Sat, Jul 12, 2008 at 04:03:48PM +0700, Vladimir Voroshilov wrote:
> 2008/7/12 Diego Biurrun <diego at biurrun.de>:
> > On Sat, Jul 12, 2008 at 02:16:48PM +0700, Vladimir Voroshilov wrote:
> >>
> >> Here is first draft version of fixed-point RealAudio sipr decoder
> >> (only 16kHz mode)
> >>
> >> This patch contains required changes in acelp_vectors
> >
> > Sure? :)
> 
> Hm. I was sure :)
[...]
>  /**
> + * \brief Decode fixed-codebook vector (5.7.1 of AMR).
> + * \param fc_v [out] decoded fixed codebook vector (2.13)

> + * \param tab1 table used for pulses

table of pulse positions
also please fix this in the code already in svn.


> + * \param pulse_indexes fixed codebook indexes

Inconsistant relative to the existing code in this file


> + * \param pulse_count number of pulses decoded using first table

pulse_pair_count number of pulse pairs decoded.


[...]
> +void ff_acelp_two_pulses_per_track(
> +        int16_t* fc_v,
> +        const uint8_t *tab1,
> +        int* pulse_indexes,
> +        int pulse_count,
> +        int bits);
> +

and the actual function looks like: (i wonder if that even compiled)

> +void ff_acelp_two_pulses_per_track(
> +        int16_t* fc_v,
> +        const uint8_t *tab1,
> +        const uint8_t *tab2,
> +        int* pulse_indexes,
> +        int pulse_count,
> +        int bits)
> +{

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080718/db63d98d/attachment.pgp>