added wxColourBase::Init() and use DEFINE_STD_WXCOLOUR_CONSTRUCTORS() in all ports...

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

diff --git a/include/wx/cocoa/colour.h b/include/wx/cocoa/colour.h
index f0d7af427148f5f11653e78909406252af5c796e..b2bc5796f19eee8d79135930387b4ef4ffb84dd6 100644 (file)
--- a/include/wx/cocoa/colour.h
+++ b/include/wx/cocoa/colour.h
@@ -24,25 +24,11 @@ class WXDLLEXPORT wxColour : public wxColourBase
 public:
     // constructors
     // ------------
-
-    // default
-    wxColour() { Init(); }
-
-    // the other standard ones: notice that we can't use
-    // DEFINE_STD_WXCOLOUR_CONSTRUCTORS here because we need to call Init() to
-    // initialize m_cocoaNSColor and the macro doesn't do it
-    wxColour( ChannelType red, ChannelType green, ChannelType blue,
-              ChannelType alpha = wxALPHA_OPAQUE )
-        { Init(); Set(red, green, blue, alpha); }
-    wxColour(unsigned long colRGB) { Init(); Set(colRGB); }
-    wxColour(const wxString& colourName) { Init(); Set(colourName); }
-    wxColour(const char *colourName) { Init(); Set(colourName); }
-    wxColour(const wchar_t *colourName) { Init(); Set(colourName); }
+    DEFINE_STD_WXCOLOUR_CONSTRUCTORS
 
     // initialization using existing NSColor
     wxColour( WX_NSColor aColor );
 
-
     // copy ctors and assignment operators
     wxColour( const wxColour& col );
     wxColour& operator = ( const wxColour& col );

diff --git a/include/wx/colour.h b/include/wx/colour.h
index 1adc13ee2ebe9fc272aad70e1aacf3ecb10bd0a7..b0296e860ed1d64088e535f72ddfcb5bd566d792 100644 (file)
--- a/include/wx/colour.h
+++ b/include/wx/colour.h
@@ -18,18 +18,21 @@
 
 class WXDLLIMPEXP_FWD_CORE wxColour;
 
-// the standard wxColour constructors;
-// this macro avoids to repeat these lines across all colour.h files, since
-// Set() is a virtual function and thus cannot be called by wxColourBase
-// constructors
+// A macro to define the standard wxColour constructors:
+//
+// It avoids the need to repeat these lines across all colour.h files, since
+// Set() is a virtual function and thus cannot be called by wxColourBase ctors
 #define DEFINE_STD_WXCOLOUR_CONSTRUCTORS                                      \
-    wxColour( ChannelType red, ChannelType green, ChannelType blue,           \
-              ChannelType alpha = wxALPHA_OPAQUE )                            \
-        { Set(red, green, blue, alpha); }                                     \
-    wxColour( unsigned long colRGB ) { Set(colRGB); }                         \
-    wxColour(const wxString& colourName) { Set(colourName); }                 \
-    wxColour(const char *colourName) { Set(colourName); }                     \
-    wxColour(const wchar_t *colourName) { Set(colourName); }
+    wxColour() { Init(); }                                                    \
+    wxColour(ChannelType red,                                                 \
+             ChannelType green,                                               \
+             ChannelType blue,                                                \
+             ChannelType alpha = wxALPHA_OPAQUE)                              \
+        { Init(); Set(red, green, blue, alpha); }                             \
+    wxColour(unsigned long colRGB) { Init(); Set(colRGB    ); }               \
+    wxColour(const wxString& colourName) { Init(); Set(colourName); }         \
+    wxColour(const char *colourName) { Init(); Set(colourName); }             \
+    wxColour(const wchar_t *colourName) { Init(); Set(colourName); }
 
 
 // flags for wxColour -> wxString conversion (see wxColour::GetAsString)
@@ -129,6 +132,10 @@ public:
 #endif
 
 protected:
+    // Some ports need Init() and while we don't, provide a stub so that the
+    // ports which don't need it are not forced to define it
+    void Init() { }
+
     virtual void
     InitRGBA(ChannelType r, ChannelType g, ChannelType b, ChannelType a) = 0;
 

diff --git a/include/wx/generic/colour.h b/include/wx/generic/colour.h
index c0d26dad039ba7e93a69f65c637b66a6e36158bd..24fefe2f6be054f26b8467e01c9afa0bdf6d9d07 100644 (file)
--- a/include/wx/generic/colour.h
+++ b/include/wx/generic/colour.h
@@ -20,9 +20,6 @@ class WXDLLEXPORT wxColour: public wxColourBase
 public:
     // constructors
     // ------------
-
-    // default
-    wxColour();
     DEFINE_STD_WXCOLOUR_CONSTRUCTORS
 
     // copy ctors and assignment operators

diff --git a/include/wx/gtk/colour.h b/include/wx/gtk/colour.h
index 07121726d6d7869e5e47e2b44e33db13e54c0187..410705fe656b79f0861fbfa60e0bad7cdbf19faf 100644 (file)
--- a/include/wx/gtk/colour.h
+++ b/include/wx/gtk/colour.h
@@ -19,9 +19,6 @@ class WXDLLIMPEXP_CORE wxColour : public wxColourBase
 public:
     // constructors
     // ------------
-
-    // default
-    wxColour() {}
     DEFINE_STD_WXCOLOUR_CONSTRUCTORS
     wxColour(const GdkColor& gdkColor);
 

diff --git a/include/wx/gtk1/colour.h b/include/wx/gtk1/colour.h
index 4d4950fbba535baff77208c516aa33de131d1eaa..b1d06275d5e6442dcad0f316e61b3932f539c5bd 100644 (file)
--- a/include/wx/gtk1/colour.h
+++ b/include/wx/gtk1/colour.h
@@ -36,9 +36,6 @@ class WXDLLIMPEXP_CORE wxColour : public wxColourBase
 public:
     // constructors
     // ------------
-
-    // default
-    wxColour() { }
     DEFINE_STD_WXCOLOUR_CONSTRUCTORS
 
     virtual ~wxColour();

diff --git a/include/wx/mac/carbon/colour.h b/include/wx/mac/carbon/colour.h
index 1036cd8507b78e38fb8bbbd9bafba8630a4a90c6..22a3925e52d3465ecf23d385fd9f64144f973ec2 100644 (file)
--- a/include/wx/mac/carbon/colour.h
+++ b/include/wx/mac/carbon/colour.h
@@ -25,14 +25,9 @@ class WXDLLEXPORT wxColour: public wxColourBase
 public:
     // constructors
     // ------------
-
-    // default
-    wxColour() { }
-    wxColour( const wxColour& col );
     DEFINE_STD_WXCOLOUR_CONSTRUCTORS
 
-    // dtor
-    virtual ~wxColour();
+    // default copy ctor and dtor are ok
 
     // accessors
     virtual bool IsOk() const { return m_cgColour; }
@@ -44,14 +39,14 @@ public:
 
     // comparison
     bool operator == (const wxColour& colour) const;
-    
+
     bool operator != (const wxColour& colour) const { return !(*this == colour); }
 
     CGColorRef GetPixel() const { return m_cgColour; };
-    
+
     CGColorRef GetCGColor() const { return m_cgColour; };
     CGColorRef CreateCGColor() const { return wxCFRetain( (CGColorRef)m_cgColour ); };
-    
+
     void GetRGBColor( RGBColor *col ) const;
 
     // Mac-specific ctor and assignment operator from the native colour
@@ -63,8 +58,6 @@ public:
     wxColour& operator=(const wxColour& col);
 
 protected :
-
-
     virtual void
     InitRGBA(ChannelType r, ChannelType g, ChannelType b, ChannelType a);
     void InitRGBColor( const RGBColor& col );

diff --git a/include/wx/motif/colour.h b/include/wx/motif/colour.h
index a74c97321f3d8c352d988d7ff26a00a2627e7cc3..e7d12c14b28a134648200ed457a66512b3c4b477 100644 (file)
--- a/include/wx/motif/colour.h
+++ b/include/wx/motif/colour.h
@@ -22,9 +22,6 @@ class WXDLLEXPORT wxColour : public wxColourBase
 public:
     // constructors
     // ------------
-
-    // default
-    wxColour() { Init(); }
     DEFINE_STD_WXCOLOUR_CONSTRUCTORS
 
     // copy ctors and assignment operators

diff --git a/include/wx/msw/colour.h b/include/wx/msw/colour.h
index 14760092270976b49c0bd87f85c192a5a854d0f6..7f25bc88f44e76f08984ec103f34ac814ac4a08e 100644 (file)
--- a/include/wx/msw/colour.h
+++ b/include/wx/msw/colour.h
@@ -23,16 +23,8 @@ class WXDLLEXPORT wxColour : public wxColourBase
 public:
     // constructors
     // ------------
-
-    // default
-    wxColour() { Init(); }
     DEFINE_STD_WXCOLOUR_CONSTRUCTORS
 
-
-    // dtor
-    virtual ~wxColour();
-
-
     // accessors
     // ---------
 
@@ -53,11 +45,10 @@ public:
             && m_alpha == colour.m_alpha;
     }
 
-    bool operator != (const wxColour& colour) const { return !(*this == colour); }
+    bool operator!=(const wxColour& colour) const { return !(*this == colour); }
 
     WXCOLORREF GetPixel() const { return m_pixel; }
 
-
 public:
     WXCOLORREF m_pixel;
 
@@ -79,5 +70,4 @@ private:
     DECLARE_DYNAMIC_CLASS(wxColour)
 };
 
-#endif
-        // _WX_COLOUR_H_
+#endif // _WX_COLOUR_H_

diff --git a/include/wx/os2/colour.h b/include/wx/os2/colour.h
index 90e1fe0290c903f601f1ff2d47197e9562164b2c..7b92b0db031e3ea46e9362ddbc9426305fca55e0 100644 (file)
--- a/include/wx/os2/colour.h
+++ b/include/wx/os2/colour.h
@@ -20,9 +20,6 @@ class WXDLLEXPORT wxColour: public wxColourBase
 public:
     // constructors
     // ------------
-
-    // default
-    wxColour();
     DEFINE_STD_WXCOLOUR_CONSTRUCTORS
 
     // Copy ctors and assignment operators

diff --git a/include/wx/x11/colour.h b/include/wx/x11/colour.h
index 411665e88eef6c68741dc0feee982dfc8997cc68..8fce06d6a0d96047dfd2eaee31addd385ef194dd 100644 (file)
--- a/include/wx/x11/colour.h
+++ b/include/wx/x11/colour.h
@@ -38,8 +38,6 @@ class WXDLLEXPORT wxColour : public wxColourBase
 public:
     // constructors
     // ------------
-
-    wxColour() {}
     DEFINE_STD_WXCOLOUR_CONSTRUCTORS
 
     virtual ~wxColour();

diff --git a/src/mac/carbon/colour.cpp b/src/mac/carbon/colour.cpp
index b2a7f5884e12fe4ee6461b0477a51d013aaa74aa..4d91726650dd40ab9e8a7005eece2e38710899b7 100644 (file)
--- a/src/mac/carbon/colour.cpp
+++ b/src/mac/carbon/colour.cpp
@@ -26,15 +26,6 @@ wxColour::wxColour(const RGBColor& col)
     InitRGBColor(col);
 }
 
-wxColour::wxColour( const wxColour& col )
-{
-    m_red = col.m_red;
-    m_green = col.m_green;
-    m_blue = col.m_blue;
-    m_alpha = col.m_alpha;
-    m_cgColour = col.m_cgColour;
-}
-
 wxColour::wxColour(CGColorRef col)
 {
     InitCGColorRef(col);
@@ -47,10 +38,6 @@ void wxColour::GetRGBColor( RGBColor *col ) const
     col->green = (m_green << 8) + m_green;
 }
 
-wxColour::~wxColour ()
-{
-}
-
 wxColour& wxColour::operator=(const RGBColor& col)
 {
     InitRGBColor(col);

diff --git a/src/msw/colour.cpp b/src/msw/colour.cpp
index acd528ef631d73a5247c9deaf8776e7b6e24ce7e..9a9a13f98cb15c5aa380039ec223acd0f9edd505 100644 (file)
--- a/src/msw/colour.cpp
+++ b/src/msw/colour.cpp
@@ -74,10 +74,6 @@ void wxColour::Init()
     m_green = 0;
 }
 
-wxColour::~wxColour()
-{
-}
-
 void wxColour::InitRGBA(unsigned char r, unsigned char g, unsigned char b,
                         unsigned char a)
 {