[FFmpeg-devel] [PATCH]Do not store more than 30 bit for r10k

[Reimar Döffinger]

On Sat, Dec 27, 2014 at 05:02:39PM +0000, Carl Eugen Hoyos wrote:
> Reimar Döffinger <Reimar.Doeffinger <at> gmx.de> writes:
> 
> > -            uint16_t b = *src++ >> 4;
> > +            uint16_t b = *src++ >> 6;
> >              if (avctx->codec_id == AV_CODEC_ID_R210)
> > -                pixel = (r << 20) | (g << 10) | b >> 2;
> > +                pixel = (r << 20) | (g << 10) | b;
> >              else
> > -                pixel = (r << 22) | (g << 12) | b;
> > +                pixel = (r << 22) | (g << 12) | (b << 2);
> 
> Please commit.

Hm, ok.
Though as noted in the commit message I am not sure that for better
compatibility the extra bits shouldn't be 1 instead...