Michael pointed out most, so just some nits: On Mon, Dec 12, 2011 at 11:36:52PM -0500, Derek Buitenhuis wrote: > + avctx->bits_per_raw_sample = 10; > + > + if (avctx->width & 1) { Trailing whitespace (line in the middle) can not be committed, try running patcheck. > + val = u[j] << 2;; One ; is enough. Reimar