Applied patch [ 642157 ] [MSW] HMENU resource leak from wxMenuBar

[wxWidgets.git] / src / gtk1 / pen.cpp

diff --git a/src/gtk1/pen.cpp b/src/gtk1/pen.cpp
index 3152074605dfd075b720b6568b2acc4cc060a114..11f787bd324d12392c91d48f3c38e350227e877e 100644 (file)
--- a/src/gtk1/pen.cpp
+++ b/src/gtk1/pen.cpp
@@ -14,7 +14,7 @@
 
 #include "wx/pen.h"
 
 
 #include "wx/pen.h"
 

-#include "gdk/gdk.h"
+#include <gdk/gdk.h>

 
 //-----------------------------------------------------------------------------
 // wxPen
 
 //-----------------------------------------------------------------------------
 // wxPen


@@ -23,170 +23,186 @@
 class wxPenRefData: public wxObjectRefData
 {
 public:
 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( 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;
+/*
+        if (data.m_dash)  TODO
+            m_dash = new
+*/
+        m_dash = data.m_dash;
+    }

 
 

-  wxPenRefData(void);
-  wxPenRefData(const wxPenRefData& data);
-
-  int        m_width;
-  int        m_style;
-  int        m_joinStyle;
-  int        m_capStyle;
-  wxColour   m_colour;
+    bool operator == (const wxPenRefData& data) const
+    {
+        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);
+    }
+            
+    int        m_width;
+    int        m_style;
+    int        m_joinStyle;
+    int        m_capStyle;
+    wxColour   m_colour;
+    int        m_countDashes;
+    wxGTKDash *m_dash;

 };
 
 };
 

-wxPenRefData::wxPenRefData()
-{
-    m_width = 1;
-    m_style = wxSOLID;
-    m_joinStyle = wxJOIN_ROUND;
-    m_capStyle = wxCAP_ROUND;
-}
-
-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;
-}
-

 //-----------------------------------------------------------------------------
 
 #define M_PENDATA ((wxPenRefData *)m_refData)
 
 IMPLEMENT_DYNAMIC_CLASS(wxPen,wxGDIObject)
 
 //-----------------------------------------------------------------------------
 
 #define M_PENDATA ((wxPenRefData *)m_refData)
 
 IMPLEMENT_DYNAMIC_CLASS(wxPen,wxGDIObject)
 

-wxPen::wxPen()
-{
-    if (wxThePenList) wxThePenList->AddPen( this );
-}
-

 wxPen::wxPen( const wxColour &colour, int width, int style )
 {
     m_refData = new wxPenRefData();
     M_PENDATA->m_width = width;
     M_PENDATA->m_style = style;
     M_PENDATA->m_colour = colour;
 wxPen::wxPen( const wxColour &colour, int width, int style )
 {
     m_refData = new wxPenRefData();
     M_PENDATA->m_width = width;
     M_PENDATA->m_style = style;
     M_PENDATA->m_colour = colour;

-
-    if (wxThePenList) wxThePenList->AddPen( this );
-}
-
-wxPen::wxPen( const wxPen& pen )
-{
-    Ref( pen );
-    if (wxThePenList) wxThePenList->AddPen( this );

 }
 
 wxPen::~wxPen()
 {
 }
 
 wxPen::~wxPen()
 {

-    if (wxThePenList) wxThePenList->RemovePen( this );
+    // m_refData unrefed in ~wxObject

 }
 
 }
 

-wxPen& wxPen::operator = ( const wxPen& pen )
+wxObjectRefData *wxPen::CreateRefData() const

 {
 {

-    if (*this == pen) return (*this);
-    Ref( pen );
-    return *this;
+    return new wxPenRefData;

 }
 
 }
 

-bool wxPen::operator == ( const wxPen& pen )
+wxObjectRefData *wxPen::CloneRefData(const wxObjectRefData *data) const

 {
 {

-    return m_refData == pen.m_refData;
+    return new wxPenRefData(*(wxPenRefData *)data);

 }
 
 }
 

-bool wxPen::operator != ( const wxPen& pen )
+bool wxPen::operator == ( const wxPen& pen ) const

 {
 {

-    return m_refData != pen.m_refData;
+    if (m_refData == pen.m_refData) return TRUE;
+    
+    if (!m_refData || !pen.m_refData) return FALSE;
+    
+    return ( *(wxPenRefData*)m_refData == *(wxPenRefData*)pen.m_refData );

 }
 
 void wxPen::SetColour( const wxColour &colour )
 {
 }
 
 void wxPen::SetColour( const wxColour &colour )
 {

-    Unshare();
+    AllocExclusive();
+    

     M_PENDATA->m_colour = colour;
 }
 
     M_PENDATA->m_colour = colour;
 }
 

+void wxPen::SetDashes( int number_of_dashes, const wxDash *dash )
+{
+    AllocExclusive();
+    
+    M_PENDATA->m_countDashes = number_of_dashes;
+    M_PENDATA->m_dash = (wxGTKDash *)dash; /* TODO */
+}
+

 void wxPen::SetColour( int red, int green, int blue )
 {
 void wxPen::SetColour( int red, int green, int blue )
 {

-    Unshare();
+    AllocExclusive();
+    

     M_PENDATA->m_colour.Set( red, green, blue );
 }
 
 void wxPen::SetCap( int capStyle )
 {
     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 )
 {
     M_PENDATA->m_capStyle = capStyle;
 }
 
 void wxPen::SetJoin( int joinStyle )
 {

-    Unshare();
+    AllocExclusive();
+    

     M_PENDATA->m_joinStyle = joinStyle;
 }
 
 void wxPen::SetStyle( int style )
 {
     M_PENDATA->m_joinStyle = joinStyle;
 }
 
 void wxPen::SetStyle( int style )
 {

-    Unshare();
+    AllocExclusive();
+    

     M_PENDATA->m_style = style;
 }
 
 void wxPen::SetWidth( int width )
 {
     M_PENDATA->m_style = style;
 }
 
 void wxPen::SetWidth( int width )
 {

-    Unshare();
+    AllocExclusive();
+    

     M_PENDATA->m_width = width;
 }
 
     M_PENDATA->m_width = width;
 }
 

+int wxPen::GetDashes( wxDash **ptr ) const
+{
+     *ptr = (M_PENDATA ? (wxDash*)M_PENDATA->m_dash : (wxDash*) NULL);
+     return (M_PENDATA ? M_PENDATA->m_countDashes : 0);
+}
+
+int wxPen::GetDashCount() const
+{
+    return (M_PENDATA->m_countDashes);
+}
+
+wxDash* wxPen::GetDash() const
+{
+    return (wxDash*)M_PENDATA->m_dash;
+}
+

 int wxPen::GetCap() const
 {
 int wxPen::GetCap() const
 {

-    wxCHECK_MSG( Ok(), -1, "invalid pen" );
+    wxCHECK_MSG( Ok(), -1, wxT("invalid pen") );

 
     return M_PENDATA->m_capStyle;
 }
 
 int wxPen::GetJoin() const
 {
 
     return M_PENDATA->m_capStyle;
 }
 
 int wxPen::GetJoin() const
 {

-    wxCHECK_MSG( Ok(), -1, "invalid pen" );
+    wxCHECK_MSG( Ok(), -1, wxT("invalid pen") );

 
     return M_PENDATA->m_joinStyle;
 }
 
 int wxPen::GetStyle() const
 {
 
     return M_PENDATA->m_joinStyle;
 }
 
 int wxPen::GetStyle() const
 {

-    wxCHECK_MSG( Ok(), -1, "invalid pen" );
+    wxCHECK_MSG( Ok(), -1, wxT("invalid pen") );

 
     return M_PENDATA->m_style;
 }
 
 int wxPen::GetWidth() const
 {
 
     return M_PENDATA->m_style;
 }
 
 int wxPen::GetWidth() const
 {

-    wxCHECK_MSG( Ok(), -1, "invalid pen" );
+    wxCHECK_MSG( Ok(), -1, wxT("invalid pen") );

 
     return M_PENDATA->m_width;
 }
 
 wxColour &wxPen::GetColour() const
 {
 
     return M_PENDATA->m_width;
 }
 
 wxColour &wxPen::GetColour() const
 {

-    wxCHECK_MSG( Ok(), wxNullColour, "invalid pen" );
+    wxCHECK_MSG( Ok(), wxNullColour, wxT("invalid pen") );

 
     return M_PENDATA->m_colour;
 }
 
 
     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;
-    }
-}
-