"Ilya Shpigor" <shpigor(a)etersoft.ru> 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.