[FFmpeg-devel] [PATCH] IFF: Make lavf/iff.c stop using ff_cmap_read_palette
Sun May 9 20:38:12 CEST 2010
Martin Storsj? a ?crit :
> On Sun, 9 May 2010, Sebastian Vater wrote:
>> Martin recommeded to do these as a separate patch.
>> It removes call to ff_cmap_read_palette in lavf/iff.c for preparation of
>> the HAM patch.
> Does an old lavc/iff.c handle a new demuxer behaving in this way, or why
> is the change to lavc/iff.c included here? In that case, this can lead to
> problems when mixing lavf/lavc versions.
Removing ff_cmap_read_palette from lavf/iff.c requires that the decoder
handles IFF-PBM uncompressed files by itself, since it can't be passed
to CODEC_ID_RAWVIDEO anymore.
But that isn't a problem, because old lavf/iff.c doesn't pass the data
at all to lavc/iff.c in that case.
However a new lavf/iff.c could pass to old lavc/iff.c in that case it
won't decode uncompressed IFF-PBM files anymore. It would threat them as
IFF-ILBM uncompressed and therefore call the decodeplane8 version which
will result in corrupted image data.
The only way I see to fix this is to increment the lavc version number
or to add the new IFF_EXTRA_CONTEXT_SIZE stuff, also. What do you suggest?
:-) Basty/CDGS (-:
More information about the ffmpeg-devel