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

Ivo ivop
Sun Apr 29 11:40:21 CEST 2007


On Sunday 29 April 2007 02:06, Michael Niedermayer wrote:
> 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

I spelt it out in all lower case :)

Patch applied.

--Ivo




More information about the ffmpeg-devel mailing list