]> git.saurik.com Git - wxWidgets.git/commitdiff
Fix r71367: don't set bgCol twice needlessly.
authorVáclav Slavík <vslavik@fastmail.fm>
Sun, 6 May 2012 16:56:38 +0000 (16:56 +0000)
committerVáclav Slavík <vslavik@fastmail.fm>
Sun, 6 May 2012 16:56:38 +0000 (16:56 +0000)
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@71368 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

src/msw/combo.cpp

index f6c705008c6ac9865f785623081f3d0c5f451439..2ef7d8925a4df77ecac03c0d629f194a933c0ca4 100644 (file)
@@ -420,15 +420,12 @@ void wxComboCtrl::OnPaintEvent( wxPaintEvent& WXUNUSED(event) )
 {
     // TODO: Convert drawing in this function to Windows API Code
 
-    const bool useVistaComboBox = ::wxGetWinVersion() >= wxWinVersion_Vista;
-
     wxSize sz = GetClientSize();
     wxDC* dcPtr = wxAutoBufferedPaintDCFactory(this);
     wxDC& dc = *dcPtr;
 
     const wxRect& rectButton = m_btnArea;
     wxRect rectTextField = m_tcArea;
-    wxColour bgCol = GetBackgroundColour();
 
     // FIXME: Either SetBackgroundColour or GetBackgroundColour
     //        doesn't work under Vista, so here's a temporary
@@ -436,7 +433,7 @@ void wxComboCtrl::OnPaintEvent( wxPaintEvent& WXUNUSED(event) )
     //        In the theme-less rendering code below, this fixes incorrect
     //        background on read-only comboboxes (they are gray, but should be
     //        white).
-    bgCol = wxSystemSettings::GetColour(wxSYS_COLOUR_WINDOW);
+    wxColour bgCol = wxSystemSettings::GetColour(wxSYS_COLOUR_WINDOW);
 
 #if wxUSE_UXTHEME
     const bool isEnabled = IsThisEnabled();
@@ -465,6 +462,8 @@ void wxComboCtrl::OnPaintEvent( wxPaintEvent& WXUNUSED(event) )
 #if wxUSE_UXTHEME
     if ( hTheme )
     {
+        const bool useVistaComboBox = ::wxGetWinVersion() >= wxWinVersion_Vista;
+
         RECT rFull;
         wxCopyRectToRECT(borderRect, rFull);