[Resend PATCH 2/2] gdi32: Do not fill in the color table iflpvBits is NULL

Jeremy White jwhite at codeweavers.com
Fri Jul 13 15:42:31 CDT 2007


> The patch mixes tabs and spaces. Also, in general, I don't think we
> should include text from MSDN.

*blush*.  Teach me to edit this on a test machine, without my .vimrc...

I'll tweak the comment, but I felt it was important to motivate
the if statement; I don't feel that it's obvious from the API
why you fill in color sometimes and not others.

Cheers,

Jeremy

p.s.  While I'm coming clean, *another* failure on my part
was to fail to credit Huw; this is mostly his insight.
I provided the clue, and the hopeless failure to get a clean patch in :-/.




More information about the wine-devel mailing list