[FFmpeg-devel] [PATCH] lavu/imgutils: create misc functions for dealing with buffers

[Stefano Sabatini]

On date Thursday 2012-06-21 12:06:11 +0200, Stefano Sabatini encoded:
> On date Tuesday 2012-06-19 23:48:23 +0200, Stefano Sabatini encoded:
> > On date Tuesday 2012-06-19 19:23:16 +0200, Stefano Sabatini encoded:
> > [...]
> > > From 20ad834f934e0ea5765be9e0a609c67cb21f8964 Mon Sep 17 00:00:00 2001
> > > From: Stefano Sabatini <stefasab at gmail.com>
> > > Date: Tue, 19 Jun 2012 02:45:36 +0200
> > > Subject: [PATCH] lavu/imgutils: create misc functions for dealing with
> > >  buffers
> > [...]
> > 
> > Updated with a few minor fixes.
> 
> Upped with a few more minor cosmetics fixes. I plan to commit this in
> three days if I read no comments.
> -- 
> FFmpeg = Free and Fostering Minimal Pacific Elegant Gorilla

> From a8a6b891331d2994b96145ce543d390d612a07c7 Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefasab at gmail.com>
> Date: Tue, 19 Jun 2012 02:45:36 +0200
> Subject: [PATCH] lavu/imgutils: create misc functions for dealing with
>  buffers
> 
> Move the lavc/imgconvert functions and rename them as follows:
> avpicture_get_size -> av_image_get_buffer_size()
> avpicture_fill     -> av_image_fill_arrays()
> avpicture_layout   -> av_image_copy_to_buffer()
> 
> The new functions have an align parameter, which allows to define the
> linesize alignment assumed in the buffer (which is set or read).
> 
> The names of the functions are consistent with the lavu/samples API
> (av_samples_get_buffer_size(), av_samples_fill_arrays()).
> 
> FIXME: add doc/APIChanges entry and minor version bump before committing

Ping, I'll apply this in two days if I read no comments.

In particular I wonder if I should keep the docs for the avpicture_*
functions, but I'd prefer the current solution since I don't want to
duplicate documentation.

Note also that in case it was not apparent, my plan is to deprecate
the avpicture_* API.
-- 
FFmpeg = Friendly and Fiendish Mind-dumbing Philosofic Efficient Gorilla