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

[Baptiste Coudurier]

On 12/12/09 11:44 AM, michael wrote:
> Author: michael
> Date: Sat Dec 12 20:44:01 2009
> New Revision: 20814
> 
> Log:
> Be less picky on invalid invalid frame_pred_frame_dct values.
> Fixed issue1615.
> 
> Modified:
>    trunk/libavcodec/mpeg12.c
> 
> Modified: trunk/libavcodec/mpeg12.c
> ==============================================================================
> --- trunk/libavcodec/mpeg12.c	Sat Dec 12 20:15:56 2009	(r20813)
> +++ trunk/libavcodec/mpeg12.c	Sat Dec 12 20:44:01 2009	(r20814)
> @@ -1557,7 +1557,7 @@ static void mpeg_decode_picture_coding_e
>          s->picture_structure= PICT_FRAME;
>      }
>  
> -    if(s->progressive_frame && !s->frame_pred_frame_dct){
> +    if(s->progressive_sequence && !s->frame_pred_frame_dct){
>          av_log(s->avctx, AV_LOG_ERROR, "invalid frame_pred_frame_dct\n");
>          s->frame_pred_frame_dct= 1;
>      }

This commit and 20654 are causing issues, see #2405.

Since frame_pred_frame_dct is affecting the bitstream syntax, overriding
it seems unstable to me :/

-- 
Baptiste COUDURIER
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org