# HG changeset patch # User michael # Date 1185997048 0 # Node ID f1db308c277b7182d6bfde3f00e35cf46b9fae76 # Parent 8ea581ae9fa2cee38319e64a531e3a0e1b3d1021 reorder if() so that the condition can be simplified saves another 4 cpu cycles diff -r 8ea581ae9fa2 -r f1db308c277b mpeg12.c --- a/mpeg12.c Wed Aug 01 19:26:24 2007 +0000 +++ b/mpeg12.c Wed Aug 01 19:37:28 2007 +0000 @@ -502,14 +502,14 @@ } cbp = get_vlc2(&s->gb, mb_pat_vlc.table, MB_PAT_VLC_BITS, 1); - if (cbp < 0 || ((cbp == 0) && (s->chroma_format < 2)) ){ - av_log(s->avctx, AV_LOG_ERROR, "invalid cbp at %d %d\n", s->mb_x, s->mb_y); - return -1; - } if(mb_block_count > 6){ cbp<<= mb_block_count-6; cbp |= get_bits(&s->gb, mb_block_count-6); } + if (cbp <= 0){ + av_log(s->avctx, AV_LOG_ERROR, "invalid cbp at %d %d\n", s->mb_x, s->mb_y); + return -1; + } #ifdef HAVE_XVMC //on 1 we memcpy blocks in xvmcvideo