[XviD-devel] [PATCH] Initialize p_fmv.x
Michael Militzer
michael at xvid.org
Fri Nov 12 11:37:49 CET 2010
Hi,
indeed, this appears to be a typo. Patch applied. Thanks.
Best regards,
Michael
Quoting Lasse Collin <lasse.collin at tukaani.org>:
> GCC gives the following warning:
>
> ../../src/decoder.c:1663:46: warning: operation on `dec->p_fmv.y'
> may be undefined
>
> It looks like a typo in the code (p_fmv.y is mentioned twice):
>
> dec->p_bmv.x = dec->p_bmv.y = dec->p_fmv.y = dec->p_fmv.y = 0;
> /* init pred vector to 0 */
> ^^^ ^^^ ^^^ ^^^
>
> --- xvid_20101108.orig/xvidcore/src/decoder.c 2010-08-10
> 18:00:34.000000000 +0300
> +++ xvid_20101108/xvidcore/src/decoder.c 2010-11-08 14:18:40.736668331 +0200
> @@ -1660,7 +1660,7 @@
> goto repeat;
> }
>
> - dec->p_bmv.x = dec->p_bmv.y = dec->p_fmv.y = dec->p_fmv.y = 0;
> /* init pred vector to 0 */
> + dec->p_bmv.x = dec->p_bmv.y = dec->p_fmv.x = dec->p_fmv.y = 0;
> /* init pred vector to 0 */
>
> /* packed_mode: special-N_VOP treament */
> if (dec->packed_mode && coding_type == N_VOP) {
>
> --
> Lasse Collin | IRC: Larhzu @ IRCnet & Freenode
> _______________________________________________
> Xvid-devel mailing list
> Xvid-devel at xvid.org
> http://list.xvid.org/mailman/listinfo/xvid-devel
>
>
More information about the Xvid-devel
mailing list