[FFmpeg-devel] Realmedia patch

Aurelien Jacobs aurel
Wed Aug 27 13:53:00 CEST 2008


Ronald S. Bultje wrote:

> Hi Diego,
> 
> On Wed, Aug 27, 2008 at 2:34 AM, Diego Biurrun <diego at biurrun.de> wrote:
> > On Tue, Aug 26, 2008 at 09:38:03PM -0400, Ronald S. Bultje wrote:
> >>
> >> On Tue, Aug 26, 2008 at 9:34 PM, Aurelien Jacobs <aurel at gnuage.org> wrote:
> >> > avformat_internal.h ? The "avformat_" here is redundant. Just call
> >> > it internal.h.
> >>
> >> Won't that clash with lavutil/internal.h?
> >
> > Since we require full #include paths for files from other directories,
> > i.e. '#include "libavutil/internal.h"', it really should not, unless
> > somebody has set -I flags.  But people really should not...

We already have libavcodec/raw.h and libavformat/raw.h.
So people compiling FFmpeg with some stupid -I flags should be
prepared to surprises.

> lavu/common.h includes internal.h, not lavu/internal.h

So what ? internal.h will be searched in the same directory than the
file which included it, ie. lavu.

Aurel




More information about the ffmpeg-devel mailing list