No changes, just remove wxSplitterWindow::{Set,Get}NeedUpdating().
authorVadim Zeitlin <vadim@wxwidgets.org>
Sun, 30 Oct 2011 19:48:34 +0000 (19:48 +0000)
committerVadim Zeitlin <vadim@wxwidgets.org>
Sun, 30 Oct 2011 19:48:34 +0000 (19:48 +0000)
GetNeedUpdating() was never used at all while SetNeedUpdating() was used only
once while m_needUpdating variable was being changed directly in other places
making the code difficult to follow.

Just use the member variable directly and remove the accessors.

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

include/wx/generic/splitter.h
src/generic/splitter.cpp

index fdaeb8023ac72257159e14410f8526068a137724..2b75bb1d98356ac514f723e47ea6b716c9283e7e 100644 (file)
@@ -211,9 +211,6 @@ public:
     // Resizes subwindows
     virtual void SizeWindows();
 
-    void SetNeedUpdating(bool needUpdating) { m_needUpdating = needUpdating; }
-    bool GetNeedUpdating() const { return m_needUpdating ; }
-
 #ifdef __WXMAC__
     virtual bool MacClipGrandChildren() const { return true ; }
 #endif
index 968c1a1d588ab70a097731783df07e9a05bcc128..c5833830c88af4e8c57ef231d6f8178780c7cc33 100644 (file)
@@ -203,7 +203,10 @@ void wxSplitterWindow::OnInternalIdle()
     }
 
     if (m_needUpdating)
+    {
+        m_needUpdating = false;
         SizeWindows();
+    }
 }
 
 void wxSplitterWindow::OnMouseEvent(wxMouseEvent& event)
@@ -709,8 +712,6 @@ void wxSplitterWindow::SizeWindows()
 
     wxClientDC dc(this);
     DrawSash(dc);
-
-    SetNeedUpdating(false);
 }
 
 // Set pane for unsplit window