[FFmpeg-devel] [PATCH 11/11] Make ff_cmap_read_palette static to libavcodec/iff.c. Delete iff.h.

Reinhard Tartler siretart
Wed Jan 26 16:50:21 CET 2011


On Tue, Jan 25, 2011 at 03:52:10 (CET), Ronald S. Bultje wrote:

> Hi,
>
> 2011/1/24 M?ns Rullg?rd <mans at mansr.com>:
>> "Ronald S. Bultje" <rsbultje at gmail.com> writes:
>>> On Mon, Jan 24, 2011 at 8:29 PM, Diego Elio Petten? <flameeyes at gmail.com> wrote:
>>>> The iff.h header only declared one function that is now static, the
>>>> libavformat/iff.c source file wasn't using it before. Drop the file
>>>> entirely.
>>>
>>> I believe we're keeping this one for backwards compatibility. You can
>>> put it under an #if VERSION...
>>
>> Compatibility with what? ?It's not a public interface.

ACK

> It was for Debian, so people can mix and match libavcodec/format
> versions. Can the Debian maintainer please remind me why we kept it?

What makes you think so? Since it is not a public header, no application
is supposed to be able to reference it.

There may be weird applications that reference the symbol
ff_cmap_read_palette, but since Diego has an outstanding research on
this issue, I trust him that this is not the case.

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4




More information about the ffmpeg-devel mailing list