[FFmpeg-devel] [PATCH] Pictor/PC Paint PIC decoder

[Michael Niedermayer]

On Sun, Jun 06, 2010 at 10:22:48AM +0200, Reimar D?ffinger wrote:
> On Sun, Jun 06, 2010 at 02:16:24PM +1000, Peter Ross wrote:
> > +                   value <<= bits_per_plane;
> > +                   mask <<= bits_per_plane;
> 
> You could align those two as well.
> No other comments from me...

looks good to me too, assuming it was tested (also with a fuzzer)
and we need a fate test

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- 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/20100607/0e0dc3ea/attachment.pgp>