On Wed, Apr 15, 2015 at 03:04:32PM +0100, Vittorio Giovara wrote:
> Fix a rather lenghty warning from clang.
leng_th_y
> --- a/libavcodec/vc1_pred.c
> +++ b/libavcodec/vc1_pred.c
> @@ -648,7 +648,8 @@ void ff_vc1_pred_mv_intfr(VC1Context *v, int n, int
> dmv_x, int dmv_y,
> } else if (c_valid) {
> px = C[0];
> py = C[1];
> - }
> + } else
> + px = py = 0;
> } else {
> @@ -656,7 +657,8 @@ void ff_vc1_pred_mv_intfr(VC1Context *v, int n, int
> dmv_x, int dmv_y,
> } else if (field_b && b_valid) {
> px = B[0];
> py = B[1];
> - }
> + } else
> + px = py = 0;
> }
> } else if (total_valid == 1) {
I think having the assignments on two lines is more readable.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
