[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