[FFmpeg-devel] [PATCH] lavc/vc1dec: fix a few "may be used uninitialized" warnings.
Clément Bœsch
ubitux at gmail.com
Mon May 13 17:09:58 CEST 2013
---
libavcodec/vc1dec.c | 15 ++++++++-------
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c
index bce0b45..068cfb9 100644
--- a/libavcodec/vc1dec.c
+++ b/libavcodec/vc1dec.c
@@ -628,7 +628,7 @@ static void vc1_mc_4mv_luma(VC1Context *v, int n, int dir, int avg)
ty = (chosen_mv[f][0][1] + chosen_mv[f][1][1]) / 2;
break;
default:
- av_assert2(0);
+ av_assert0(0);
}
s->current_picture.motion_val[1][s->block_index[0] + v->blocks_off][0] = tx;
s->current_picture.motion_val[1][s->block_index[0] + v->blocks_off][1] = ty;
@@ -1671,7 +1671,7 @@ static inline void vc1_pred_mv_intfr(VC1Context *v, int n, int dmv_x, int dmv_y,
MpegEncContext *s = &v->s;
int xy, wrap, off = 0;
int A[2], B[2], C[2];
- int px, py;
+ int px = 0, py = 0;
int a_valid = 0, b_valid = 0, c_valid = 0;
int field_a, field_b, field_c; // 0: same, 1: opposit
int total_valid, num_samefield, num_oppfield;
@@ -1809,8 +1809,7 @@ static inline void vc1_pred_mv_intfr(VC1Context *v, int n, int dmv_x, int dmv_y,
else if (b_valid) { px = B[0]; py = B[1]; }
else if (c_valid) { px = C[0]; py = C[1]; }
else av_assert2(0);
- } else
- px = py = 0;
+ }
}
} else {
if (a_valid)
@@ -1863,13 +1862,12 @@ static inline void 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 (total_valid == 1) {
px = (a_valid) ? A[0] : ((b_valid) ? B[0] : C[0]);
py = (a_valid) ? A[1] : ((b_valid) ? B[1] : C[1]);
- } else
- px = py = 0;
+ }
}
/* store MV using signed modulus of MV range defined in 4.11 */
@@ -4320,6 +4318,9 @@ static void vc1_decode_b_mb_intfi(VC1Context *v)
case 2:
bmvtype = BMV_TYPE_INTERPOLATED;
interpmvp = get_bits1(gb);
+ break;
+ default:
+ av_assert0(0);
}
}
v->bmvtype = bmvtype;
--
1.8.2.3
More information about the ffmpeg-devel
mailing list