[PATCH] comctl32: Initialize early some edit control properties.

Nikolay Sivov nsivov at codeweavers.com
Thu Mar 26 04:01:40 CDT 2020



On 3/25/20 2:12 PM, Roman Pišl wrote:
> Hello Nikolay,
> bug 37144 is also marked as user32 and closed as duplicate of 19239 
> but also fixed by this. Should I mention the bug 37144 in commit 
> message or will be properly re-closed in bugzilla?
>
> My thought was - make the initialization early, it can't hurt 
> anything, but it is also obvious that EDIT_EM_EmptyUndoBuffer is 
> useless - the buffer is already zeroed by heap_alloc_zero() so the 
> removal is intentional. Should I put it back to WM_Create?
>
> Ok, will prepare tests.

I've attached test diff to https://bugs.winehq.org/show_bug.cgi?id=48803 
after your comment, which shows that moving initialization earlier has 
side effects.

>
> Thanks
> Roman
>
> Dne 25. 03. 20 v 11:52 Nikolay Sivov napsal(a):
>> Please add a test that crashes without this change. Bug 19239 is 
>> linked to user32, so that should be addressed separately.
>>
>> P.S. there is no need to more comment, or to remove 
>> EDIT_EM_EmptyUndoBuffer().
>>
>




More information about the wine-devel mailing list