Re: [QUARTZ] Add FFMpeg video wrapper filter

[Robert Shearman]

Christian Costa wrote:
> Index: dlls/quartz/quartz_private.h
> ===================================================================
> RCS file: /home/wine/wine/dlls/quartz/quartz_private.h,v
> retrieving revision 1.24
> diff -u -r1.24 quartz_private.h
> --- dlls/quartz/quartz_private.h	6 May 2005 14:34:02 -0000	1.24
> +++ dlls/quartz/quartz_private.h	9 Jun 2005 21:54:23 -0000
> @@ -53,6 +53,9 @@
> HRESULT QUARTZ_CreateSystemClock(IUnknown * pUnkOuter, LPVOID * ppv);
> HRESULT ACMWrapper_create(IUnknown * pUnkOuter, LPVOID * ppv);
> HRESULT WAVEParser_create(IUnknown * pUnkOuter, LPVOID * ppv);
> +#ifdef HAVE_LIBAVCODEC
> +HRESULT FFMpegVideoWrapper_create(IUnknown * pUnkOuter, LPVOID * ppv);
> +#endif
>
> HRESULT EnumMonikerImpl_Create(IMoniker ** ppMoniker, ULONG nMonikerCount, IEnumMoniker ** ppEnum);
>
> @@ -81,5 +84,9 @@
> void DeleteMediaType(AM_MEDIA_TYPE * pmt);
> BOOL CompareMediaTypes(const AM_MEDIA_TYPE * pmt1, const AM_MEDIA_TYPE * pmt2, BOOL bWildcards);
> void dump_AM_MEDIA_TYPE(const AM_MEDIA_TYPE * pmt);
> +
> +#ifdef HAVE_LIBAVCODEC
> +extern CLSID CLSID_FFMpegVideoWrapper;
> +#endif
>
> #endif /* __QUARTZ_PRIVATE_INCLUDED__ */
> Index: dlls/quartz/regsvr.c
> ===================================================================
> RCS file: /home/wine/wine/dlls/quartz/regsvr.c,v
> retrieving revision 1.21
> diff -u -r1.21 regsvr.c
> --- dlls/quartz/regsvr.c	17 May 2005 14:41:37 -0000	1.21
> +++ dlls/quartz/regsvr.c	9 Jun 2005 21:54:36 -0000
> @@ -18,6 +18,8 @@
>  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
>  */
>
> +#include "config.h"
> +
> #define NONAMELESSUNION
> #define NONAMELESSSTRUCT
> #define COBJMACROS
> @@ -840,6 +842,10 @@
>     return res;
> }
>
> +#ifdef HAVE_LIBAVCODEC
> +CLSID CLSID_FFMpegVideoWrapper = { 0x30355649, 0x0000, 0x0010, { 0x80, 0x00, 0x00, 0xAA, 0x00, 0x38, 0x9B, 0x72 } };
> +#endif
> +
> /***********************************************************************
>  *		coclass list
>  */
> @@ -928,6 +934,14 @@
>   "quartz.dll",
>   "Both"
>     },
> +#ifdef HAVE_LIBAVCODEC
> +    {   &CLSID_FFMpegVideoWrapper,
> +	"FFMpeg Video Wrapper",
> +	NULL,
> +	"quartz.dll",
> +	"Both"
> +    },
> +#endif
>     { NULL }			/* list terminator */
> };
>
> @@ -1118,6 +1132,26 @@
>       { 0xFFFFFFFF },
>   }
>     },
> +    /* Wine specific filters */
> +#ifdef HAVE_LIBAVCODEC
> +    {   &CLSID_FFMpegVideoWrapper,
> +	&CLSID_LegacyAmFilterCategory,
> +	{'F','F','M','p','e','g',' ','V','i','d','e','o',' ','W','r','a','p','p','e','r',0},
> +	0x5FFFFF, /* Just below AVIDec */
> +	{   {   0,
> +		{   { &MEDIATYPE_Video, &GUID_NULL },
> +		    { NULL }
> +		},
> +	    },
> +	    {   REG_PINFLAG_B_OUTPUT,
> +		{   { &MEDIATYPE_Video, &GUID_NULL },
> +		    { NULL }
> +		},
> +	    },
> +	    { 0xFFFFFFFF },
> +	}
> +    },
> +#endif
>     { NULL }		/* list terminator */
> };
>
> Index: dlls/quartz/main.c
> ===================================================================
> RCS file: /home/wine/wine/dlls/quartz/main.c,v
> retrieving revision 1.47
> diff -u -r1.47 main.c
> --- dlls/quartz/main.c	6 Jun 2005 19:50:36 -0000	1.47
> +++ dlls/quartz/main.c	9 Jun 2005 21:54:37 -0000
> @@ -72,7 +72,10 @@
>     { &CLSID_AVIDec, AVIDec_create },
>     { &CLSID_SystemClock, &QUARTZ_CreateSystemClock },
>     { &CLSID_ACMWrapper, &ACMWrapper_create },
> -    { &CLSID_WAVEParser, &WAVEParser_create }
> +    { &CLSID_WAVEParser, &WAVEParser_create },
> +#ifdef HAVE_LIBAVCODEC
> +    { &CLSID_FFMpegVideoWrapper, &FFMpegVideoWrapper_create },
> +#endif
> };
>
> static HRESULT WINAPI

I think it would be better to always register the 
CLSID_FFMpegVideoWrapper class and just make FFMpegVideoWrapper_create 
fail if the libraries or headers aren't present. This will allow people 
to install these at a later date and after a recompile have the filter 
working, without having to re-register the DLL. It will also have the 
advantage of decreasing the number of ugly ifdefs everywhere.

-- 
Rob Shearman