[PATCH] comctl32/listview: Don't send LVN_ENDLABELEDIT twice under some circumstances
dark.shadow4 at web.de
Sat Aug 25 06:32:13 CDT 2018
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
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.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the wine-devel