[PATCH] riched20: Process WM_GETMINMAXINFO which arrives before WM_NCCREATE.

[Alexandre Julliard]

Dmitry Timoshkov <dmitry at codeweavers.com> writes:

> Austin Lund <austin.lund at gmail.com> wrote:
>> I'm not sure how that's relevant because if the messages aren't passed
>> then as long as a WM_NCCREATE is passed then the editor variable is
>> initialised by this.
>
> What happens if an application doesn't pass WM_NCCREATE?

It would be legitimate for a window to not be properly initialized in
that case. Richedit may or may not have a workaround for this, but it's
not a requirement of the API.

>> I'm reading this comment carefully for a suggestion as to how to "fix"
>> things, but it seems totally obfuscated.
>
> Editor should be unconditionally initialized if it's NULL, and WM_NCDESTROY
> should be made an explicit case in the 'switch' statement.

Of course with appropriate test cases demonstrating that this is
actually the correct behavior...

-- 
Alexandre Julliard
julliard at winehq.org