[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