[FFmpeg-devel] [PATCH 1/1] Make avpicture_fill() directly call av_fill_image_linesizes() and av_fill_image_pointers() rather than their wrappers ff_fill_linesize() and ff_fill_pointer().

Stefano Sabatini stefano.sabatini-lala
Fri Jul 30 10:44:38 CEST 2010


On date Friday 2010-07-30 01:53:11 +0200, Stefano Sabatini encoded:
> Improve performance.
> ---
>  libavcodec/imgconvert.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/libavcodec/imgconvert.c b/libavcodec/imgconvert.c
> index 70e6c0b..d0fc1ce 100644
> --- a/libavcodec/imgconvert.c
> +++ b/libavcodec/imgconvert.c
> @@ -509,10 +509,10 @@ int avpicture_fill(AVPicture *picture, uint8_t *ptr,
>      if(avcodec_check_dimensions(NULL, width, height))
>          return -1;
>  
> -    if (ff_fill_linesize(picture, pix_fmt, width))
> +    if (av_fill_image_linesizes(picture->linesize, pix_fmt, width))
>          return -1;
>  
> -    return ff_fill_pointer(picture, ptr, pix_fmt, height);
> +    return av_fill_image_pointers(picture->data, pix_fmt, height, ptr, picture->linesize);
>  }
>  
>  int avpicture_layout(const AVPicture* src, enum PixelFormat pix_fmt, int width, int height,

Trivial and applied.

Now I'm considering to kill those functions at the next major bump, regards.
-- 
FFmpeg = Fast & Frightening Magical Purposeless Enlightened Gadget



More information about the ffmpeg-devel mailing list