X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/7ecb8b06ade47981aa828a3f7fb4278f822456d3..6af984df34c71bc14648a7b5bf1ad3c3cd07fc05:/src/gtk1/pen.cpp

diff --git a/src/gtk1/pen.cpp b/src/gtk1/pen.cpp
index c398c71b58..53464758d1 100644
--- a/src/gtk1/pen.cpp
+++ b/src/gtk1/pen.cpp
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
-// Name:        pen.cpp
+// Name:        src/gtk1/pen.cpp
 // Purpose:
 // Author:      Robert Roebling
 // Id:          $Id$
@@ -7,13 +7,15 @@
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
-
-#ifdef __GNUG__
-#pragma implementation "pen.h"
-#endif
+// For compilers that support precompilation, includes "wx.h".
+#include "wx/wxprec.h"
 
 #include "wx/pen.h"
 
+#ifndef WX_PRECOMP
+    #include "wx/colour.h"
+#endif
+
 #include <gdk/gdk.h>
 
 //-----------------------------------------------------------------------------
@@ -23,43 +25,62 @@
 class wxPenRefData: public wxObjectRefData
 {
 public:
+    wxPenRefData()
+    {
+        m_width = 1;
+        m_style = wxSOLID;
+        m_joinStyle = wxJOIN_ROUND;
+        m_capStyle = wxCAP_ROUND;
+        m_dash = (wxGTKDash*) NULL;
+        m_countDashes = 0;
+    }
 
-  wxPenRefData();
-  wxPenRefData( const wxPenRefData& data );
-
-  int        m_width;
-  int        m_style;
-  int        m_joinStyle;
-  int        m_capStyle;
-  wxColour   m_colour;
-  int        m_countDashes;
-  wxGTKDash *m_dash;
-};
+    wxPenRefData( const wxPenRefData& data )
+        : wxObjectRefData()
+    {
+        m_style = data.m_style;
+        m_width = data.m_width;
+        m_joinStyle = data.m_joinStyle;
+        m_capStyle = data.m_capStyle;
+        m_colour = data.m_colour;
+        m_countDashes = data.m_countDashes;
+        m_dash = data.m_dash;
+    }
 
-wxPenRefData::wxPenRefData()
-{
-    m_width = 1;
-    m_style = wxSOLID;
-    m_joinStyle = wxJOIN_ROUND;
-    m_capStyle = wxCAP_ROUND;
-    m_dash = (wxGTKDash*) NULL;
-    m_countDashes = 0;
-}
+    bool operator == (const wxPenRefData& data) const
+    {
+        if ( m_countDashes != data.m_countDashes )
+            return false;
+
+        if ( m_dash )
+        {
+            if ( !data.m_dash ||
+                 memcmp(m_dash, data.m_dash, m_countDashes*sizeof(wxGTKDash)) )
+            {
+                return false;
+            }
+        }
+        else if ( data.m_dash )
+        {
+            return false;
+        }
+
+
+        return m_style == data.m_style &&
+               m_width == data.m_width &&
+               m_joinStyle == data.m_joinStyle &&
+               m_capStyle == data.m_capStyle &&
+               m_colour == data.m_colour;
+    }
 
-wxPenRefData::wxPenRefData( const wxPenRefData& data )
-{
-    m_style = data.m_style;
-    m_width = data.m_width;
-    m_joinStyle = data.m_joinStyle;
-    m_capStyle = data.m_capStyle;
-    m_colour = data.m_colour;
-    m_countDashes = data.m_countDashes;
-/*
-    if (data.m_dash)  TODO
-      m_dash = new
-*/
-    m_dash = data.m_dash;
-}
+    int        m_width;
+    int        m_style;
+    int        m_joinStyle;
+    int        m_capStyle;
+    wxColour   m_colour;
+    int        m_countDashes;
+    wxGTKDash *m_dash;
+};
 
 //-----------------------------------------------------------------------------
 
@@ -67,10 +88,6 @@ wxPenRefData::wxPenRefData( const wxPenRefData& data )
 
 IMPLEMENT_DYNAMIC_CLASS(wxPen,wxGDIObject)
 
-wxPen::wxPen()
-{
-}
-
 wxPen::wxPen( const wxColour &colour, int width, int style )
 {
     m_refData = new wxPenRefData();
@@ -79,73 +96,77 @@ wxPen::wxPen( const wxColour &colour, int width, int style )
     M_PENDATA->m_colour = colour;
 }
 
-wxPen::wxPen( const wxPen& pen )
+wxPen::~wxPen()
 {
-    Ref( pen );
+    // m_refData unrefed in ~wxObject
 }
 
-wxPen::~wxPen()
+wxObjectRefData *wxPen::CreateRefData() const
 {
+    return new wxPenRefData;
 }
 
-wxPen& wxPen::operator = ( const wxPen& pen )
+wxObjectRefData *wxPen::CloneRefData(const wxObjectRefData *data) const
 {
-    if ( m_refData != pen.m_refData )
-        Ref( pen );
-
-    return *this;
+    return new wxPenRefData(*(wxPenRefData *)data);
 }
 
 bool wxPen::operator == ( const wxPen& pen ) const
 {
-    return m_refData == pen.m_refData;
-}
+    if (m_refData == pen.m_refData) return true;
 
-bool wxPen::operator != ( const wxPen& pen ) const
-{
-    return m_refData != pen.m_refData;
+    if (!m_refData || !pen.m_refData) return false;
+
+    return ( *(wxPenRefData*)m_refData == *(wxPenRefData*)pen.m_refData );
 }
 
 void wxPen::SetColour( const wxColour &colour )
 {
-    Unshare();
+    AllocExclusive();
+
     M_PENDATA->m_colour = colour;
 }
 
 void wxPen::SetDashes( int number_of_dashes, const wxDash *dash )
 {
-    Unshare();
+    AllocExclusive();
+
     M_PENDATA->m_countDashes = number_of_dashes;
-    M_PENDATA->m_dash = (wxGTKDash *)dash; /* TODO */
+    M_PENDATA->m_dash = (wxGTKDash *)dash;
 }
 
-void wxPen::SetColour( int red, int green, int blue )
+void wxPen::SetColour( unsigned char red, unsigned char green, unsigned char blue )
 {
-    Unshare();
+    AllocExclusive();
+
     M_PENDATA->m_colour.Set( red, green, blue );
 }
 
 void wxPen::SetCap( int capStyle )
 {
-    Unshare();
+    AllocExclusive();
+
     M_PENDATA->m_capStyle = capStyle;
 }
 
 void wxPen::SetJoin( int joinStyle )
 {
-    Unshare();
+    AllocExclusive();
+
     M_PENDATA->m_joinStyle = joinStyle;
 }
 
 void wxPen::SetStyle( int style )
 {
-    Unshare();
+    AllocExclusive();
+
     M_PENDATA->m_style = style;
 }
 
 void wxPen::SetWidth( int width )
 {
-    Unshare();
+    AllocExclusive();
+
     M_PENDATA->m_width = width;
 }
 
@@ -199,23 +220,3 @@ wxColour &wxPen::GetColour() const
 
     return M_PENDATA->m_colour;
 }
-
-bool wxPen::Ok() const
-{
-    return (m_refData != NULL);
-}
-
-void wxPen::Unshare()
-{
-    if (!m_refData)
-    {
-        m_refData = new wxPenRefData();
-    }
-    else
-    {
-        wxPenRefData* ref = new wxPenRefData( *(wxPenRefData*)m_refData );
-        UnRef();
-        m_refData = ref;
-    }
-}
-