В сообщении от 9 декабря 2007 Dmitry Timoshkov написал(a):
"Vitaly Lipatov" lav@etersoft.ru wrote:
--- dlls/commdlg/printdlg.c 11 Jan 2006 11:37:03 -0000 1.100 +++ dlls/comdlg32/printdlg.c 24 Feb 2006 17:47:53 -0000
The way you've generated this patch looks pretty strange.
Just a too old patch, partially updated by script.
/* Call user paint hook if enable */ if (pda->dlga->Flags & PSD_ENABLEPAGEPAINTHOOK)
if (pda->dlga->lpfnPagePaintHook(hwndDlg, uMsg, wParam, lParam))
if (pda->dlga->lpfnPagePaintHook &&pda->dlga->lpfnPagePaintHook(hwndDlg, uMsg, wParam, lParam))
You have to add the checks existing in PageSetupDlgA to PageSetupDlgW instead.
Instead or also? I guess, this check is needed in this place in any way.
Also, in order to increase the probability of your patches to be included should either point to existing bug that each of your patches fix, or/and add a test case which doesn't pass without your patch and does with it.
I do not know a way to check print dialogs without installed printer drivers in the system/wine.