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

Måns Rullgård mans
Sun Jun 13 17:56:09 CEST 2010


Michael Karcher <ffmpeg at mkarcher.dialup.fu-berlin.de> writes:

> 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.

So fix the broken line instead of repeating it.

-- 
M?ns Rullg?rd
mans at mansr.com



More information about the ffmpeg-devel mailing list