"Ilya Shpigor" <[EMAIL PROTECTED]> wrote: What is the difference between wpParent1 and wpParent2 besides slightly different indentation and 1 superfluous 'break' statement?
> > - /* why do we notify to es->hwndParent, and we send this one to > > GetParent()? */ > > + /* All notifies are send to es->hwndParent > > + * except the WM_CTLCOLORSTATIC and WM_CTLCOLOREDIT. > > + * Its are send to the current parent. > > + */ > > hbrush = (HBRUSH)SendMessageW(GetParent(es->hwndSelf), msg, (WPARAM)hdc, > > (LPARAM)es->hwndSelf); > > if (!hbrush) > > hbrush = (HBRUSH)DefWindowProcW(GetParent(es->hwndSelf), msg, (WPARAM)hdc, > > (LPARAM)es->hwndSelf); > > This tests confirming my statement in the comment. If we change > GetParent(es->hwndSelf) to es->hwndParent the tests will be failed. I don't see the tests confirming the comment. Why are you adding the whole new message handling code to edit.c (which complicates things a lot, and makes it less clear what you are trying to do) while msg.c already has everything required, and it already contains the edit message tests? -- Dmitry.