[FFmpeg-devel] [PATCH 1/2] avfilter/video: pick sar from link

Paul B Mahol onemda at gmail.com
Fri Dec 15 21:10:58 EET 2017


On 12/10/17, Paul B Mahol <onemda at gmail.com> wrote:
> It should not be needed for each filter that sets sample aspect ratio
> to set it explicitly also for each and every frame, instead that is
> automatically done in get_buffer call.
>
> Signed-off-by: Paul B Mahol <onemda at gmail.com>
> ---
>  libavfilter/video.c | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/libavfilter/video.c b/libavfilter/video.c
> index 6f9020b9fe..7a8e587798 100644
> --- a/libavfilter/video.c
> +++ b/libavfilter/video.c
> @@ -43,6 +43,7 @@ AVFrame *ff_null_get_video_buffer(AVFilterLink *link, int
> w, int h)
>
>  AVFrame *ff_default_get_video_buffer(AVFilterLink *link, int w, int h)
>  {
> +    AVFrame *frame = NULL;
>      int pool_width = 0;
>      int pool_height = 0;
>      int pool_align = 0;
> @@ -86,7 +87,13 @@ AVFrame *ff_default_get_video_buffer(AVFilterLink *link,
> int w, int h)
>          }
>      }
>
> -    return ff_frame_pool_get(link->frame_pool);
> +    frame = ff_frame_pool_get(link->frame_pool);
> +    if (!frame)
> +        return NULL;
> +
> +    frame->sample_aspect_ratio = link->sample_aspect_ratio;
> +
> +    return frame;
>  }
>
>  AVFrame *ff_get_video_buffer(AVFilterLink *link, int w, int h)
> --
> 2.11.0
>
>

Will apply this 2 sample aspect ratio patches really soon.


More information about the ffmpeg-devel mailing list