[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