[FFmpeg-devel] Realmedia patch

[Ronald S. Bultje]

Hi Aurel,

On Wed, Aug 27, 2008 at 7:53 AM, Aurelien Jacobs <aurel at gnuage.org> wrote:
> Ronald S. Bultje wrote:
>> 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.

OK, I didn't know that. Patch attached with your suggested modification then.

Ronald
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: sdp-export-data_to_hex.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080827/ed20379e/attachment.txt>