Re: [PATCH 11/31] oledlg: Initial management of private structure fordialog box control.

[Dmitry Timoshkov]

"Huw Davies"  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.