[PATCH] comctl32/listview: Don't send LVN_ENDLABELEDIT twice under some circumstances

[Fabian Maurer]

Hello Nikolay,

thanks for your comments.

> If we really end up doing this (see test comments), it'd be better to
> use a single field to mask change notifications and label one, see
> bDoChangeNotify.

Sure, that's reasonable.

> The question is if the focus is still on edit at this point. I remember
> we already had some problems with it, because applications can easily
> depend on exact sequence of events here, e.g. LVN_ENDLABELEDIT handler
> can interact with edit box directly, or EN_* messages could be intercepted.




> This should use a message sequence test to avoid another global variable.

Sure, this will take care of the "exact sequence" issue too.

> Does this fix some application or you just noticed that? We should fix
> that regardless, I'm just curious.

It affects wine regedit. Just open it, got to "HKCU/Environment" and try renaming TEMP to 
TEMp. You'll get the error message twice, due to this bug. It's not a big issue really, but I'd 
like it fixed.


Regards,
Fabian Maurer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20180825/106620dc/attachment.html>