code cleanup for the pen width condition (patch 1034942 + more)
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@29476 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
This commit is contained in:
Vadim Zeitlin
299
src/msw/pen.cpp
299
src/msw/pen.cpp
@@ -60,8 +60,8 @@ wxPenRefData::wxPenRefData(const wxPenRefData& data)
|
|||||||
|
|
||||||
wxPenRefData::~wxPenRefData()
|
wxPenRefData::~wxPenRefData()
|
||||||
{
|
{
|
||||||
if ( m_hPen )
|
if ( m_hPen )
|
||||||
::DeleteObject((HPEN) m_hPen);
|
::DeleteObject((HPEN) m_hPen);
|
||||||
}
|
}
|
||||||
|
|
||||||
// Pens
|
// Pens
|
||||||
@@ -89,31 +89,7 @@ wxPen::wxPen(const wxColour& col, int Width, int Style)
|
|||||||
M_PENDATA->m_dash = (wxDash*)NULL;
|
M_PENDATA->m_dash = (wxDash*)NULL;
|
||||||
M_PENDATA->m_hPen = 0 ;
|
M_PENDATA->m_hPen = 0 ;
|
||||||
|
|
||||||
#ifndef __WIN32__
|
|
||||||
// In Windows, only a pen of width = 1 can be dotted or dashed!
|
|
||||||
if ((Style == wxDOT) || (Style == wxLONG_DASH) ||
|
|
||||||
(Style == wxSHORT_DASH) || (Style == wxDOT_DASH) ||
|
|
||||||
(Style == wxUSER_DASH))
|
|
||||||
M_PENDATA->m_width = 1;
|
|
||||||
#else
|
|
||||||
/***
|
|
||||||
DWORD vers = GetVersion() ;
|
|
||||||
WORD high = HIWORD(vers) ; // high bit=0 for NT, 1 for Win32s
|
|
||||||
// Win32s doesn't support wide dashed pens
|
|
||||||
|
|
||||||
if ((high&0x8000)!=0)
|
|
||||||
***/
|
|
||||||
if (wxGetOsVersion()==wxWIN32S)
|
|
||||||
{
|
|
||||||
// In Windows, only a pen of width = 1 can be dotted or dashed!
|
|
||||||
if ((Style == wxDOT) || (Style == wxLONG_DASH) ||
|
|
||||||
(Style == wxSHORT_DASH) || (Style == wxDOT_DASH) ||
|
|
||||||
(Style == wxUSER_DASH))
|
|
||||||
M_PENDATA->m_width = 1;
|
|
||||||
}
|
|
||||||
#endif
|
|
||||||
RealizeResource();
|
RealizeResource();
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
wxPen::wxPen(const wxBitmap& stipple, int Width)
|
wxPen::wxPen(const wxBitmap& stipple, int Width)
|
||||||
@@ -131,152 +107,157 @@ wxPen::wxPen(const wxBitmap& stipple, int Width)
|
|||||||
M_PENDATA->m_hPen = 0 ;
|
M_PENDATA->m_hPen = 0 ;
|
||||||
|
|
||||||
RealizeResource();
|
RealizeResource();
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
bool wxPen::RealizeResource()
|
bool wxPen::RealizeResource()
|
||||||
{
|
{
|
||||||
if (M_PENDATA && (M_PENDATA->m_hPen == 0))
|
if ( !M_PENDATA || M_PENDATA->m_hPen )
|
||||||
|
return false;
|
||||||
|
|
||||||
|
if (M_PENDATA->m_style==wxTRANSPARENT)
|
||||||
{
|
{
|
||||||
if (M_PENDATA->m_style==wxTRANSPARENT)
|
M_PENDATA->m_hPen = (WXHPEN) ::GetStockObject(NULL_PEN);
|
||||||
{
|
return true;
|
||||||
M_PENDATA->m_hPen = (WXHPEN) ::GetStockObject(NULL_PEN);
|
}
|
||||||
return true;
|
|
||||||
}
|
|
||||||
|
|
||||||
COLORREF ms_colour = M_PENDATA->m_colour.GetPixel();
|
static const int os = wxGetOsVersion();
|
||||||
|
COLORREF ms_colour = M_PENDATA->m_colour.GetPixel();
|
||||||
|
|
||||||
// Join style, Cap style, Pen Stippling only on Win32.
|
// Join style, Cap style, Pen Stippling
|
||||||
// Currently no time to find equivalent on Win3.1, sorry
|
#if !defined(__WXMICROWIN__) && !defined(__WXWINCE__)
|
||||||
// [if such equiv exist!!]
|
// Only NT can display dashed or dotted lines with width > 1
|
||||||
#if defined(__WIN32__) && !defined(__WXMICROWIN__) && !defined(__WXWINCE__)
|
if ( os != wxWINDOWS_NT &&
|
||||||
if (M_PENDATA->m_join==wxJOIN_ROUND &&
|
(M_PENDATA->m_style & (wxDOT |
|
||||||
M_PENDATA->m_cap==wxCAP_ROUND &&
|
wxLONG_DASH |
|
||||||
M_PENDATA->m_style!=wxUSER_DASH &&
|
wxSHORT_DASH |
|
||||||
M_PENDATA->m_style!=wxSTIPPLE &&
|
wxDOT_DASH |
|
||||||
M_PENDATA->m_width <= 1)
|
wxUSER_DASH)) &&
|
||||||
{
|
M_PENDATA->m_width > 1 )
|
||||||
M_PENDATA->m_hPen =
|
{
|
||||||
(WXHPEN) CreatePen( wx2msPenStyle(M_PENDATA->m_style),
|
M_PENDATA->m_width = 1;
|
||||||
M_PENDATA->m_width,
|
}
|
||||||
ms_colour );
|
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
DWORD ms_style = PS_GEOMETRIC | wx2msPenStyle(M_PENDATA->m_style);
|
|
||||||
|
|
||||||
switch(M_PENDATA->m_join)
|
if (M_PENDATA->m_join==wxJOIN_ROUND &&
|
||||||
{
|
M_PENDATA->m_cap==wxCAP_ROUND &&
|
||||||
case wxJOIN_BEVEL: ms_style |= PS_JOIN_BEVEL; break;
|
M_PENDATA->m_style!=wxUSER_DASH &&
|
||||||
case wxJOIN_MITER: ms_style |= PS_JOIN_MITER; break;
|
M_PENDATA->m_style!=wxSTIPPLE &&
|
||||||
default:
|
M_PENDATA->m_width <= 1)
|
||||||
case wxJOIN_ROUND: ms_style |= PS_JOIN_ROUND; break;
|
{
|
||||||
}
|
|
||||||
|
|
||||||
switch(M_PENDATA->m_cap)
|
|
||||||
{
|
|
||||||
case wxCAP_PROJECTING: ms_style |= PS_ENDCAP_SQUARE; break;
|
|
||||||
case wxCAP_BUTT: ms_style |= PS_ENDCAP_FLAT; break;
|
|
||||||
default:
|
|
||||||
case wxCAP_ROUND: ms_style |= PS_ENDCAP_ROUND; break;
|
|
||||||
}
|
|
||||||
|
|
||||||
LOGBRUSH logb;
|
|
||||||
|
|
||||||
switch(M_PENDATA->m_style)
|
|
||||||
{
|
|
||||||
case wxSTIPPLE:
|
|
||||||
logb.lbStyle = BS_PATTERN ;
|
|
||||||
if (M_PENDATA->m_stipple.Ok())
|
|
||||||
logb.lbHatch = (LONG)M_PENDATA->m_stipple.GetHBITMAP();
|
|
||||||
else
|
|
||||||
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_SOLID;
|
|
||||||
#ifdef __WXDEBUG__
|
|
||||||
// this should be unnecessary (it's unused) but suppresses the Purify
|
|
||||||
// messages about uninitialized memory read
|
|
||||||
logb.lbHatch = 0;
|
|
||||||
#endif
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
|
|
||||||
logb.lbColor = ms_colour;
|
|
||||||
|
|
||||||
wxMSWDash *real_dash;
|
|
||||||
if (M_PENDATA->m_style==wxUSER_DASH && M_PENDATA->m_nbDash && M_PENDATA->m_dash)
|
|
||||||
{
|
|
||||||
real_dash = new wxMSWDash[M_PENDATA->m_nbDash];
|
|
||||||
int rw = M_PENDATA->m_width > 1 ? M_PENDATA->m_width : 1;
|
|
||||||
for ( int i = 0; i < M_PENDATA->m_nbDash; i++ )
|
|
||||||
real_dash[i] = M_PENDATA->m_dash[i] * rw;
|
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
real_dash = (wxMSWDash*)NULL;
|
|
||||||
}
|
|
||||||
|
|
||||||
// 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,
|
|
||||||
(LPDWORD)real_dash );
|
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
M_PENDATA->m_hPen =
|
|
||||||
(WXHPEN) CreatePen( wx2msPenStyle(M_PENDATA->m_style),
|
|
||||||
M_PENDATA->m_width,
|
|
||||||
ms_colour );
|
|
||||||
}
|
|
||||||
|
|
||||||
if (real_dash)
|
|
||||||
delete [] real_dash;
|
|
||||||
}
|
|
||||||
#else
|
|
||||||
M_PENDATA->m_hPen =
|
M_PENDATA->m_hPen =
|
||||||
(WXHPEN) CreatePen( wx2msPenStyle(M_PENDATA->m_style),
|
(WXHPEN) CreatePen( wx2msPenStyle(M_PENDATA->m_style),
|
||||||
M_PENDATA->m_width,
|
M_PENDATA->m_width,
|
||||||
ms_colour );
|
ms_colour );
|
||||||
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
DWORD ms_style = PS_GEOMETRIC | wx2msPenStyle(M_PENDATA->m_style);
|
||||||
|
|
||||||
|
switch(M_PENDATA->m_join)
|
||||||
|
{
|
||||||
|
case wxJOIN_BEVEL: ms_style |= PS_JOIN_BEVEL; break;
|
||||||
|
case wxJOIN_MITER: ms_style |= PS_JOIN_MITER; break;
|
||||||
|
default:
|
||||||
|
case wxJOIN_ROUND: ms_style |= PS_JOIN_ROUND; break;
|
||||||
|
}
|
||||||
|
|
||||||
|
switch(M_PENDATA->m_cap)
|
||||||
|
{
|
||||||
|
case wxCAP_PROJECTING: ms_style |= PS_ENDCAP_SQUARE; break;
|
||||||
|
case wxCAP_BUTT: ms_style |= PS_ENDCAP_FLAT; break;
|
||||||
|
default:
|
||||||
|
case wxCAP_ROUND: ms_style |= PS_ENDCAP_ROUND; break;
|
||||||
|
}
|
||||||
|
|
||||||
|
LOGBRUSH logb;
|
||||||
|
|
||||||
|
switch(M_PENDATA->m_style)
|
||||||
|
{
|
||||||
|
case wxSTIPPLE:
|
||||||
|
logb.lbStyle = BS_PATTERN ;
|
||||||
|
if (M_PENDATA->m_stipple.Ok())
|
||||||
|
logb.lbHatch = (LONG)M_PENDATA->m_stipple.GetHBITMAP();
|
||||||
|
else
|
||||||
|
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_SOLID;
|
||||||
|
#ifdef __WXDEBUG__
|
||||||
|
// this should be unnecessary (it's unused) but suppresses the Purify
|
||||||
|
// messages about uninitialized memory read
|
||||||
|
logb.lbHatch = 0;
|
||||||
#endif
|
#endif
|
||||||
#ifdef WXDEBUG_CREATE
|
break;
|
||||||
if (M_PENDATA->m_hPen==0)
|
}
|
||||||
wxError("Cannot create pen","Internal error") ;
|
|
||||||
#endif
|
logb.lbColor = ms_colour;
|
||||||
return true;
|
|
||||||
}
|
wxMSWDash *real_dash;
|
||||||
return false;
|
if (M_PENDATA->m_style==wxUSER_DASH && M_PENDATA->m_nbDash && M_PENDATA->m_dash)
|
||||||
|
{
|
||||||
|
real_dash = new wxMSWDash[M_PENDATA->m_nbDash];
|
||||||
|
int rw = M_PENDATA->m_width > 1 ? M_PENDATA->m_width : 1;
|
||||||
|
for ( int i = 0; i < M_PENDATA->m_nbDash; i++ )
|
||||||
|
real_dash[i] = M_PENDATA->m_dash[i] * rw;
|
||||||
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
real_dash = (wxMSWDash*)NULL;
|
||||||
|
}
|
||||||
|
|
||||||
|
// Win32s doesn't have ExtCreatePen function...
|
||||||
|
if (os==wxWINDOWS_NT || os==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,
|
||||||
|
(LPDWORD)real_dash );
|
||||||
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
M_PENDATA->m_hPen =
|
||||||
|
(WXHPEN) CreatePen( wx2msPenStyle(M_PENDATA->m_style),
|
||||||
|
M_PENDATA->m_width,
|
||||||
|
ms_colour );
|
||||||
|
}
|
||||||
|
|
||||||
|
delete [] real_dash;
|
||||||
|
}
|
||||||
|
#else // WinCE
|
||||||
|
M_PENDATA->m_hPen =
|
||||||
|
(WXHPEN) CreatePen( wx2msPenStyle(M_PENDATA->m_style),
|
||||||
|
M_PENDATA->m_width,
|
||||||
|
ms_colour );
|
||||||
|
#endif // !WinCE/WinCE
|
||||||
|
|
||||||
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
WXHANDLE wxPen::GetResourceHandle() const
|
WXHANDLE wxPen::GetResourceHandle() const
|
||||||
|
|||||||
Reference in New Issue
Block a user