On Thu, 23 Mar 2017 12:29:14 +0100
Luca Barbato <[email protected]> wrote:

> Use the named format to avoid confusion.
> ---
>  avtools/avplay.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/avtools/avplay.c b/avtools/avplay.c
> index 18879e1..26279e8 100644
> --- a/avtools/avplay.c
> +++ b/avtools/avplay.c
> @@ -1441,7 +1441,8 @@ static int configure_video_filters(AVFilterGraph 
> *graph, PlayerState *is, const
>      snprintf(sws_flags_str, sizeof(sws_flags_str), "flags=%"PRId64, 
> sws_flags);
>      graph->scale_sws_opts = av_strdup(sws_flags_str);
>  
> -    snprintf(buffersrc_args, sizeof(buffersrc_args), "%d:%d:%d:%d:%d:%d:%d",
> +    snprintf(buffersrc_args, sizeof(buffersrc_args),
> +             "width=%d:height=%d:pix_fmt=%d:time_base=%d/%d:sar=%d/%d",
>               codec->width, codec->height, codec->pix_fmt,
>               is->video_st->time_base.num, is->video_st->time_base.den,
>               codec->sample_aspect_ratio.num, codec->sample_aspect_ratio.den);

Why not use av_buffersrc_parameters_set()?
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to