[PATCH 11/31] oledlg: Initial management of private structure
fordialog box control.
Dmitry Timoshkov
dmitry at codeweavers.com
Thu Oct 19 10:07:14 CDT 2006
"Huw Davies" <huw at codeweavers.com> wrote:
> case WM_COMMAND:
> switch(LOWORD(wp))
> {
> case IDOK:
> case IDCANCEL:
> + if(wp == IDOK)
> + update_structure(hdlg, ps_struct);
> EndDialog(hdlg, wp);
> + free_structure(ps_struct);
> return TRUE;
> }
Probably moving update_structure(hdlg, ps_struct); under case IDOK:
with falling through then could help with avoiding not entirely correct
'if(wp == IDOK)' since it doesn't ignore high word of wp.
--
Dmitry.
More information about the wine-devel
mailing list