applied MIME patch(es) from Chris Elliott

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

diff --git a/src/msw/pen.cpp b/src/msw/pen.cpp
index 53b26ab1e346c448077fbb597fc08e5dba325c13..9b3d3e4f9839e18ac0d62de67a4d3c09ea2a7db8 100644 (file)
--- a/src/msw/pen.cpp
+++ b/src/msw/pen.cpp
@@ -1,12 +1,12 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        pen.cpp
+// Name:        msw/pen.cpp

 // Purpose:     wxPen
 // Author:      Julian Smart
 // Modified by:
 // Created:     04/01/98
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart and Markus Holzem
 // Purpose:     wxPen
 // Author:      Julian Smart
 // Modified by:
 // Created:     04/01/98
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart and Markus Holzem

-// Licence:    wxWindows license
+// Licence:     wxWindows license

 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__


@@ -32,9 +32,7 @@
 #include "wx/msw/private.h"
 #include "assert.h"
 
 #include "wx/msw/private.h"
 #include "assert.h"
 

-#if !USE_SHARED_LIBRARIES

 IMPLEMENT_DYNAMIC_CLASS(wxPen, wxGDIObject)
 IMPLEMENT_DYNAMIC_CLASS(wxPen, wxGDIObject)

-#endif

 
 wxPenRefData::wxPenRefData()
 {
 
 wxPenRefData::wxPenRefData()
 {


@@ -43,7 +41,7 @@ wxPenRefData::wxPenRefData()
   m_join = wxJOIN_ROUND ;
   m_cap = wxCAP_ROUND ;
   m_nbDash = 0 ;
   m_join = wxJOIN_ROUND ;
   m_cap = wxCAP_ROUND ;
   m_nbDash = 0 ;

-  m_dash = 0 ;
+  m_dash = (wxMSWDash*)NULL;

   m_hPen = 0;
 }
 
   m_hPen = 0;
 }
 


@@ -61,8 +59,8 @@ wxPenRefData::wxPenRefData(const wxPenRefData& data)
 
 wxPenRefData::~wxPenRefData()
 {
 
 wxPenRefData::~wxPenRefData()
 {

-       if ( m_hPen )
-               ::DeleteObject((HPEN) m_hPen);
+        if ( m_hPen )
+                ::DeleteObject((HPEN) m_hPen);

 }
 
 // Pens
 }
 
 // Pens


@@ -91,7 +89,7 @@ wxPen::wxPen(const wxColour& col, int Width, int Style)
   M_PENDATA->m_join = wxJOIN_ROUND ;
   M_PENDATA->m_cap = wxCAP_ROUND ;
   M_PENDATA->m_nbDash = 0 ;
   M_PENDATA->m_join = wxJOIN_ROUND ;
   M_PENDATA->m_cap = wxCAP_ROUND ;
   M_PENDATA->m_nbDash = 0 ;

-  M_PENDATA->m_dash = 0 ;
+  M_PENDATA->m_dash = (wxMSWDash*)NULL;

   M_PENDATA->m_hPen = 0 ;
 
 #ifndef __WIN32__
   M_PENDATA->m_hPen = 0 ;
 
 #ifndef __WIN32__


@@ -116,7 +114,7 @@ wxPen::wxPen(const wxColour& col, int Width, int Style)
         (Style == wxUSER_DASH))
       M_PENDATA->m_width = 1;
   }
         (Style == wxUSER_DASH))
       M_PENDATA->m_width = 1;
   }

-#endif 
+#endif

   RealizeResource();
 
   if ( wxThePenList )
   RealizeResource();
 
   if ( wxThePenList )


@@ -134,7 +132,7 @@ wxPen::wxPen(const wxBitmap& stipple, int Width)
     M_PENDATA->m_join = wxJOIN_ROUND ;
     M_PENDATA->m_cap = wxCAP_ROUND ;
     M_PENDATA->m_nbDash = 0 ;
     M_PENDATA->m_join = wxJOIN_ROUND ;
     M_PENDATA->m_cap = wxCAP_ROUND ;
     M_PENDATA->m_nbDash = 0 ;

-    M_PENDATA->m_dash = 0 ;
+    M_PENDATA->m_dash = (wxMSWDash*)NULL;

     M_PENDATA->m_hPen = 0 ;
 
     RealizeResource();
     M_PENDATA->m_hPen = 0 ;
 
     RealizeResource();


@@ -164,18 +162,18 @@ bool wxPen::RealizeResource()
            M_PENDATA->m_cap==wxCAP_ROUND          &&
            M_PENDATA->m_style!=wxUSER_DASH        &&
            M_PENDATA->m_style!=wxSTIPPLE          &&
            M_PENDATA->m_cap==wxCAP_ROUND          &&
            M_PENDATA->m_style!=wxUSER_DASH        &&
            M_PENDATA->m_style!=wxSTIPPLE          &&

-          M_PENDATA->m_width <= 1)
+           M_PENDATA->m_width <= 1)

        {
        {

-          M_PENDATA->m_hPen =
-            (WXHPEN) CreatePen( wx2msPenStyle(M_PENDATA->m_style),
-                                M_PENDATA->m_width,
-                                ms_colour );
+           M_PENDATA->m_hPen =
+             (WXHPEN) CreatePen( wx2msPenStyle(M_PENDATA->m_style),
+                                 M_PENDATA->m_width,
+                                 ms_colour );

        }
        else
        {
        }
        else
        {

-          DWORD ms_style = PS_GEOMETRIC | wx2msPenStyle(M_PENDATA->m_style);
+           DWORD ms_style = PS_GEOMETRIC | wx2msPenStyle(M_PENDATA->m_style);

 
 

-          switch(M_PENDATA->m_join)
+           switch(M_PENDATA->m_join)

            {
                case wxJOIN_BEVEL: ms_style |= PS_JOIN_BEVEL; break;
                case wxJOIN_MITER: ms_style |= PS_JOIN_MITER; break;
            {
                case wxJOIN_BEVEL: ms_style |= PS_JOIN_BEVEL; break;
                case wxJOIN_MITER: ms_style |= PS_JOIN_MITER; break;


@@ -191,9 +189,9 @@ bool wxPen::RealizeResource()
                case wxCAP_ROUND:      ms_style |= PS_ENDCAP_ROUND;   break;
            }
 
                case wxCAP_ROUND:      ms_style |= PS_ENDCAP_ROUND;   break;
            }
 

-          LOGBRUSH logb;
+           LOGBRUSH logb;

 
 

-          switch(M_PENDATA->m_style)
+           switch(M_PENDATA->m_style)

            {
                case wxSTIPPLE:
                    logb.lbStyle = BS_PATTERN ;
            {
                case wxSTIPPLE:
                    logb.lbStyle = BS_PATTERN ;


@@ -203,72 +201,72 @@ bool wxPen::RealizeResource()
                        logb.lbHatch = (LONG)0;
                    break;
                case wxBDIAGONAL_HATCH:
                        logb.lbHatch = (LONG)0;
                    break;
                case wxBDIAGONAL_HATCH:

-                  logb.lbStyle = BS_HATCHED;
-                  logb.lbHatch = HS_BDIAGONAL;
-                  break;
-              case wxCROSSDIAG_HATCH:
-                  logb.lbStyle = BS_HATCHED;
-                  logb.lbHatch = HS_DIAGCROSS;
-                  break;
-              case wxFDIAGONAL_HATCH:
-                  logb.lbStyle = BS_HATCHED;
-                  logb.lbHatch = HS_FDIAGONAL;
-                  break;
-              case wxCROSS_HATCH:
-                  logb.lbStyle = BS_HATCHED;
-                  logb.lbHatch = HS_CROSS;
-                  break;
-              case wxHORIZONTAL_HATCH:
-                  logb.lbStyle = BS_HATCHED;
-                  logb.lbHatch = HS_HORIZONTAL;
-                  break;
-              case wxVERTICAL_HATCH:
-                  logb.lbStyle = BS_HATCHED;
-                  logb.lbHatch = HS_VERTICAL;
-                  break;
-              default:
+                   logb.lbStyle = BS_HATCHED;
+                   logb.lbHatch = HS_BDIAGONAL;
+                   break;
+               case wxCROSSDIAG_HATCH:
+                   logb.lbStyle = BS_HATCHED;
+                   logb.lbHatch = HS_DIAGCROSS;
+                   break;
+               case wxFDIAGONAL_HATCH:
+                   logb.lbStyle = BS_HATCHED;
+                   logb.lbHatch = HS_FDIAGONAL;
+                   break;
+               case wxCROSS_HATCH:
+                   logb.lbStyle = BS_HATCHED;
+                   logb.lbHatch = HS_CROSS;
+                   break;
+               case wxHORIZONTAL_HATCH:
+                   logb.lbStyle = BS_HATCHED;
+                   logb.lbHatch = HS_HORIZONTAL;
+                   break;
+               case wxVERTICAL_HATCH:
+                   logb.lbStyle = BS_HATCHED;
+                   logb.lbHatch = HS_VERTICAL;
+                   break;
+               default:

                    logb.lbStyle = BS_SOLID;
 #ifdef __WXDEBUG__
                    logb.lbStyle = BS_SOLID;
 #ifdef __WXDEBUG__

-                  // this should be unnecessary (it's unused) but suppresses the Purigy
+                   // this should be unnecessary (it's unused) but suppresses the Purigy

                    // messages about uninitialized memory read
                    // messages about uninitialized memory read

-                  logb.lbHatch = 0;
+                   logb.lbHatch = 0;

 #endif
 #endif

-                  break;
-          }
+                   break;
+           }

 
 

-          logb.lbColor = ms_colour;
+           logb.lbColor = ms_colour;

 
 

-          wxDash *real_dash;
+           wxMSWDash *real_dash;

            if (M_PENDATA->m_style==wxUSER_DASH && M_PENDATA->m_nbDash && M_PENDATA->m_dash)
            if (M_PENDATA->m_style==wxUSER_DASH && M_PENDATA->m_nbDash && M_PENDATA->m_dash)

-          {
-              real_dash = new wxDash[M_PENDATA->m_nbDash];
+           {
+               real_dash = new wxMSWDash[M_PENDATA->m_nbDash];

                int i;
                for (i=0; i<M_PENDATA->m_nbDash; i++)
                int i;
                for (i=0; i<M_PENDATA->m_nbDash; i++)

-               real_dash[i] = M_PENDATA->m_dash[i] * M_PENDATA->m_width;
+                   real_dash[i] = M_PENDATA->m_dash[i] * M_PENDATA->m_width;

            }
            }

-          else
+           else

            {
            {

-              real_dash = 0;
+               real_dash = (wxMSWDash*)NULL;

            }
 
            // Win32s doesn't have ExtCreatePen function...
            }
 
            // Win32s doesn't have ExtCreatePen function...

-          if (wxGetOsVersion()==wxWINDOWS_NT || wxGetOsVersion()==wxWIN95)
-          {
-              M_PENDATA->m_hPen =
-                (WXHPEN) ExtCreatePen( ms_style,
-                                       M_PENDATA->m_width,
-                                       &logb,
-                                       M_PENDATA->m_style==wxUSER_DASH
-                                         ? M_PENDATA->m_nbDash
-                                         : 0,
-                                       (const DWORD*)real_dash );
-          }
-          else
+           if (wxGetOsVersion()==wxWINDOWS_NT || wxGetOsVersion()==wxWIN95)

            {
            {

-              M_PENDATA->m_hPen =
-                 (WXHPEN) CreatePen( wx2msPenStyle(M_PENDATA->m_style),
-                                     M_PENDATA->m_width,
-                                     ms_colour );
+               M_PENDATA->m_hPen =
+                 (WXHPEN) ExtCreatePen( ms_style,
+                                        M_PENDATA->m_width,
+                                        &logb,
+                                        M_PENDATA->m_style == wxUSER_DASH
+                                          ? M_PENDATA->m_nbDash
+                                          : 0,
+                                        (LPDWORD)real_dash );
+           }
+           else
+           {
+               M_PENDATA->m_hPen =
+                  (WXHPEN) CreatePen( wx2msPenStyle(M_PENDATA->m_style),
+                                      M_PENDATA->m_width,
+                                      ms_colour );

            }
 
            if (real_dash)
            }
 
            if (real_dash)


@@ -276,9 +274,9 @@ bool wxPen::RealizeResource()
        }
 #else
        M_PENDATA->m_hPen =
        }
 #else
        M_PENDATA->m_hPen =

-        (WXHPEN) CreatePen( wx2msPenStyle(M_PENDATA->m_style),
-                            M_PENDATA->m_width,
-                            ms_colour );
+         (WXHPEN) CreatePen( wx2msPenStyle(M_PENDATA->m_style),
+                             M_PENDATA->m_width,
+                             ms_colour );

 #endif
 #ifdef WXDEBUG_CREATE
        if (M_PENDATA->m_hPen==0)
 #endif
 #ifdef WXDEBUG_CREATE
        if (M_PENDATA->m_hPen==0)


@@ -291,10 +289,10 @@ bool wxPen::RealizeResource()
 
 WXHANDLE wxPen::GetResourceHandle()
 {
 
 WXHANDLE wxPen::GetResourceHandle()
 {

-       if ( !M_PENDATA )
-               return 0;
-       else
-               return (WXHANDLE)M_PENDATA->m_hPen;
+        if ( !M_PENDATA )
+                return 0;
+        else
+                return (WXHANDLE)M_PENDATA->m_hPen;

 }
 
 bool wxPen::FreeResource(bool force)
 }
 
 bool wxPen::FreeResource(bool force)


@@ -315,17 +313,17 @@ bool wxPen::IsFree() const
 
 void wxPen::Unshare()
 {
 
 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)
 }
 
 void wxPen::SetColour(const wxColour& col)


@@ -333,7 +331,7 @@ void wxPen::SetColour(const wxColour& col)
     Unshare();
 
     M_PENDATA->m_colour = col;
     Unshare();
 
     M_PENDATA->m_colour = col;

-  
+

     RealizeResource();
 }
 
     RealizeResource();
 }
 


@@ -342,7 +340,7 @@ void wxPen::SetColour(unsigned char r, unsigned char g, unsigned char b)
     Unshare();
 
     M_PENDATA->m_colour.Set(r, g, b);
     Unshare();
 
     M_PENDATA->m_colour.Set(r, g, b);

-  
+

     RealizeResource();
 }
 
     RealizeResource();
 }
 


@@ -370,7 +368,7 @@ void wxPen::SetStipple(const wxBitmap& Stipple)
 
     M_PENDATA->m_stipple = Stipple;
     M_PENDATA->m_style = wxSTIPPLE;
 
     M_PENDATA->m_stipple = Stipple;
     M_PENDATA->m_style = wxSTIPPLE;

-  
+

     RealizeResource();
 }
 
     RealizeResource();
 }
 


@@ -379,8 +377,8 @@ void wxPen::SetDashes(int nb_dashes, const wxDash *Dash)
     Unshare();
 
     M_PENDATA->m_nbDash = nb_dashes;
     Unshare();
 
     M_PENDATA->m_nbDash = nb_dashes;

-    M_PENDATA->m_dash = (wxDash *)Dash;
-  
+    M_PENDATA->m_dash = (wxMSWDash *)Dash;
+

     RealizeResource();
 }
 
     RealizeResource();
 }
 


@@ -406,30 +404,30 @@ int wx2msPenStyle(int wx_style)
 {
     int cstyle;
     switch (wx_style)
 {
     int cstyle;
     switch (wx_style)

-    { 
+    {

        case wxDOT:
            cstyle = PS_DOT;
        case wxDOT:
            cstyle = PS_DOT;

-          break;
+           break;

 
        case wxDOT_DASH:
 
        case wxDOT_DASH:

-          cstyle = PS_DASHDOT;
-          break;
+           cstyle = PS_DASHDOT;
+           break;

 
        case wxSHORT_DASH:
        case wxLONG_DASH:
            cstyle = PS_DASH;
 
        case wxSHORT_DASH:
        case wxLONG_DASH:
            cstyle = PS_DASH;

-          break;
+           break;

 
        case wxTRANSPARENT:
            cstyle = PS_NULL;
 
        case wxTRANSPARENT:
            cstyle = PS_NULL;

-          break;
+           break;

 
        case wxUSER_DASH:
 #ifdef __WIN32__
            // Win32s doesn't have PS_USERSTYLE
 
        case wxUSER_DASH:
 #ifdef __WIN32__
            // Win32s doesn't have PS_USERSTYLE

-          if (wxGetOsVersion()==wxWINDOWS_NT || wxGetOsVersion()==wxWIN95)
-              cstyle = PS_USERSTYLE;
-          else
+           if (wxGetOsVersion()==wxWINDOWS_NT || wxGetOsVersion()==wxWIN95)
+               cstyle = PS_USERSTYLE;
+           else

                cstyle = PS_DOT; // We must make a choice... This is mine!
 #else
            cstyle = PS_DASH;
                cstyle = PS_DOT; // We must make a choice... This is mine!
 #else
            cstyle = PS_DASH;