On Fri, Apr 03, 2015 at 07:14:16PM +0200, Andreas Cadhalpun wrote:
> Signed-off-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com>
> ---
>  libavcodec/h263dec.c    | 2 +-
>  libavcodec/mpeg4video.h | 3 +++
>  2 files changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/libavcodec/h263dec.c b/libavcodec/h263dec.c
> index a1981a3..0dde897 100644
> --- a/libavcodec/h263dec.c
> +++ b/libavcodec/h263dec.c
> @@ -459,7 +459,7 @@ retry:
>          }
>      }
>  
> -    if (s->bitstream_buffer_size && (s->divx_packed || buf_size < 20)) // 
> divx 5.01+/xvid frame reorder
> +    if (s->bitstream_buffer_size && (s->divx_packed || buf_size < 
> MAX_NVOP_SIZE)) // divx 5.01+/xvid frame reorder

its nitpicking but shouldnt it be <= if the constant is a maximum ?

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Let us carefully observe those good qualities wherein our enemies excel us
and endeavor to excel them, by avoiding what is faulty, and imitating what
is excellent in them. -- Plutarch

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to