Hi,

On Sun, Mar 27, 2011 at 3:41 PM, Reinhard Tartler <[email protected]> wrote:
> From: Anatoly Nenashev <[email protected]>
>
> Problem description, preliminary review discussion at
> http://thread.gmane.org/gmane.comp.video.ffmpeg.devel/127731
> ---
>  libavcodec/mpegvideo.c |    9 +++++++--
>  1 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c
> index 1c5ff27..1fb2522 100644
> --- a/libavcodec/mpegvideo.c
> +++ b/libavcodec/mpegvideo.c
> @@ -991,8 +991,13 @@ int MPV_frame_start(MpegEncContext *s, AVCodecContext 
> *avctx)
>         s->pict_type, s->dropable);*/
>
>     if(s->codec_id != CODEC_ID_H264){
> -        if((s->last_picture_ptr==NULL || s->last_picture_ptr->data[0]==NULL) 
> && s->pict_type!=FF_I_TYPE){
> -            av_log(avctx, AV_LOG_ERROR, "warning: first frame is no 
> keyframe\n");
> +        if((s->last_picture_ptr==NULL || s->last_picture_ptr->data[0]==NULL) 
> &&
> +           (s->pict_type!=FF_I_TYPE || s->picture_structure != PICT_FRAME)){
> +            if (s->pict_type != FF_I_TYPE)
> +                av_log(avctx, AV_LOG_ERROR, "warning: first frame is no 
> keyframe\n");
> +            else if (s->picture_structure != PICT_FRAME)
> +                av_log(avctx, AV_LOG_INFO, "allocate dummy last picture for 
> field based first keyframe\n");

I think this is OK.

Ronald
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to