[MPlayer-dev-eng] inclusion of loader/drv.h causes 1000+ conflicts with mingw headers

Diego Biurrun diego at biurrun.de
Thu Mar 18 14:42:35 CET 2010


On Thu, Mar 18, 2010 at 02:17:40PM +0100, Gianluigi Tiesi wrote:
> On Thu, Mar 18, 2010 at 11:49:16AM +0100, Diego Biurrun wrote:
> > On Thu, Mar 18, 2010 at 01:29:25AM +0100, Gianluigi Tiesi wrote:
> > > I think it's better to declare the function instead
> > > of the include
> > > 
> > > something like:
> > > extern void SetCodecPath(const char* path);
> > > 
> > > it only uses const char * as argument
> > 
> > I don't see a patch attached to this email.
> 
> there is no patch :D
> 
> but anyway I've have one
> 
> --- main/mencoder.c	2010-03-18 00:39:09.734375000 +0100
> +++ sherpya/mencoder.c	2010-03-18 01:31:55.125000000 +0100
> @@ -66,7 +66,7 @@
>  
>  #ifdef CONFIG_WIN32DLL
> -#include "loader/drv.h"         // for SetCodecPath()
> +extern void SetCodecPath(const char* path);
>  #endif
>  
> --- main/mplayer.c	2010-03-18 00:39:09.734375000 +0100
> +++ sherpya/mplayer.c	2010-03-18 01:31:43.015625000 +0100
> @@ -96,7 +96,7 @@
>  
>  #ifdef CONFIG_WIN32DLL
> -#include "loader/drv.h"         // for SetCodecPath()
> +extern void SetCodecPath(const char* path);
>  #endif

Rejected.  This adds the declaration twice and the #ifdefs are
no longer needed.

Diego



More information about the MPlayer-dev-eng mailing list