[FFmpeg-cvslog] r9890 - trunk/libavcodec/mpeg12.c

[Aurelien Jacobs]

On Sat,  4 Aug 2007 13:01:11 +0200 (CEST)
michael <subversion at mplayerhq.hu> wrote:

> Author: michael
> Date: Sat Aug  4 13:01:11 2007
> New Revision: 9890
> 
> Log:
> make mpeg1/2 decoder work ith the libmpeg2 style bitstream reader
> 
> 
> Modified:
>    trunk/libavcodec/mpeg12.c
> 
> Modified: trunk/libavcodec/mpeg12.c
> ==============================================================================
> --- trunk/libavcodec/mpeg12.c	(original)
> +++ trunk/libavcodec/mpeg12.c	Sat Aug  4 13:01:11 2007
> @@ -737,7 +737,9 @@ static inline int mpeg1_decode_block_int
>              if(((int32_t)GET_CACHE(re, &s->gb)) <= (int32_t)0xBFFFFFFF)
>                  goto end;
>          }
> -
> +#if MIN_CACHE_BITS < 19
> +        UPDATE_CACHE(re, &s->gb);
> +#endif
> 
> [...]

To me, if() would looks much better than #if.
What about the attached patch ?
(it would obviously be followed be an indentation patch)

Aurel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mpeg12_if.diff
Type: text/x-diff
Size: 3191 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-cvslog/attachments/20070809/33b31ef9/attachment.diff>