[FFmpeg-devel] [PATCH] G.729 Frame erasure support for fixed-codebook vector decoding

Michael Niedermayer michaelni
Sun Jun 28 23:59:17 CEST 2009


On Mon, Jun 29, 2009 at 04:32:51AM +0700, Vladimir Voroshilov wrote:
[...]
> >> >> +
> >> >> ? ? ?return 0;
> >> >> ?}
> >> >>
> >> >> @@ -336,6 +340,15 @@ static int decode_frame(AVCodecContext *avctx, void *data, int *data_size,
> >> >> ? ? ? ? ?/* Round pitch delay to nearest (used everywhere except ff_acelp_interpolate). */
> >> >> ? ? ? ? ?pitch_delay_int ?= (pitch_delay_3x + 1) / 3;
> >> >>
> >> >> + ? ? ? ?if (frame_erasure) {
> >> >> + ? ? ? ? ? ?ctx->rand_value = g729_prng(ctx->rand_value);
> >> >> + ? ? ? ? ? ?fc_indexes ? = ctx->rand_value & ((1 << format.fc_indexes_bits) - 1);
> >> >> +
> >> >> + ? ? ? ? ? ?ctx->rand_value = g729_prng(ctx->rand_value);
> >> >> + ? ? ? ? ? ?pulses_signs = ctx->rand_value & ((1 << format.fc_signs_bits) - 1);
> >> >> + ? ? ? ?}
> >>
> >> > a single call to the prng is probably enough also is the & really needed?
> >>
> >> As i said before, seed initialization as long as double call to
> >> g729_prng and "&" are defined
> >> in main body of G.729, part 4.4.4 "Generation of the replacement
> >> excitation". Is this
> >> enough reason ?to keep them ?
> >
> > no, its illogic for the handling of damaged frames to require specific bits
> > of a specific PRNG to be used
> > that said, are the & needed or not?
[...]
> P.S. "&" applied to pulse_signs can be safely removed, since extra
> bits are just ignored.

good

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

GMX, the mailprovider that uses RBL lists to reject mails from your friends
running their own mailserver at home. The mailprovider that obscures the
origin of mails (mis)identified as viruses. The mailprovider that improves
security my disallowing more secure forms of authentication.
-------------- 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/20090628/4533ff1b/attachment.pgp>



More information about the ffmpeg-devel mailing list