[PATCH] user32: Handle VK_RETURN KeyDown events more properly in EDITcontrols.

El. eth.bell at gmail.com
Sat Apr 19 00:46:22 CDT 2008


> "El." <elptr at users.sourceforge.net> wrote:
>
> > @@ -4606,6 +4606,10 @@ static LRESULT EDIT_WM_KeyDown(EDITSTATE *es, INT key)
> >      {
> >    HWND hwndParent = GetParent(es->hwndSelf);
> >    DWORD dw = SendMessageW( hwndParent, DM_GETDEFID, 0, 0 );
> > +
> > +  if (GetClassLongW(hwndParent, GCW_ATOM) != WC_DIALOG)
> > +      break;
>
> You need to add a test case first to confirm that the fix is correct, and
> use EDIT_IsInsideDialog to detect if the parent is dialog.

Well I'm not sure what to test in the first place, I didn't assume that
modifying some aspect of the code to be in compliance with official
"standards" required writing any test case.

I might be missing what you mean by the "correctness" of the fix,
since I'm not really familiar with Wine's patching practices.

http://msdn2.microsoft.com/en-us/library/bb775464(VS.85).aspx

As for using EDIT_IsInsideDialog, that I can manage.



More information about the wine-devel mailing list