commit a39cbbc0ebb3311ebf0c9db6d30bd8f5d0b8df78
author Serge Ivanov <sergei@corel.ca> Fri Feb 25 20:47:26 2000 +0000
committer Alexandre Julliard <julliard@winehq.org> Fri Feb 25 20:47:26 2000 +0000
tree c014c8ad2f5b2d482a4e3d8473af3be191041d9a
parent 4f15239c3a214c78ccc079193219fb75edc6a688

In function IMAGELIST_InternalDrawMask we have to mask out overlay mask in
fStyle. Otherwise comparison fStyle == ILD_NORMAL will not work, which leads
to improper display of overlay images.

dlls/comctl32/imagelist.c

diff --git a/dlls/comctl32/imagelist.c b/dlls/comctl32/imagelist.c
index 6e2cd7a..5177bee7 100644
--- a/dlls/comctl32/imagelist.c
+++ b/dlls/comctl32/imagelist.c
@@ -191,10 +191,10 @@
     HBITMAP hOldBitmapImage, hOldBitmapMask;
     HIMAGELIST himlLocal = pimldp->himl;
     COLORREF oldBkColor, oldFgColor;
+    UINT fStyle = pimldp->fStyle & (~ILD_OVERLAYMASK);
 
     bUseCustomBackground = (himlLocal->clrBk != CLR_NONE);
-    bBlendFlag = (pimldp->fStyle & ILD_BLEND50 ) 
-        || (pimldp->fStyle & ILD_BLEND25);
+    bBlendFlag = (fStyle & ILD_BLEND50 ) || (fStyle & ILD_BLEND25);
 
     hImageDC = CreateCompatibleDC(0);
     hMaskDC = CreateCompatibleDC(0);
@@ -203,9 +203,8 @@
     hOldBitmapMask = SelectObject(hMaskDC, himlLocal->hbmMask);
     /* Draw the Background for the appropriate Styles
     */
-    if( bUseCustomBackground && (pimldp->fStyle == ILD_NORMAL 
-          || (pimldp->fStyle & ILD_IMAGE)
-          || bBlendFlag))
+    if( bUseCustomBackground &&
+	(fStyle == ILD_NORMAL || fStyle & ILD_IMAGE || bBlendFlag))
     {
         hBrush = CreateSolidBrush (himlLocal->clrBk);
         hOldBrush = SelectObject (pimldp->hdcDst, hBrush);
@@ -218,9 +217,9 @@
 
     /* Draw Image Transparently over the current background
     */
-    if(pimldp->fStyle == ILD_NORMAL
-        || (pimldp->fStyle & ILD_TRANSPARENT)
-        || ((pimldp->fStyle & ILD_IMAGE) && bUseCustomBackground)
+    if(fStyle == ILD_NORMAL
+        || (fStyle & ILD_TRANSPARENT)
+        || ((fStyle & ILD_IMAGE) && bUseCustomBackground)
         || bBlendFlag)
     {
         /* to obtain a transparent look, background color should be set
@@ -246,7 +245,7 @@
     }
     /* Draw the image when no Background is specified
     */
-    else if((pimldp->fStyle & ILD_IMAGE) && !bUseCustomBackground)
+    else if((fStyle & ILD_IMAGE) && !bUseCustomBackground)
     {
         BitBlt(pimldp->hdcDst, 
             pimldp->x, pimldp->y, cx, cy,
@@ -256,7 +255,7 @@
     }
     /* Draw the mask with or without a background
     */
-    else if(pimldp->fStyle & ILD_MASK)
+    else if(fStyle & ILD_MASK)
     {
         BitBlt(pimldp->hdcDst, 
             pimldp->x, pimldp->y, cx, cy,
@@ -388,7 +387,7 @@
     HDC     hImageDC;
     HBITMAP hOldBitmap;
 
-    nOvlIdx = (pimldp->fStyle & 0x0700) >> 8;
+    nOvlIdx = (pimldp->fStyle & ILD_OVERLAYMASK) >> 8;
     if ((nOvlIdx >= 1) && (nOvlIdx <= MAX_OVERLAYIMAGE))
     {
         nOvlIdx = pimldp->himl->nOvlIdx[nOvlIdx - 1];
@@ -1284,7 +1283,7 @@
     /*
         Apply the Overlay if needed
     */
-    if (pimldp->fStyle & 0x0700)
+    if (pimldp->fStyle & ILD_OVERLAYMASK)
     {
         IMAGELIST_InternalDrawOverlay(pimldp, cx, cy);
     }
@@ -1891,6 +1890,7 @@
 
 
 /* helper for _read_bitmap currently unused */
+#if 0
 static int may_use_dibsection(HDC hdc) {
     int bitspixel = GetDeviceCaps(hdc,BITSPIXEL)*GetDeviceCaps(hdc,PLANES);
     if (bitspixel>8)
@@ -1899,6 +1899,7 @@
 	return FALSE;
     return GetDeviceCaps(hdc,94) & 0x10;
 }
+#endif
 
 /* helper for ImageList_Read, see comments below */
 static HBITMAP _read_bitmap(LPSTREAM pstm,int ilcFlag,int cx,int cy) {