COMDLG32: PageSetupDlg improvements

Alexandre Julliard julliard at winehq.org
Thu Jun 23 06:25:01 CDT 2005


Vitaly Lipatov <lav at etersoft.ru> writes:

> +#define CALLANYPAINTHOOK(msg, wParam, lParam) \
> +    (((pda->dlga->Flags & PSD_ENABLEPAGEPAINTHOOK) && \
> +    pda->dlga->lpfnPagePaintHook(hWnd,msg,wParam,lParam)) || \
> +    PRINTDLG_DefaultPagePaintHook(hWnd,msg,wParam,lParam))

It would be a lot cleaner to always call your DefaultPagePaintHook
function and have it call the user's hook first if it's set.

> +    if (CALLANYPAINTHOOK(WM_PSD_PAGESETUPDLG, MAKELONG(papersize, orientation), (LPARAM)pda))
> +        return FALSE;

The LPARAM should probably be pda->dlga here.

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list