[FFmpeg-devel] [PATCH] IFF: Add the HAM stuff

Michael Karcher ffmpeg
Sun Jun 13 17:49:40 CEST 2010


Am Sonntag, den 13.06.2010, 16:48 +0200 schrieb Stefano Sabatini:
> >      uint8_t * planebuf;
> >      int init; // 1 if buffer and palette data already initialized, 0 otherwise
> > +    uint8_t * ham_buf;      ///< temporary buffer for planar to chunky conversation
> Nit: *ham_buf;

That might be the common ffmpeg style, but it would look weird to me (as
an outsider regarding the FFmpeg project) to introduce an inconsistency
with the declaration of planebuf two lines above.

Regards,
  Michael Karcher





More information about the ffmpeg-devel mailing list