[FFmpeg-devel] [PATCH] R10k deocder

[James Darnley]

2010/6/27 Zhou Zongyi <zhouzy at os.pku.edu.cn>:
> Hi,
>
> New patch here.
>

Hello, I thought I'd provide you with a suggestion to get this
committed so you can cleanup your patch(es) for your bitstream filter.

I've attached a patch for the decoder which removes most of the
duplication.  Almost everything is identical to the r210 decoder.

One question, why do you use:
*dst++ = r + (r >> 10);
*dst++ = g + (g >> 10);
*dst++ = b + (b >> 10);

When the original does:
*dst++ = r | (r >> 10);
*dst++ = g | (g >> 10);
*dst++ = b | (b >> 10);
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-r10k-decoder.patch
Type: application/octet-stream
Size: 3470 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100815/ce084281/attachment.obj>