Added a comment about the 'UpdateWindow' issue in SetScrollBars()

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@5385 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

diff --git a/src/generic/scrolwin.cpp b/src/generic/scrolwin.cpp
index caec7706091846f614af911e76edc2de0bb8de1a..3ae5a628894eeae33e2eeec94d7a647154ea3afe 100644 (file)
--- a/src/generic/scrolwin.cpp
+++ b/src/generic/scrolwin.cpp
@@ -154,9 +154,15 @@ void wxScrolledWindow::SetScrollbars (int pixelsPerUnitX, int pixelsPerUnitY,
     AdjustScrollbars();
    
     if (do_refresh && !noRefresh) 
     AdjustScrollbars();
    
     if (do_refresh && !noRefresh) 

-       m_targetWindow->Refresh(); 
+        m_targetWindow->Refresh(); 

    
 #ifdef __WXMSW__
    
 #ifdef __WXMSW__

+    // GRG: if this turns out to be really necessary, we could
+    //   at least move it to the above if { ... } so that it is
+    //   only done if noRefresh = FALSE (the default). OTOH, if
+    //   this doesn't break anything, which seems to be the
+    //   case, we could just leave it out.
+

     // Necessary?
     // UpdateWindow ((HWND) m_targetWindow->GetHWND());
 #endif
     // Necessary?
     // UpdateWindow ((HWND) m_targetWindow->GetHWND());
 #endif