On Thu, Mar 18, 2010 at 12:11:14AM +0100, Aurelien Jacobs wrote: > + uint8_t *extradata = sh->bih + 1; I think that misses a cast to avoid a warning (i.e. it should be uint8_t *extradata = (uint8_t *)(sh->bih + 1); Otherwise great!