[FFmpeg-devel] [PATCH] split png.c

Baptiste Coudurier baptiste.coudurier
Tue Jul 17 14:16:12 CEST 2007


Hi Attila,

Attila Kinali wrote:
> On Sun, 15 Jul 2007 17:12:46 +0100
> M?ns Rullg?rd <mans at mansr.com> wrote:
> 
>> I'd like to apply this.  Objections?
> 
> Would it be possible to call the header file lavc_png.h or 
> something similar? png.h clashes with the global header file
> of libpng which is placed in /usr/include. This already caused
> compile problems in MPlayer[1] and i'm sure it will cause problems
> for other projects too.

Sorry but Im against this, png.h is internal header and is not meant to
be included from any project directly. IMHO Mplayer is faulty here.

> [...]
> 
> [1] Yes M?ns, you broke MPlayer again ;-)

Expect Mplayer to be broken oftenly if it keeps using internal ffmpeg
structures, and not officially exported ones.

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312




More information about the ffmpeg-devel mailing list