[FFmpeg-devel] [PATCH] fix inclusion of avformat.h in avio.h

Måns Rullgård mans
Thu Feb 19 14:30:42 CET 2009


Diego Biurrun <diego at biurrun.de> writes:

> On Thu, Feb 19, 2009 at 10:37:51AM +0000, M?ns Rullg?rd wrote:
>> Stefano Sabatini <stefano.sabatini-lala at poste.it> writes:
>> 
>> > On date Wednesday 2009-02-18 12:37:50 +0100, Benoit Fouet encoded:
>> >> 
>> >> as LIBAVFORMAT_VERSION_MAJOR is used in avio.h, the latter needs to
>> >> include avformat.h
>> >> so far, so good...
>> >> the "problem" is that avformat.h includes avio.h, so the only way to do
>> >> it (simply) is to include avformat.h in avio.h *before* its multiple
>> >> inclusion guards.
>> >> 
>> >> Attached patch does that, and is waiting for your comments :)
>> >
>> > [...]
>> >
>> > I wonder if that would make sense to define version.h headers for each
>> > lib, only containing the versioning stuff.
>> 
>> That idea crossed my mind as well, and I think it is probably the
>> cleanest solution (even though the hack was my idea).
>
> The idea had crossed my mind as well, but I wondered if it wasn't
> overkill.  Since clean alternatives do not really exist...

Whoever decides to implement this, remember to update the version
extraction towards the end of configure.

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




More information about the ffmpeg-devel mailing list