[FFmpeg-devel] [PATCH] fix inclusion of avformat.h in avio.h
Thu Feb 19 12:49:41 CET 2009
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...
More information about the ffmpeg-devel