From 7b9fc769e40a7709fa59a54e2a810f76364eee4b Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Sun, 14 Oct 2012 09:18:36 +0000 Subject: [PATCH] svq1dec: use more common way to check if result is not zero Signed-off-by: Paul B Mahol --- libavcodec/svq1dec.c | 28 ++++++++++------------------ 1 file changed, 10 insertions(+), 18 deletions(-) diff --git a/libavcodec/svq1dec.c b/libavcodec/svq1dec.c index 8e4a4827bd..8341a9a8d4 100644 --- a/libavcodec/svq1dec.c +++ b/libavcodec/svq1dec.c @@ -333,8 +333,7 @@ static int svq1_motion_inter_block(MpegEncContext *s, GetBitContext *bitbuf, } result = svq1_decode_motion_vector(bitbuf, &mv, pmv); - - if (result != 0) + if (result) return result; motion[0].x = @@ -378,8 +377,7 @@ static int svq1_motion_inter_4v_block(MpegEncContext *s, GetBitContext *bitbuf, } result = svq1_decode_motion_vector(bitbuf, &mv, pmv); - - if (result != 0) + if (result) return result; /* predict and decode motion vector (1) */ @@ -391,8 +389,7 @@ static int svq1_motion_inter_4v_block(MpegEncContext *s, GetBitContext *bitbuf, pmv[1] = &motion[(x / 8) + 3]; } result = svq1_decode_motion_vector(bitbuf, &motion[0], pmv); - - if (result != 0) + if (result) return result; /* predict and decode motion vector (2) */ @@ -400,8 +397,7 @@ static int svq1_motion_inter_4v_block(MpegEncContext *s, GetBitContext *bitbuf, pmv[2] = &motion[(x / 8) + 1]; result = svq1_decode_motion_vector(bitbuf, &motion[(x / 8) + 2], pmv); - - if (result != 0) + if (result) return result; /* predict and decode motion vector (3) */ @@ -409,8 +405,7 @@ static int svq1_motion_inter_4v_block(MpegEncContext *s, GetBitContext *bitbuf, pmv[3] = &motion[(x / 8) + 3]; result = svq1_decode_motion_vector(bitbuf, pmv[3], pmv); - - if (result != 0) + if (result) return result; /* form predictions */ @@ -467,8 +462,7 @@ static int svq1_decode_delta_block(MpegEncContext *s, GetBitContext *bitbuf, case SVQ1_BLOCK_INTER: result = svq1_motion_inter_block(s, bitbuf, current, previous, pitch, motion, x, y); - - if (result != 0) { + if (result) { av_dlog(s->avctx, "Error in svq1_motion_inter_block %i\n", result); break; } @@ -478,8 +472,7 @@ static int svq1_decode_delta_block(MpegEncContext *s, GetBitContext *bitbuf, case SVQ1_BLOCK_INTER_4V: result = svq1_motion_inter_4v_block(s, bitbuf, current, previous, pitch, motion, x, y); - - if (result != 0) { + if (result) { av_dlog(s->avctx, "Error in svq1_motion_inter_4v_block %i\n", result); break; @@ -617,8 +610,7 @@ static int svq1_decode_frame(AVCodecContext *avctx, void *data, } result = svq1_decode_frame_header(&s->gb, s); - - if (result != 0) { + if (result) { av_dlog(s->avctx, "Error in svq1_decode_frame_header %i\n", result); return result; } @@ -672,7 +664,7 @@ static int svq1_decode_frame(AVCodecContext *avctx, void *data, for (x = 0; x < width; x += 16) { result = svq1_decode_block_intra(&s->gb, ¤t[x], linesize); - if (result != 0) { + if (result) { av_log(s->avctx, AV_LOG_INFO, "Error in svq1_decode_block %i (keyframe)\n", result); @@ -690,7 +682,7 @@ static int svq1_decode_frame(AVCodecContext *avctx, void *data, result = svq1_decode_delta_block(s, &s->gb, ¤t[x], previous, linesize, pmv, x, y); - if (result != 0) { + if (result) { av_dlog(s->avctx, "Error in svq1_decode_delta_block %i\n", result);