Cast for argument to delete[] - VC6 doesn't want to delete a const pointer

[wxWidgets.git] / include / wx / mac / carbon / colour.h

diff --git a/include/wx/mac/carbon/colour.h b/include/wx/mac/carbon/colour.h
index f84671f77fdaa885d0a7ff2a44971eba9fcab4c7..431149f06060fa5554665cadb19a263608323b33 100644 (file)
--- a/include/wx/mac/carbon/colour.h
+++ b/include/wx/mac/carbon/colour.h
@@ -15,85 +15,61 @@
 #include "wx/object.h"
 #include "wx/string.h"
 
+#include "wx/mac/corefoundation/cfref.h"
+
+struct RGBColor;
+
 // Colour
-class WXDLLEXPORT wxColour: public wxObject
+class WXDLLIMPEXP_CORE wxColour: public wxColourBase
 {
 public:
     // constructors
     // ------------
+    DEFINE_STD_WXCOLOUR_CONSTRUCTORS
 
-    // default
-    wxColour() { Init(); }
-
-    // from separate RGB
-    wxColour( unsigned char red, unsigned char green, unsigned char blue )
-        { Set(red, green, blue); }
-
-    // from packed RGB
-    wxColour( unsigned long colRGB )
-        { Set(colRGB); }
-
-    // implicit conversion from the colour name
-    wxColour( const wxString &colourName )
-        { InitFromName(colourName); }
-    wxColour( const wxChar *colourName )
-        { InitFromName(colourName); }
-
-    // copy ctors and assignment operators
-    wxColour( const wxColour& col );
-    wxColour& operator = ( const wxColour& col );
-
-    // dtor
-    ~wxColour();
-
-    // 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));
-    }
+    // default copy ctor and dtor are ok
 
     // accessors
-    bool Ok() const {return m_isInit; }
+    virtual bool IsOk() const { return m_cgColour; }
 
-    unsigned char Red() const { return m_red; }
-    unsigned char Green() const { return m_green; }
-    unsigned char Blue() const { return m_blue; }
+    ChannelType Red() const { return m_red; }
+    ChannelType Green() const { return m_green; }
+    ChannelType Blue() const { return m_blue; }
+    ChannelType Alpha() const { return m_alpha; }
 
     // comparison
-    bool operator == (const wxColour& colour) const
-    {
-        return (m_isInit == colour.m_isInit
-                && m_red == colour.m_red
-                && m_green == colour.m_green
-                && m_blue == colour.m_blue);
-    }
+    bool operator == (const wxColour& colour) const;
+
     bool operator != (const wxColour& colour) const { return !(*this == colour); }
 
-    const WXCOLORREF& GetPixel() const { return m_pixel; };
+    CGColorRef GetPixel() const { return m_cgColour; };
 
-    void InitFromName(const wxString& col);
+    CGColorRef GetCGColor() const { return m_cgColour; };
+    CGColorRef CreateCGColor() const { return wxCFRetain( (CGColorRef)m_cgColour ); };
 
-protected :
+    void GetRGBColor( RGBColor *col ) const;
 
-    // Helper function
-    void Init();
+    // Mac-specific ctor and assignment operator from the native colour
+    // assumes ownership of CGColorRef
+    wxColour( CGColorRef col );
+    wxColour(const RGBColor& col);
+    wxColour& operator=(const RGBColor& col);
+    wxColour& operator=(CGColorRef col);
+    wxColour& operator=(const wxColour& col);
 
+protected :
+    virtual void
+    InitRGBA(ChannelType r, ChannelType g, ChannelType b, ChannelType a);
+    void InitRGBColor( const RGBColor& col );
+    void InitCGColorRef( CGColorRef col );
 private:
-    bool          m_isInit;
-    unsigned char m_red;
-    unsigned char m_blue;
-    unsigned char m_green;
+    wxCFRef<CGColorRef>     m_cgColour;
 
-public:
-    WXCOLORREF m_pixel ;
-    void Set( const WXCOLORREF* color ) ;
+    ChannelType             m_red;
+    ChannelType             m_blue;
+    ChannelType             m_green;
+    ChannelType             m_alpha;
 
-private:
     DECLARE_DYNAMIC_CLASS(wxColour)
 };