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.


Yes, during LVN_ENDLABELEDIT the edit is still focused, that's why taking focus away (example: by shwoing a mesagebox) triggers LVN_ENDLABELEDIT again.

 

 

> 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