Update: comctl32: fix listview select-all

Vitaliy Margolen wine-devel at kievinfo.com
Sat Jul 5 20:13:26 CDT 2008


winehq.10.drkshadow at spamgourmet.com wrote:
> Fixes the Listview control to not individually select and focus each 
> items. Has the added bugfix that it keeps the same item focused as was 
> to begin with, and doesn't scroll the list.
> 
> Updated: my if was incorrect and it was handling cases it shouldn't have 
> handled. It's improved now, and as far as I can tell handles only the 
> select-all case.

Please fix the formatting - 4 spaces indentation. No tabs. Do not reformat 
the lines that you did not modify. Follow the file style with curly brackets 
- here they go on the separate line.

Is there a reason you added whole new function SelectAll()  which is used in 
only one place and has only 4 lines?

And the most important - you missed your real name. Patches from anonymous 
posters are not accepted.

Vitaliy.



More information about the wine-devel mailing list