Re: commctrl.h (+ other headers) patch

"Filip Navara" <xnavara-VIXq6x/[email protected]> Fri, 22 Aug 2003 23:04:28 +0200
Newsgroups gmane.comp.gnu.mingw.patches
Message-ID <001101c368f0$fa440f70$3144b13e@hornet>
----- Original Message ----- 
From: "Danny Smith" <[email protected]>
To: <[email protected]>
Sent: Friday, August 22, 2003 10:22 PM
Subject: Re: [MinGW-patches] commctrl.h (+ other headers) patch


>
> ----- Original Message -----
> From: "Filip Navara"
>
> > I tried compiling comctl32.dll from Wine with w32api and this patch is
> the
> > result of my work. Hope, it will get the way into w32api 2.4 :)
> >
> > FiN <xnavara-VIXq6x/[email protected]>
> >
> > REFERENCES:
> > Mostly MSDN and sometimes guessing & testing. If you want URLs for
> online
> > MSDN I can create and post it, but it would involve lot of work :(
> >
> > CHANGLOG (Also in attachment):
> >
> > 2003-08-22  Filip Navara  <xnavara-VIXq6x/[email protected]>
> >
> >  * include/objidl.h: Get rid of COBJMACRORS define.
> >  * include/unknwn.h: Ditto.
> >  * include/comcat.h: Ditto.
>
>

Ok, as I investigated it, this part of the patch was probably completly
wrong. I just did it that way because it was done same way in some other
(not w32api) headers. Pls, reject this part of patch. The rest should be OK.

Sorry.

FiN

> All of the hunks like this:
> -#ifdef COBJMACROS
> +#if !defined(__cplusplus) || defined(CINTERFACE)
>
>
> should probably be:
>
> -#ifdef COBJMACROS
> +#if (!defined (__cplusplus) || defined (CINTERFACE)) \
> +      && defined  (COBJMACROS)
>
> Danny
>
>
>
> -------------------------------------------------------
> This SF.net email is sponsored by: VM Ware
> With VMware you can run multiple operating systems on a single machine.
> WITHOUT REBOOTING! Mix Linux / Windows / Novell virtual machines
> at the same time. Free trial click
here:http://www.vmware.com/wl/offer/358/0
> _______________________________________________
> MinGW-patches mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/mingw-patches
>



-------------------------------------------------------
This SF.net email is sponsored by: VM Ware
With VMware you can run multiple operating systems on a single machine.
WITHOUT REBOOTING! Mix Linux / Windows / Novell virtual machines
at the same time. Free trial click here:http://www.vmware.com/wl/offer/358/0