[FFmpeg-devel] [PATCH] ffprobe: don't use AVStream.codec to set decoder framerate

[Josh de Kock]

On 27/09/2016 02:57, James Almer wrote:
> Also don't set time_base. It's deprecated for decoding and avcodec_open2()
> will overwrite it
>
> Signed-off-by: James Almer <jamrial at gmail.com>
> ---
>  ffprobe.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/ffprobe.c b/ffprobe.c
> index b59f11e..bb3979c 100644
> --- a/ffprobe.c
> +++ b/ffprobe.c
> @@ -2612,10 +2612,7 @@ static int open_input_file(InputFile *ifile, const char *filename)
>                  exit(1);
>
>              av_codec_set_pkt_timebase(ist->dec_ctx, stream->time_base);
> -#if FF_API_LAVF_AVCTX
> -            ist->dec_ctx->time_base = stream->codec->time_base;
> -            ist->dec_ctx->framerate = stream->codec->framerate;
> -#endif
> +            ist->dec_ctx->framerate = stream->avg_frame_rate;
>
>              if (avcodec_open2(ist->dec_ctx, codec, &opts) < 0) {
>                  av_log(NULL, AV_LOG_WARNING, "Could not open codec for input stream %d\n",
>

LGTM

--
Josh