made wxFFile a bit more safe: don't crash when Tell() and Length() are called on...

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

diff --git a/src/cocoa/colour.mm b/src/cocoa/colour.mm
index c6b73d70f196ace5dabcfcaefb29c50d00950328..dd7fd3e73c8005d78d217a82f2073187668094a6 100644 (file)
--- a/src/cocoa/colour.mm
+++ b/src/cocoa/colour.mm
@@ -9,19 +9,25 @@
 // Licence:    wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
-#import <AppKit/NSColor.h>
+#include "wx/wxprec.h"
+#ifndef WX_PRECOMP
+#endif //WX_PRECOMP
 
 #include "wx/gdicmn.h"
 #include "wx/colour.h"
 
 #include "wx/cocoa/autorelease.h"
 
+#import <AppKit/NSColor.h>
+
 IMPLEMENT_DYNAMIC_CLASS(wxColour, wxObject)
 
-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)
@@ -43,21 +49,20 @@ wxColour& wxColour::operator =(const wxColour& col)
     return *this;
 }
 
-void wxColour::InitFromName(const wxString& col)
+void wxColour::InitFromName(const wxString& name)
 {
-    wxColour *the_colour = wxTheColourDatabase->FindColour (col);
-    if (the_colour)
+    if ( wxTheColourDatabase )
     {
-        *this = *the_colour;
-    }
-    else
-    {
-        [m_cocoaNSColor release];
-        m_cocoaNSColor = NULL;
-        m_red = 0;
-        m_green = 0;
-        m_blue = 0;
+        wxColour col = wxTheColourDatabase->Find(name);
+        if ( col.Ok() )
+        {
+            *this = col;
+            return;
+        }
     }
+
+    // leave invalid
+    Init();
 }
 
 wxColour::~wxColour ()