From afe0e776b94eb1544da706c5166a246578d80ab8 Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Sat, 20 Jan 2018 14:14:05 +0100 Subject: [PATCH] Remove confusing "base_type" typedef from wxGTK3 wxDC code This typedef isn't really useful in base class initializer lists and it wasn't really used anywhere else. --- include/wx/gtk/dc.h | 7 ------- src/gtk/dc.cpp | 25 +++++++++++++------------ 2 files changed, 13 insertions(+), 19 deletions(-) diff --git a/include/wx/gtk/dc.h b/include/wx/gtk/dc.h index 2c08c1f200..6c8fb391e3 100644 --- a/include/wx/gtk/dc.h +++ b/include/wx/gtk/dc.h @@ -15,7 +15,6 @@ class wxGTKCairoDCImpl: public wxGCDCImpl { - typedef wxGCDCImpl base_type; public: wxGTKCairoDCImpl(wxDC* owner); wxGTKCairoDCImpl(wxDC* owner, int); @@ -42,7 +41,6 @@ protected: class wxWindowDCImpl: public wxGTKCairoDCImpl { - typedef wxGTKCairoDCImpl base_type; public: wxWindowDCImpl(wxWindowDC* owner, wxWindow* window); @@ -52,7 +50,6 @@ public: class wxClientDCImpl: public wxGTKCairoDCImpl { - typedef wxGTKCairoDCImpl base_type; public: wxClientDCImpl(wxClientDC* owner, wxWindow* window); @@ -62,7 +59,6 @@ public: class wxPaintDCImpl: public wxGTKCairoDCImpl { - typedef wxGTKCairoDCImpl base_type; public: wxPaintDCImpl(wxPaintDC* owner, wxWindow* window); @@ -72,7 +68,6 @@ public: class wxScreenDCImpl: public wxGTKCairoDCImpl { - typedef wxGTKCairoDCImpl base_type; public: wxScreenDCImpl(wxScreenDC* owner); @@ -82,7 +77,6 @@ public: class wxMemoryDCImpl: public wxGTKCairoDCImpl { - typedef wxGTKCairoDCImpl base_type; public: wxMemoryDCImpl(wxMemoryDC* owner); wxMemoryDCImpl(wxMemoryDC* owner, wxBitmap& bitmap); @@ -102,7 +96,6 @@ private: class WXDLLIMPEXP_CORE wxGTKCairoDC: public wxDC { - typedef wxDC base_type; public: wxGTKCairoDC(cairo_t* cr, wxWindow* window); diff --git a/src/gtk/dc.cpp b/src/gtk/dc.cpp index 401551b7f7..711b96d517 100644 --- a/src/gtk/dc.cpp +++ b/src/gtk/dc.cpp @@ -21,21 +21,21 @@ #include wxGTKCairoDCImpl::wxGTKCairoDCImpl(wxDC* owner) - : base_type(owner) + : wxGCDCImpl(owner) { m_width = 0; m_height = 0; } wxGTKCairoDCImpl::wxGTKCairoDCImpl(wxDC* owner, int) - : base_type(owner, 0) + : wxGCDCImpl(owner, 0) { m_width = 0; m_height = 0; } wxGTKCairoDCImpl::wxGTKCairoDCImpl(wxDC* owner, double scaleFactor) - : base_type(owner, 0) + : wxGCDCImpl(owner, 0) { m_width = 0; m_height = 0; @@ -43,7 +43,7 @@ wxGTKCairoDCImpl::wxGTKCairoDCImpl(wxDC* owner, double scaleFactor) } wxGTKCairoDCImpl::wxGTKCairoDCImpl(wxDC* owner, wxWindow* window) - : base_type(owner, 0) + : wxGCDCImpl(owner, 0) { m_window = window; m_font = window->GetFont(); @@ -236,10 +236,11 @@ void* wxGTKCairoDCImpl::GetCairoContext() const cr = static_cast(m_graphicContext->GetNativeContext()); return cr; } + //----------------------------------------------------------------------------- wxWindowDCImpl::wxWindowDCImpl(wxWindowDC* owner, wxWindow* window) - : base_type(owner, window) + : wxGTKCairoDCImpl(owner, window) { GtkWidget* widget = window->m_wxwindow; if (widget == NULL) @@ -285,7 +286,7 @@ wxWindowDCImpl::wxWindowDCImpl(wxWindowDC* owner, wxWindow* window) //----------------------------------------------------------------------------- wxClientDCImpl::wxClientDCImpl(wxClientDC* owner, wxWindow* window) - : base_type(owner, window) + : wxGTKCairoDCImpl(owner, window) { GtkWidget* widget = window->m_wxwindow; if (widget == NULL) @@ -325,7 +326,7 @@ wxClientDCImpl::wxClientDCImpl(wxClientDC* owner, wxWindow* window) //----------------------------------------------------------------------------- wxPaintDCImpl::wxPaintDCImpl(wxPaintDC* owner, wxWindow* window) - : base_type(owner, window) + : wxGTKCairoDCImpl(owner, window) { cairo_t* cr = window->GTKPaintContext(); wxCHECK_RET(cr, "using wxPaintDC without being in a native paint event"); @@ -339,7 +340,7 @@ wxPaintDCImpl::wxPaintDCImpl(wxPaintDC* owner, wxWindow* window) //----------------------------------------------------------------------------- wxScreenDCImpl::wxScreenDCImpl(wxScreenDC* owner) - : base_type(owner, 0) + : wxGTKCairoDCImpl(owner, 0) { GdkWindow* window = gdk_get_default_root_window(); m_width = gdk_window_get_width(window); @@ -353,20 +354,20 @@ wxScreenDCImpl::wxScreenDCImpl(wxScreenDC* owner) //----------------------------------------------------------------------------- wxMemoryDCImpl::wxMemoryDCImpl(wxMemoryDC* owner) - : base_type(owner) + : wxGTKCairoDCImpl(owner) { m_ok = false; } wxMemoryDCImpl::wxMemoryDCImpl(wxMemoryDC* owner, wxBitmap& bitmap) - : base_type(owner, 0) + : wxGTKCairoDCImpl(owner, 0) , m_bitmap(bitmap) { Setup(); } wxMemoryDCImpl::wxMemoryDCImpl(wxMemoryDC* owner, wxDC*) - : base_type(owner) + : wxGTKCairoDCImpl(owner) { m_ok = false; } @@ -411,7 +412,7 @@ void wxMemoryDCImpl::Setup() //----------------------------------------------------------------------------- wxGTKCairoDC::wxGTKCairoDC(cairo_t* cr, wxWindow* window) - : base_type(new wxGTKCairoDCImpl(this, window->GetContentScaleFactor())) + : wxDC(new wxGTKCairoDCImpl(this, window->GetContentScaleFactor())) { wxGraphicsContext* gc = wxGraphicsContext::CreateFromNative(cr); gc->EnableOffset(window->GetContentScaleFactor() <= 1);