Michael Niedermayer <michaelni <at> gmx.at> writes: > > - s->palette[i] = 0xFF000000 | (AV_RB24(p) << 2); > > + s->palette[i] = 0xFF000000 | (AV_RB24(p) << 2) | (AV_RB24(p) >> 6) & 0x30303; > > this looks a bit odd, naively id say it should be >>4 Definitely, thank you. > the others LGTM Patches pushed, Carl Eugen