[FFmpeg-devel] [PATCH 1/2] lavc/libdavs2: remove incorrect frame settings

[Mark Thompson]

On 30/10/18 14:01, hwren wrote:
> Signed-off-by: hwren <hwrenx at 126.com>
> ---
>  libavcodec/libdavs2.c | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/libavcodec/libdavs2.c b/libavcodec/libdavs2.c
> index cadf995..581d568 100644
> --- a/libavcodec/libdavs2.c
> +++ b/libavcodec/libdavs2.c
> @@ -96,13 +96,6 @@ static int davs2_dump_frames(AVCodecContext *avctx, davs2_picture_t *pic,
>                     pic->widths[plane] * bytes_per_sample);
>      }
>  
> -    frame->width     = cad->headerset.width;
> -    frame->height    = cad->headerset.height;
> -    frame->pts       = cad->out_frame.pts;
> -    frame->pict_type = pic->type;
> -    frame->format    = avctx->pix_fmt;
> -
> -    cad->decoded_frames++;
>      return 1;
>  }
>  
> 

This seems like it loses track of the timestamps?  Output pts is always AV_NOPTS_VALUE after this change.

(Patch 3/3 should also be squashed into this one, since this is removing the last use of the decoded_frames counter.)

Thanks,

- Mark