[FFmpeg-devel] Realmedia patch

[Måns Rullgård]

Michael Niedermayer wrote:
> inOn Wed, Aug 27, 2008 at 08:39:11AM -0400, Ronald S. Bultje wrote:
>> Hi Michael,
>>
>> On Wed, Aug 27, 2008 at 8:27 AM, Michael Niedermayer <michaelni at gmx.at>
>> wrote:
>> > Let me try again :(
>> >
>> > ff_data_to_hex must not be in a public header not directly or indirectly.
>> > NO public header must EVER include a non public header.
>> > avformat.h is a public header
>>
>> Sorry, the same scheme is used in lavu/common.h, I just moved on from
>> there...
>
> no
>
> lavu/common.h
> uses:
> #ifdef HAVE_AV_CONFIG_H
> #    include "internal.h"
> #endif /* HAVE_AV_CONFIG_H */
>
> So it only includes the internal header when common.h is used internally

And it does this because these files (common.h and internal.h) contain
stuff that practically every file in FFmpeg needs.

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