From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Fri, 15 Jun 2012 23:23:23 +0000 (+0000)
Subject: Remove unnecessary base class OnPaint() call from wxGenericColourDialog.
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/4a699e3a59b19c21b6faae714b56cac5a75df2e2?ds=sidebyside

Remove unnecessary base class OnPaint() call from wxGenericColourDialog.

Events should be skipped instead of calling the base class event handler
directly, but in this case even this is not needed as wxGenericColourDialog
takes care of processing the event completely.

Closes #14405.

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

diff --git a/src/generic/colrdlgg.cpp b/src/generic/colrdlgg.cpp
index 3b5695ab7d..49e4bb6a45 100644
--- a/src/generic/colrdlgg.cpp
+++ b/src/generic/colrdlgg.cpp
@@ -200,14 +200,8 @@ void wxGenericColourDialog::OnMouseEvent(wxMouseEvent& event)
       event.Skip();
 }
 
-void wxGenericColourDialog::OnPaint(wxPaintEvent& event)
+void wxGenericColourDialog::OnPaint(wxPaintEvent& WXUNUSED(event))
 {
-#if !defined(__WXMOTIF__) && !defined(__WXPM__) && !defined(__WXCOCOA__) && !defined(__WXOSX__)
-    wxDialog::OnPaint(event);
-#else
-    wxUnusedVar(event);
-#endif
-
     wxPaintDC dc(this);
 
     PaintBasicColours(dc);