X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/c58ae5f298f4f052e22bf544d1249cf9572770f8..51f5e2822cdcb8b433b1d117f186dfd8bb06e240:/include/wx/cocoa/colour.h

diff --git a/include/wx/cocoa/colour.h b/include/wx/cocoa/colour.h
index 8d8671f060..db7a4e9444 100644
--- a/include/wx/cocoa/colour.h
+++ b/include/wx/cocoa/colour.h
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
-// Name:        colour.h
+// Name:        wx/cocoa/colour.h
 // Purpose:     wxColour class
 // Author:      David Elliott
 // Modified by:
@@ -18,27 +18,20 @@
 // ========================================================================
 // wxColour
 // ========================================================================
-class WXDLLEXPORT wxColour: public wxObject
+
+class WXDLLEXPORT wxColour : public wxColourBase
 {
 public:
+    // constructors
+    // ------------
+
+    // default
     wxColour() { Init(); }
+    DEFINE_STD_WXCOLOUR_CONSTRUCTORS
+
+    // initialization using existing NSColor
+    wxColour( WX_NSColor aColor );
 
-    // from RGB
-    wxColour( unsigned char red, unsigned char green, unsigned char blue )
-    :   m_cocoaNSColor(NULL)
-    {   Set(red,green,blue); }
-    wxColour( unsigned long colRGB )
-    :   m_cocoaNSColor(NULL)
-    {   Set(colRGB); }
-
-    // implicit conversion from the colour name
-    wxColour( const wxString &colourName )
-    {   InitFromName(colourName); }
-    wxColour( const char *colourName )
-    {   InitFromName(wxString::FromAscii(colourName)); }
-#if wxUSE_UNICODE
-    wxColour( const wxChar *colourName ) { InitFromName( wxString(colourName) ); }
-#endif
 
     // copy ctors and assignment operators
     wxColour( const wxColour& col );
@@ -57,32 +50,23 @@ public:
     // comparison
     bool operator == (const wxColour& colour) const
     {
-        // VZ: sure we want to compare NSColor objects for equality here?
+        // TODO: Really compare the NSColor
         return (m_cocoaNSColor == colour.m_cocoaNSColor
-            && m_red == colour.m_red
+            || (m_red == colour.m_red
             && m_green == colour.m_green
-            && m_blue == colour.m_blue);
+            && m_blue == colour.m_blue));
     }
     bool operator != (const wxColour& colour) const
     {   return !(*this == colour); }
 
     // Set() functions
-    void Set( unsigned char red, unsigned char green, unsigned char blue );
-    void Set( unsigned long colRGB )
-    {
-        // we don't need to know sizeof(long) here because we assume that the three
-        // least significant bytes contain the R, G and B values
-        Set((unsigned char)colRGB,
-            (unsigned char)(colRGB >> 8),
-            (unsigned char)(colRGB >> 16));
-    }
+    void Set( WX_NSColor aColor );
 
 protected:
     // puts the object in an invalid, uninitialized state
     void Init();
 
-    // create the object from name, leaves it uninitialized if it failed
-    void InitFromName(const wxString& col);
+    virtual void InitWith( unsigned char red, unsigned char green, unsigned char blue );
 
 private:
     WX_NSColor m_cocoaNSColor;