On Wed, 15 Apr 2015 15:04:32 +0100
Vittorio Giovara <[email protected]> wrote:
> Fix a rather lenghty warning from clang.
> ---
> libavcodec/vc1_pred.c | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/vc1_pred.c b/libavcodec/vc1_pred.c
> index 07e9d91..5a57f1b 100644
> --- 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 {
> if (field_a && a_valid) {
> px = A[0];
> @@ -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) {
> px = (a_valid) ? A[0] : ((b_valid) ? B[0] : C[0]);
Why not just init them on declaration, instead of in every single code
path?
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
