Re: [PATCH 0/6] MR9191: comctl32/tests: Add CCM_SETVERSION tests.

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about include/commctrl.h:
> WINCOMMCTRLAPI HRESULT WINAPI LoadIconWithScaleDown(HINSTANCE, const WCHAR *, int, int, HICON *);
>  WINCOMMCTRLAPI HRESULT WINAPI LoadIconMetric(HINSTANCE, const WCHAR *, int, HICON *);
>
> -#define COMCTL32_VERSION                5  /* dll version */
> +#define COMCTL32_VERSION                6  /* dll version */

I wouldn't mind if we removed it instead completely, it doesn't look useful and we are not going to introduce sdk checks in the headers. Current MS SDK has it conditional to winxp sdk version, and same could be done for mingw-w64 headers for compatibility.