[FFmpeg-devel] [PATCH] [RFC] Second try at pixdesc.h:write_line()

Michael Niedermayer michaelni
Sat Apr 18 20:48:03 CEST 2009


On Sat, Apr 18, 2009 at 08:31:35PM +0200, Stefano Sabatini wrote:
> On date Saturday 2009-04-18 19:00:45 +0200, Michael Niedermayer encoded:
> [...]
> > here is my suggestion again:
> > 
> > > > for(){
> > > >     *buf |= *pixel++ << s;
> > > >     s-= C;

> > > >     buf -= s>>8;

that should be s>>3


> > > >     s&=7;
> > > > }
> 
> Maybe I'm dumb, I failed to get it the first time, and failing again.
> 

> What does C stand for? What is s supposed to be?

constant, that is a value that doesnt change during the loop


> Are you suggesting to
> use a pointer to int8_t, int32_t, what?

*buf is uint8_t


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- 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/20090418/2a6af491/attachment.pgp>



More information about the ffmpeg-devel mailing list