Re: [PATCH] includes: CHOOSECOLOR.lCustData is an LPARAM.

[Nikolay Sivov]

On 17.09.2017 11:48, Stefan Dösinger wrote:
> This matches the PSDK and MinGW headers.
>
> Signed-off-by: Stefan Dösinger <stefan(a)codeweavers.com>
> ---
>  include/commdlg.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/commdlg.h b/include/commdlg.h
> index df56c143c0..eeefce2bfe 100644
> --- a/include/commdlg.h
> +++ b/include/commdlg.h
> @@ -183,7 +183,7 @@ typedef struct {
>   DWORD	        rgbResult;
>   LPDWORD         lpCustColors;
>   DWORD 		Flags;
> -	DWORD		lCustData;
> +	LPARAM		lCustData;
>          LPCCHOOKPROC    lpfnHook;
>   LPCSTR 		lpTemplateName;
>  } CHOOSECOLORA;
> @@ -196,7 +196,7 @@ typedef struct {
>   DWORD	        rgbResult;
>   LPDWORD         lpCustColors;
>   DWORD 		Flags;
> -	DWORD		lCustData;
> +	LPARAM		lCustData;
>          LPCCHOOKPROC    lpfnHook;
>   LPCWSTR 	lpTemplateName;
>  } CHOOSECOLORW;

Another difference, not near as important, is that SDK and mingw are
using COLORREFs for color fields.