I don't like this, but add the treeview refresh hack.
authorGed Murphy <gedmurphy@reactos.org>
Wed, 9 Jun 2010 13:31:02 +0000 (13:31 +0000)
committerGed Murphy <gedmurphy@reactos.org>
Wed, 9 Jun 2010 13:31:02 +0000 (13:31 +0000)
I don't see this bug on my (windows) build but others are reporting it still happens on the (linux) build machine iso.
Maybe it's because linux is akin to the devil? Yes, I like this, let's blame linux!

svn path=/trunk/; revision=47712

reactos/dll/win32/comctl32/comctl32_ros.diff
reactos/dll/win32/comctl32/treeview.c

index c4f34bf..8b0db1f 100644 (file)
@@ -120,3 +120,22 @@ Index: tooltips.c
 +    }
      return 0;
  }
+Index: treeview.c
+===================================================================
+--- treeview.c (revision 38890)
++++ treeview.c (working copy)
+@@ -2830,7 +2830,12 @@
+       }
+     }
+-    TREEVIEW_UpdateScrollBars(infoPtr);
++    //
++    // This is correct, but is causes and infinite loop of WM_PAINT messages, resulting
++    // in continuous painting of the scroll bar in reactos. Comment out until the real
++    // bug is found
++    // 
++    //TREEVIEW_UpdateScrollBars(infoPtr);
+     if (infoPtr->cdmode & CDRF_NOTIFYPOSTPAINT)
+       infoPtr->cdmode =
index f1ff9c5..8d2a6f3 100644 (file)
@@ -2865,7 +2865,12 @@ TREEVIEW_Refresh(TREEVIEW_INFO *infoPtr, HDC hdc, const RECT *rc)
        }
     }
 
-    TREEVIEW_UpdateScrollBars(infoPtr);
+    //
+    // This is correct, but is causes and infinite loop of WM_PAINT messages, resulting
+    // in continuous painting of the scroll bar in reactos. Comment out until the real
+    // bug is found
+    // 
+    //TREEVIEW_UpdateScrollBars(infoPtr);
 
     if (infoPtr->cdmode & CDRF_NOTIFYPOSTPAINT)
        infoPtr->cdmode =