bug with object being passed through ... corrected

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

diff --git a/src/motif/pen.cpp b/src/motif/pen.cpp
index a9536ca90c803e1cf31e6b4a8dfedc2b44a81f05..8d0c0a47cf3fd8bfb10c91f579cd1b520e235874 100644 (file)
--- a/src/motif/pen.cpp
+++ b/src/motif/pen.cpp
@@ -29,9 +29,6 @@ wxPenRefData::wxPenRefData()
     m_cap = wxCAP_ROUND ;
     m_nbDash = 0 ;
     m_dash = 0 ;
     m_cap = wxCAP_ROUND ;
     m_nbDash = 0 ;
     m_dash = 0 ;

-/* TODO: null data
-    m_hPen = 0;
-*/

 }
 
 wxPenRefData::wxPenRefData(const wxPenRefData& data)
 }
 
 wxPenRefData::wxPenRefData(const wxPenRefData& data)


@@ -43,14 +40,10 @@ wxPenRefData::wxPenRefData(const wxPenRefData& data)
     m_nbDash = data.m_nbDash;
     m_dash = data.m_dash;
     m_colour = data.m_colour;
     m_nbDash = data.m_nbDash;
     m_dash = data.m_dash;
     m_colour = data.m_colour;

-/* TODO: null data
-    m_hPen = 0;
-*/

 }
 
 wxPenRefData::~wxPenRefData()
 {
 }
 
 wxPenRefData::~wxPenRefData()
 {

-    // TODO: delete data

 }
 
 // Pens
 }
 
 // Pens


@@ -71,7 +64,7 @@ wxPen::~wxPen()
 wxPen::wxPen(const wxColour& col, int Width, int Style)
 {
     m_refData = new wxPenRefData;
 wxPen::wxPen(const wxColour& col, int Width, int Style)
 {
     m_refData = new wxPenRefData;

-
+    

     M_PENDATA->m_colour = col;
     M_PENDATA->m_width = Width;
     M_PENDATA->m_style = Style;
     M_PENDATA->m_colour = col;
     M_PENDATA->m_width = Width;
     M_PENDATA->m_style = Style;


@@ -79,17 +72,17 @@ wxPen::wxPen(const wxColour& col, int Width, int Style)
     M_PENDATA->m_cap = wxCAP_ROUND ;
     M_PENDATA->m_nbDash = 0 ;
     M_PENDATA->m_dash = 0 ;
     M_PENDATA->m_cap = wxCAP_ROUND ;
     M_PENDATA->m_nbDash = 0 ;
     M_PENDATA->m_dash = 0 ;

-
+    

     RealizeResource();
     RealizeResource();

-
-    if ( wxThePenList )
+    
+    if (wxThePenList)

         wxThePenList->AddPen(this);
 }
 
 wxPen::wxPen(const wxBitmap& stipple, int Width)
 {
     m_refData = new wxPenRefData;
         wxThePenList->AddPen(this);
 }
 
 wxPen::wxPen(const wxBitmap& stipple, int Width)
 {
     m_refData = new wxPenRefData;

-
+    

     M_PENDATA->m_stipple = stipple;
     M_PENDATA->m_width = Width;
     M_PENDATA->m_style = wxSTIPPLE;
     M_PENDATA->m_stipple = stipple;
     M_PENDATA->m_width = Width;
     M_PENDATA->m_style = wxSTIPPLE;


@@ -97,133 +90,105 @@ wxPen::wxPen(const wxBitmap& stipple, int Width)
     M_PENDATA->m_cap = wxCAP_ROUND ;
     M_PENDATA->m_nbDash = 0 ;
     M_PENDATA->m_dash = 0 ;
     M_PENDATA->m_cap = wxCAP_ROUND ;
     M_PENDATA->m_nbDash = 0 ;
     M_PENDATA->m_dash = 0 ;

-
+    

     RealizeResource();
     RealizeResource();

-
-    if ( wxThePenList )
-        wxThePenList->AddPen(this);
-}
-
-wxPen::wxPen(const wxString& col, int Width, int Style)
-{
-    m_refData = new wxPenRefData;
-
-    M_PENDATA->m_colour = col;
-    M_PENDATA->m_width = Width;
-    M_PENDATA->m_style = Style;
-    M_PENDATA->m_join = wxJOIN_ROUND ;
-    M_PENDATA->m_cap = wxCAP_ROUND ;
-    M_PENDATA->m_nbDash = 0 ;
-    M_PENDATA->m_dash = 0 ;
-
-    RealizeResource();
-
+    

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

-       // Don't change shared data
-       if (!m_refData)
+    // Don't change shared data
+    if (!m_refData)

     {
     {

-               m_refData = new wxPenRefData();
-       }
+        m_refData = new wxPenRefData();
+    }

     else
     {
     else
     {

-               wxPenRefData* ref = new wxPenRefData(*(wxPenRefData*)m_refData);
-               UnRef();
-               m_refData = ref;
-       }
+        wxPenRefData* ref = new wxPenRefData(*(wxPenRefData*)m_refData);
+        UnRef();
+        m_refData = ref;
+    }

 }
 
 void wxPen::SetColour(const wxColour& col)
 {
     Unshare();
 }
 
 void wxPen::SetColour(const wxColour& col)
 {
     Unshare();

-
+    

     M_PENDATA->m_colour = col;
     M_PENDATA->m_colour = col;

-  
+    

     RealizeResource();
 }
 
     RealizeResource();
 }
 

-void wxPen::SetColour(const wxString& col)
+void wxPen::SetColour(unsigned char r, unsigned char g, unsigned char b)

 {
     Unshare();
 {
     Unshare();

-
-    M_PENDATA->m_colour = col;
-  
-    RealizeResource();
-}
-
-void wxPen::SetColour(const unsigned char r, const unsigned char g, const unsigned char b)
-{
-    Unshare();
-
+    

     M_PENDATA->m_colour.Set(r, g, b);
     M_PENDATA->m_colour.Set(r, g, b);

-  
+    

     RealizeResource();
 }
 
 void wxPen::SetWidth(int Width)
 {
     Unshare();
     RealizeResource();
 }
 
 void wxPen::SetWidth(int Width)
 {
     Unshare();

-
+    

     M_PENDATA->m_width = Width;
     M_PENDATA->m_width = Width;

-
+    

     RealizeResource();
 }
 
 void wxPen::SetStyle(int Style)
 {
     Unshare();
     RealizeResource();
 }
 
 void wxPen::SetStyle(int Style)
 {
     Unshare();

-
+    

     M_PENDATA->m_style = Style;
     M_PENDATA->m_style = Style;

-
+    

     RealizeResource();
 }
 
 void wxPen::SetStipple(const wxBitmap& Stipple)
 {
     Unshare();
     RealizeResource();
 }
 
 void wxPen::SetStipple(const wxBitmap& Stipple)
 {
     Unshare();

-
+    

     M_PENDATA->m_stipple = Stipple;
     M_PENDATA->m_style = wxSTIPPLE;
     M_PENDATA->m_stipple = Stipple;
     M_PENDATA->m_style = wxSTIPPLE;

-  
+    

     RealizeResource();
 }
 
 void wxPen::SetDashes(int nb_dashes, const wxDash *Dash)
 {
     Unshare();
     RealizeResource();
 }
 
 void wxPen::SetDashes(int nb_dashes, const wxDash *Dash)
 {
     Unshare();

-
+    

     M_PENDATA->m_nbDash = nb_dashes;
     M_PENDATA->m_dash = (wxDash *)Dash;
     M_PENDATA->m_nbDash = nb_dashes;
     M_PENDATA->m_dash = (wxDash *)Dash;

-  
+    

     RealizeResource();
 }
 
 void wxPen::SetJoin(int Join)
 {
     Unshare();
     RealizeResource();
 }
 
 void wxPen::SetJoin(int Join)
 {
     Unshare();

-
+    

     M_PENDATA->m_join = Join;
     M_PENDATA->m_join = Join;

-
+    

     RealizeResource();
 }
 
 void wxPen::SetCap(int Cap)
 {
     Unshare();
     RealizeResource();
 }
 
 void wxPen::SetCap(int Cap)
 {
     Unshare();

-
+    

     M_PENDATA->m_cap = Cap;
     M_PENDATA->m_cap = Cap;

-
+    

     RealizeResource();
 }
 
 bool wxPen::RealizeResource()
 {
     RealizeResource();
 }
 
 bool wxPen::RealizeResource()
 {

-    // TODO: create actual pen
-    return FALSE;
+    // Nothing more to do
+    return TRUE;

 }
 
 }
 

-