From 83344066d326e6bad20feb66825ace12708eb084 Mon Sep 17 00:00:00 2001 From: Diego Biurrun Date: Mon, 16 Feb 2009 11:49:08 +0000 Subject: [PATCH] Get rid of some '#if CONFIG_MPEG_XVMC_DECODER' preprocessor checks around if statements by merging the CONFIG_MPEG_XVMC_DECODER check into the if condition. Originally committed as revision 17369 to svn://svn.ffmpeg.org/ffmpeg/trunk --- libavcodec/error_resilience.c | 12 ++++-------- libavcodec/mpegvideo.c | 13 ++++--------- 2 files changed, 8 insertions(+), 17 deletions(-) diff --git a/libavcodec/error_resilience.c b/libavcodec/error_resilience.c index d97a96b..a196326 100644 --- a/libavcodec/error_resilience.c +++ b/libavcodec/error_resilience.c @@ -563,10 +563,9 @@ static int is_intra_more_likely(MpegEncContext *s){ if(undamaged_count < 5) return 0; //almost all MBs damaged -> use temporal prediction -#if CONFIG_MPEG_XVMC_DECODER //prevent dsp.sad() check, that requires access to the image - if(s->avctx->xvmc_acceleration && s->pict_type==FF_I_TYPE) return 1; -#endif + if(CONFIG_MPEG_XVMC_DECODER && s->avctx->xvmc_acceleration && s->pict_type == FF_I_TYPE) + return 1; skip_amount= FFMAX(undamaged_count/50, 1); //check only upto 50 MBs is_intra_likely=0; @@ -935,10 +934,9 @@ void ff_er_frame_end(MpegEncContext *s){ }else guess_mv(s); -#if CONFIG_MPEG_XVMC_DECODER /* the filters below are not XvMC compatible, skip them */ - if(s->avctx->xvmc_acceleration) goto ec_clean; -#endif + if(CONFIG_MPEG_XVMC_DECODER && s->avctx->xvmc_acceleration) + goto ec_clean; /* fill DC for inter blocks */ for(mb_y=0; mb_ymb_height; mb_y++){ for(mb_x=0; mb_xmb_width; mb_x++){ @@ -1024,9 +1022,7 @@ void ff_er_frame_end(MpegEncContext *s){ v_block_filter(s, s->current_picture.data[2], s->mb_width , s->mb_height , s->uvlinesize, 0); } -#if CONFIG_MPEG_XVMC_DECODER ec_clean: -#endif /* clean a few tables */ for(i=0; imb_num; i++){ const int mb_xy= s->mb_index2xy[i]; diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c index da19243..41ad43c 100644 --- a/libavcodec/mpegvideo.c +++ b/libavcodec/mpegvideo.c @@ -936,10 +936,9 @@ alloc: update_noise_reduction(s); } -#if CONFIG_MPEG_XVMC_DECODER - if(s->avctx->xvmc_acceleration) + if(CONFIG_MPEG_XVMC_DECODER && s->avctx->xvmc_acceleration) return ff_xvmc_field_start(s, avctx); -#endif + return 0; } @@ -948,12 +947,10 @@ void MPV_frame_end(MpegEncContext *s) { int i; /* draw edge for correct motion prediction if outside */ -#if CONFIG_MPEG_XVMC_DECODER //just to make sure that all data is rendered. - if(s->avctx->xvmc_acceleration){ + if(CONFIG_MPEG_XVMC_DECODER && s->avctx->xvmc_acceleration){ ff_xvmc_field_end(s); }else -#endif if(!(s->avctx->codec->capabilities&CODEC_CAP_HWACCEL_VDPAU) && s->unrestricted_mv && s->current_picture.reference @@ -1733,12 +1730,10 @@ void MPV_decode_mb_internal(MpegEncContext *s, DCTELEM block[12][64], { int mb_x, mb_y; const int mb_xy = s->mb_y * s->mb_stride + s->mb_x; -#if CONFIG_MPEG_XVMC_DECODER - if(s->avctx->xvmc_acceleration){ + if(CONFIG_MPEG_XVMC_DECODER && s->avctx->xvmc_acceleration){ ff_xvmc_decode_mb(s);//xvmc uses pblocks return; } -#endif mb_x = s->mb_x; mb_y = s->mb_y; -- 2.7.4