list view completely mangled in WINE

Robert Shearman rob at codeweavers.com
Tue Aug 3 03:41:46 CDT 2004


Dimitrie O. Paun wrote:

>On Tue, Aug 03, 2004 at 01:42:28AM +0200, Filip Navara wrote:
>  
>
>>have inserted debug messages on all the places where the listview 
>>display mode can be changed  and none of them appeared. After going once 
>>more through the whole code I found this. Obviously it seemed redundant 
>>and the only reason for doing that was workarounding problems with 
>>changed WS_VSCROLL and WS_HSCROLL styles (which correctly don't recieve 
>>WM_STYLECHANGED notifications from the scrolling code). So I tried 
>>removing it and ... whoa ... to my surprise it started to work.
>>    
>>
>
>The problem is that WM_STYLECHANGED is sometimes sent, sometimes not,
>and it seems so much easier to just update it every time we come in
>so we can be sure we're using the latest value, as we should. 
>

The toolbar control works fine just using WM_STYLECHANGED to notify when 
it needs to update its internal state and it seems to send the message 
for all style changes that it needs (it doesn't depend on WS_ styles).

>It's
>redundant, I agree, but it's neglijable in terms of performance so
>I've put it in due to it's simplicity. This way the dwStyle has a
>very simple to understand semantics. In fact, I've meant to remove
>it eventually, just like you did, but only as an optimization. However,
>doing so to fix a bug in a way that we don't understand worries me
>greatly. I'd rather keep it in, and fix the problem.
>

Rob





More information about the wine-devel mailing list