Fix out of bounds string access in wxMSW wxDirDialog.

[wxWidgets.git] / src / cocoa / colour.mm

diff --git a/src/cocoa/colour.mm b/src/cocoa/colour.mm
index b755914c4c77d25d61ef422feddf0d7bd7657980..df7e25e0d054b1415f1b7894ba3c3c5d80205145 100644 (file)
--- a/src/cocoa/colour.mm
+++ b/src/cocoa/colour.mm
@@ -1,25 +1,33 @@
 /////////////////////////////////////////////////////////////////////////////
-// Name:        colour.mm
+// Name:        src/cococa/colour.mm
 // Purpose:     wxColour class
 // Author:      David Elliott
 // Modified by:
 // Created:     2003/06/17
 // RCS-ID:      $Id$
 // Copyright:   (c) 2003 David Elliott
-// Licence:    wxWindows licence
+// Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
-#import <AppKit/NSColor.h>
+#include "wx/wxprec.h"
 
-#include "wx/gdicmn.h"
 #include "wx/colour.h"
 
-IMPLEMENT_DYNAMIC_CLASS(wxColour, wxObject)
+#ifndef WX_PRECOMP
+    #include "wx/gdicmn.h"
+#endif //WX_PRECOMP
+
+#include "wx/cocoa/autorelease.h"
+#include "wx/cocoa/ObjcRef.h"
+
+#import <AppKit/NSColor.h>
 
-wxColour::wxColour ()
-:   m_cocoaNSColor(NULL)
+void wxColour::Init()
 {
-    m_red = m_blue = m_green = 0;
+    m_cocoaNSColor = NULL;
+    m_red =
+    m_blue =
+    m_green = 0;
 }
 
 wxColour::wxColour (const wxColour& col)
@@ -27,48 +35,58 @@ 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)
+{
+    wxGCSafeRetain(m_cocoaNSColor);
+}
+
+wxColour::wxColour( WX_NSColor aColor )
+:   m_cocoaNSColor(nil)
 {
-    [m_cocoaNSColor retain];
+    Set(aColor);
 }
 
 wxColour& wxColour::operator =(const wxColour& col)
 {
-    m_cocoaNSColor = col.m_cocoaNSColor;
+    m_cocoaNSColor = wxGCSafeRetain(col.m_cocoaNSColor);
     m_red = col.m_red;
     m_green = col.m_green;
     m_blue = col.m_blue;
-    [m_cocoaNSColor retain];
+    m_alpha = col.m_alpha;
     return *this;
 }
 
-void wxColour::InitFromName(const wxString& col)
-{
-    wxColour *the_colour = wxTheColourDatabase->FindColour (col);
-    if (the_colour)
-    {
-        *this = *the_colour;
-    }
-    else
-    {
-        [m_cocoaNSColor release];
-        m_cocoaNSColor = NULL;
-        m_red = 0;
-        m_green = 0;
-        m_blue = 0;
-    }
-}
-
 wxColour::~wxColour ()
 {
-    [m_cocoaNSColor release];
+    wxGCSafeRelease(m_cocoaNSColor);
 }
 
-void wxColour::Set (unsigned char r, unsigned char g, unsigned char b)
+void wxColour::InitRGBA(unsigned char r,
+                        unsigned char g,
+                        unsigned char b,
+                        unsigned char a)
 {
-    [m_cocoaNSColor release];
-    m_cocoaNSColor = [[NSColor colorWithCalibratedRed:r/255.0 green:g/255.0 blue:b/255.0 alpha:1.0] retain];
+    wxAutoNSAutoreleasePool pool;
+    wxGCSafeRelease(m_cocoaNSColor);
+    m_cocoaNSColor = wxGCSafeRetain([NSColor colorWithCalibratedRed:r/255.0 green:g/255.0 blue:b/255.0 alpha:a/255.0]);
     m_red = r;
     m_green = g;
     m_blue = b;
+    m_alpha = a;
 }
 
+void wxColour::Set( WX_NSColor aColor )
+{
+    wxGCSafeRetain(aColor);
+    wxGCSafeRelease(m_cocoaNSColor);
+    m_cocoaNSColor = aColor;
+
+    /* Make a temporary color in RGB format and get the values.  Note that
+       unless the color was actually RGB to begin with it's likely that
+       these will be fairly bogus. Particulary if the color is a pattern. */
+    NSColor *rgbColor = [m_cocoaNSColor colorUsingColorSpaceName:NSCalibratedRGBColorSpace];
+    m_red   = (wxUint8) ([rgbColor redComponent]   * 255.0);
+    m_green = (wxUint8) ([rgbColor greenComponent] * 255.0);
+    m_blue  = (wxUint8) ([rgbColor blueComponent]  * 255.0);
+    m_alpha  = (wxUint8) ([rgbColor alphaComponent]  * 255.0);
+}