otherwise we always fall back to blitting, even if we can provide better results

[wxWidgets.git] / src / mac / carbon / colour.cpp

diff --git a/src/mac/carbon/colour.cpp b/src/mac/carbon/colour.cpp
index 13b88761bc91ac7428e0930b9dafd3ccfb4342a1..49cd199d3a21b84a17bde72aaf287635528b31ca 100644 (file)
--- a/src/mac/carbon/colour.cpp
+++ b/src/mac/carbon/colour.cpp
@@ -9,10 +9,12 @@
 // Licence:       wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Licence:       wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 

-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)

 #pragma implementation "colour.h"
 #endif
 
 #pragma implementation "colour.h"
 #endif
 

+#include "wx/wxprec.h"
+

 #include "wx/gdicmn.h"
 #include "wx/colour.h"
 
 #include "wx/gdicmn.h"
 #include "wx/colour.h"
 


@@ -25,7 +27,7 @@ IMPLEMENT_DYNAMIC_CLASS(wxColour, wxObject)
 #include "wx/mac/private.h"
 
 static void wxComposeRGBColor( WXCOLORREF* color , int red, int blue, int green ) ;
 #include "wx/mac/private.h"
 
 static void wxComposeRGBColor( WXCOLORREF* color , int red, int blue, int green ) ;

-static void wxComposeRGBColor( WXCOLORREF* color , int red, int blue, int green ) 
+static void wxComposeRGBColor( WXCOLORREF* color , int red, int blue, int green )

 {
     RGBColor* col = (RGBColor*) color ;
     col->red = (red << 8) + red;
 {
     RGBColor* col = (RGBColor*) color ;
     col->red = (red << 8) + red;


@@ -35,11 +37,11 @@ static void wxComposeRGBColor( WXCOLORREF* color , int red, int blue, int green
 
 void wxColour::Init()
 {
 
 void wxColour::Init()
 {

-    m_isInit = FALSE;
+    m_isInit = false;

     m_red =
     m_blue =
     m_green = 0;
     m_red =
     m_blue =
     m_green = 0;

-    
+

     wxComposeRGBColor( &m_pixel , m_red , m_blue , m_green ) ;
 }
 
     wxComposeRGBColor( &m_pixel , m_red , m_blue , m_green ) ;
 }
 


@@ -54,39 +56,32 @@ wxColour::wxColour (const wxColour& col)
     memcpy( &m_pixel , &col.m_pixel , 6 ) ;
 }
 
     memcpy( &m_pixel , &col.m_pixel , 6 ) ;
 }
 

-wxColour::wxColour (const wxColour* col)
-{
-    m_red = col->m_red;
-    m_green = col->m_green;
-    m_blue = col->m_blue;
-    m_isInit = col->m_isInit;
-
-    memcpy( &m_pixel , &col->m_pixel , 6 ) ;
-}
-

 wxColour& wxColour::operator =(const wxColour& col)
 {
     m_red = col.m_red;
     m_green = col.m_green;
     m_blue = col.m_blue;
     m_isInit = col.m_isInit;
 wxColour& wxColour::operator =(const wxColour& col)
 {
     m_red = col.m_red;
     m_green = col.m_green;
     m_blue = col.m_blue;
     m_isInit = col.m_isInit;

-    
+

     memcpy( &m_pixel , &col.m_pixel , 6 ) ;
     memcpy( &m_pixel , &col.m_pixel , 6 ) ;

-    
+

     return *this;
 }
 
 void wxColour::InitFromName(const wxString& name)
 {
     return *this;
 }
 
 void wxColour::InitFromName(const wxString& name)
 {

-    wxColour col = wxTheColourDatabase->Find(name);
-    if ( col.Ok() )
-    {
-        *this = col;
-    }
-    else
+    if ( wxTheColourDatabase )

     {
     {

-        Init();
+        wxColour col = wxTheColourDatabase->Find(name);
+        if ( col.Ok() )
+        {
+            *this = col;
+            return;
+        }

     }
     }

+
+    // leave invalid
+    Init();

 }
 
 wxColour::~wxColour ()
 }
 
 wxColour::~wxColour ()


@@ -98,13 +93,13 @@ void wxColour::Set (unsigned char r, unsigned char g, unsigned char b)
     m_red = r;
     m_green = g;
     m_blue = b;
     m_red = r;
     m_green = g;
     m_blue = b;

-    m_isInit = TRUE;
+    m_isInit = true;

 
     wxComposeRGBColor( &m_pixel , m_red , m_blue , m_green ) ;
 }
 
 void wxColour::Set( const WXCOLORREF* color )
 
     wxComposeRGBColor( &m_pixel , m_red , m_blue , m_green ) ;
 }
 
 void wxColour::Set( const WXCOLORREF* color )

-{ 
+{

     RGBColor* col = (RGBColor*) color ;
     memcpy( &m_pixel , color , 6 ) ;
     m_red = col->red>>8 ;
     RGBColor* col = (RGBColor*) color ;
     memcpy( &m_pixel , color , 6 ) ;
     m_red = col->red>>8 ;