Hi there,<br><br>here are 4 improvements for DrawIconEx in "dlls/user32/cursoricon.c".<br>The testsuite passes after applying each patch, as long as they are applied<br>in correct order:<br><br>1.) Fix for Bug <a href="http://bugs.winehq.org/show_bug.cgi?id=20145">http://bugs.winehq.org/show_bug.cgi?id=20145</a><br>
Only changed hierarchy of if statements to get monochrome 1 bpp icons into the "correct" <br>branch which doesn't compute alpha blending.<br><br>2.) Extremely small change to make a "wine todo" test pass<br>
Changed rendering flag for BitBlt<br>Remove "wine todo" from corresponding test<br><br>3.) Another small change to make the remaining two "wine todo" tests of DrawIconEx pass<br>Changed rendering flag to depend on DI_MASK flag<br>
Remove "wine todo" from both corresponding tests<br><br>4.) Replacing the DrawIcon implementation with now equivalent call to DrawIconEx <br>(as suggested by <a href="http://msdn.microsoft.com/en-us/library/ms648065(VS.85).aspx">http://msdn.microsoft.com/en-us/library/ms648065(VS.85).aspx</a> ), <br>
therefore eliminating lots of duplicated code.<br><br>Best regards,<br><br><br>Wilfried Pasquazzo<br><br><br>