[FFmpeg-devel] [PATCH 8/8] Replace "depth" with "nb_bits" in the header of avcodec_pix_fmt_string().

Michael Niedermayer michaelni
Tue Nov 16 23:57:44 CET 2010


On Mon, Nov 15, 2010 at 03:04:28PM +0100, Stefano Sabatini wrote:
> On date Monday 2010-11-15 14:07:00 +0100, Michael Niedermayer encoded:
> > On Mon, Nov 15, 2010 at 12:15:15PM +0100, Stefano Sabatini wrote:
> > 
> > why?
> 
> bits per pixel != PixFmtInfo.depth
> 
> less confusing

well, iam not sure if "nb" is less confusing and people will know it means
number, especially non native english speakers
Also nb_bits is quite ambigous
number (used) bits of all samples (like 15 for rgb15)
or number bits each pixel occupies (like 16 for rgb15)
sadly i dont really have a good suggestion either ...

[...]
-- 
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: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101116/0be65632/attachment.pgp>



More information about the ffmpeg-devel mailing list