[Ffmpeg-devel] [PATCH] V.Flash PTX decoder

Michael Niedermayer michaelni
Sun Apr 29 02:06:21 CEST 2007


Hi

On Sat, Apr 28, 2007 at 11:50:04PM +0200, Ivo wrote:
[...]
> > > +
> > > +    ptr    = p->data[0];
> > > +    stride = p->linesize[0];
> > >
> > > +    Bpp    = 2;
> >
> > hmm, why B instead of b?
> 
> I chose Bpp (bytes per pixel) to distinguish it from bpp (bits per pixel). I 
> could change it to bytes_per_pixel though if that eliminates all confusion. 
> Or add a doxygen comment?

i would prefer a name begining with a lower case letter

except that patch ok

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

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070429/1a6bcac0/attachment.pgp>



More information about the ffmpeg-devel mailing list