[FFmpeg-devel] [PATCH 1/1] Move fill_image_linesize() and fill_image_data_ptr() from libavcodec and make them public in libavcore/imgutils.h.

Michael Niedermayer michaelni
Wed Jul 28 23:54:52 CEST 2010


On Wed, Jul 28, 2010 at 07:24:19PM +0200, Stefano Sabatini wrote:
> On date Wednesday 2010-07-28 16:27:56 +0200, Michael Niedermayer encoded:
> > On Wed, Jul 28, 2010 at 11:27:11AM +0200, Stefano Sabatini wrote:
> > > ---
> > >  libavcodec/imgconvert.c |  144 +----------------------------------------
> > >  libavcore/Makefile      |    4 +-
> > >  libavcore/imgutils.c    |  164 +++++++++++++++++++++++++++++++++++++++++++++++
> > >  libavcore/imgutils.h    |   49 ++++++++++++++
> > >  4 files changed, 219 insertions(+), 142 deletions(-)
> > >  create mode 100644 libavcore/imgutils.c
> > >  create mode 100644 libavcore/imgutils.h
> > 
> > 
> > ok
> 
> Now it's a good time to bikeshed on the names, in my order of
> preference:
> 
> av_fill_image_pointer:      simple, consistent with av_fill_image_linesize

semantically wrong due to singular


> av_fill_image_pointers:     simple, av_fill_image_linesize should be changed to linesizes


> av_fill_image_data:         simple, possibly confusing as it's not clear what "data" is

semantically wrong due to no "pointer"


> av_fill_image_data_ptr:     redundant, sligthly ugly
> av_fill_image_data_pointer: redundant, long

semantically wrong due to singular


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I hate to see young programmers poisoned by the kind of thinking
Ulrich Drepper puts forward since it is simply too narrow -- Roman Shaposhnik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 190 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100728/0f779f3d/attachment.pgp>



More information about the ffmpeg-devel mailing list