[FFmpeg-devel] [PATCH] Implement function to compute bits per pixel

Michael Niedermayer michaelni
Mon Apr 27 14:40:42 CEST 2009


On Mon, Apr 27, 2009 at 02:16:15PM +0200, Michael Niedermayer wrote:
> On Sun, Apr 26, 2009 at 03:15:24PM +0200, Stefano Sabatini wrote:

[...]
> [...]
> > Index: libavfilter-soc/ffmpeg/libavcodec/pixdesc.h
> > ===================================================================
> > --- libavfilter-soc.orig/ffmpeg/libavcodec/pixdesc.h	2009-04-26 11:30:27.000000000 +0200
> > +++ libavfilter-soc/ffmpeg/libavcodec/pixdesc.h	2009-04-26 12:15:37.000000000 +0200
> > @@ -193,4 +193,10 @@
> >      }
> >  }
> >  
> > +/**
> > + * Returns the number of bits per pixel used by the pixel format
> > + * described by pixdesc.
> > + */
> > +int av_pixdesc_get_bits_per_pixel(const AVPixFmtDescriptor *pixdesc);
> 
> av_get_bits_per_pixel
> or
> av_bits_per_pixel
> 
> seem better names

The correct name that fits best in the current naming is
av_get_bits_per_pixel(), please use this

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

Avoid a single point of failure, be that a person or equipment.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090427/6e27efec/attachment.pgp>



More information about the ffmpeg-devel mailing list