<br><br><div class="gmail_quote">2011/11/18 Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org">jb@videolan.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
On Fri, Nov 18, 2011 at 05:56:21PM +0700, Sergey Radionov wrote :<br>
> +#ifndef _MSC_VER<br>
>  ////////////////////////////////////////////////////////////////////////////////////////////////<br>
>  DEFINE_GUID(IID_IGlobalInterfaceTable,     0x00000146, 0x0000, 0x0000, 0xc0,0x00, 0x00,0x00,0x00,0x00,0x00,0x46);<br>
>  DEFINE_GUID(CLSID_StdGlobalInterfaceTable, 0x00000323, 0x0000, 0x0000, 0xc0,0x00, 0x00,0x00,0x00,0x00,0x00,0x46);<br>
> @@ -40,6 +41,7 @@ DEFINE_GUID(CLSID_StdGlobalInterfaceTable, 0x00000323, 0x0000, 0x0000, 0xc0,0x00<br>
>  const GUID  IID_IGlobalInterfaceTable = { 0x00000146, 0, 0, {0xc0, 0, 0, 0, 0, 0, 0, 0x46} };<br>
>  const CLSID CLSID_StdGlobalInterfaceTable = { 0x00000323, 0, 0, {0xc0, 0, 0, 0, 0, 0, 0, 0x46} };<br>
>  ////////////////////////////////////////////////////////////////////////////////////////////////<br>
> +#endif<br>
<br>
Why not #ifdef _MINGW? It seems the issue is a lack of those GUID in the<br>
mingw header, not a fault of MSC<br></blockquote><div>Ok.<br>  <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
> +#ifdef __MINGW32__<br>
>  #include <_mingw.h><br>
> +#endif<br>
Ok.<br>
<br>
> -#ifdef __MINGW64_VERSION_MAJOR<br>
> -<br>
> -#include <guiddef.h><br>
>  #include <objsafe.h><br>
><br>
> -#else /* ! __MINGW64_VERSION_MAJOR */<br>
> -<br>
> +#if defined(__MINGW32_MAJOR_VERSION) && !defined(__MINGW64_VERSION_MAJOR)<br>
>  /*<br>
>  ** Widl generated code requires guiddef.h,<br>
>  ** which is not available under MinGW32<br>
> @@ -54,13 +52,11 @@<br>
>  /* CATID_InternetAware is declared as extern but not present in library */<br>
>  DEFINE_GUID(CATID_InternetAware, \<br>
>      0x0DE86A58, 0x2BAA, 0x11CF, 0xA2, 0x29, 0x00,0xAA,0x00,0x3D,0x73,0x52);<br>
> -static DEFINE_GUID(CATID_SafeForInitializing, \<br>
> -     0x7DD95802, 0x9882, 0x11CF, 0x9F, 0xA9, 0x00,0xAA,0x00,0x6C,0x42,0xC4);<br>
> -static DEFINE_GUID(CATID_SafeForScripting, \<br>
> -     0x7DD95801, 0x9882, 0x11CF, 0x9F, 0xA9, 0x00,0xAA,0x00,0x6C,0x42,0xC4);<br>
> -<br>
> -#endif /* __MINGW64_VERSION_MAJOR */<br>
> -<br>
> +DEFINE_GUID(CATID_SafeForInitializing, \<br>
> +    0x7DD95802, 0x9882, 0x11CF, 0x9F, 0xA9, 0x00,0xAA,0x00,0x6C,0x42,0xC4);<br>
> +DEFINE_GUID(CATID_SafeForScripting, \<br>
> +    0x7DD95801, 0x9882, 0x11CF, 0x9F, 0xA9, 0x00,0xAA,0x00,0x6C,0x42,0xC4);<br>
> +#endif<br>
OK.<br>
<br>
> +#ifdef _MSC_VER<br>
> +#define PROGID_STR "VideoLAN.VLCPlugin"<br>
> +#else<br>
>  #define PROGID_STR COMPANY_STR"."PROGRAM_STR<br>
> +#endif<br>
<br>
Still not at ease with this, but well...<br>
<br>
>          // ToolboxBitmap32 key value<br>
>          {<br>
> -            TCHAR iconPath[pathLen+3];<br>
> +            std::vector<TCHAR> iconPathBuf(pathLen+3, 0);<br>
> +            TCHAR* iconPath = &iconPathBuf[0];<br>
Ok.<br>
<br>
> +#ifdef _MSC_VER<br>
> +#include "../vc_config.h"<br>
> +#endif //_MSC_VER<br>
> +<br>
>  using namespace std;<br>
><br>
>  ////////////////////////////////////////////////////////////////////////<br>
> diff --git a/activex/position.h b/activex/position.h<br>
> index ec1c7f8..8d179df 100644<br>
> --- a/activex/position.h<br>
> +++ b/activex/position.h<br>
> @@ -22,6 +22,10 @@<br>
>  #ifndef POSITION_H<br>
>  #define POSITION_H<br>
><br>
> +#ifdef _MSC_VER<br>
> +#include "../vc_config.h"<br>
> +#endif //_MSC_VER<br>
<br>
I am not such a fan of this... Can we do something clever in autotools?<br></blockquote><div>Visual Studio developers usually have no autotools (and MinGW at all) on computers, so this file must be generated by someone else and included to repo, or be some sort of this. <br>
Or it can have some other, <span id="result_box" class="short_text" lang="en"><span class="hps">more appropriate</span> <span class="hps">name</span></span>...<br> <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<br>
Best Regards,<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Jean-Baptiste Kempf<br>
<a href="http://www.jbkempf.com/" target="_blank">http://www.jbkempf.com/</a> - +33 672 704 734<br>
Sent from my Electronic Device<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</font></span></blockquote></div><br>