X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/2f1ae4143271ae63a17e052a1a471d16e9cd8c44..622be94c78deeae8490d9783c11a24c599ae1464:/src/mac/dc.cpp?ds=inline diff --git a/src/mac/dc.cpp b/src/mac/dc.cpp index 1682209492..27d92a4efc 100644 --- a/src/mac/dc.cpp +++ b/src/mac/dc.cpp @@ -14,7 +14,15 @@ #endif #include "wx/dc.h" +#include "wx/app.h" #include "wx/mac/uma.h" +#include "wx/dcmemory.h" +#include "wx/region.h" +#include "wx/image.h" + +#if __MSL__ >= 0x6000 +#include "math.h" +#endif #if !USE_SHARED_LIBRARY IMPLEMENT_ABSTRACT_CLASS(wxDC, wxObject) @@ -30,8 +38,18 @@ IMPLEMENT_ABSTRACT_CLASS(wxDC, wxObject) #define twips2mm 0.0176388888889 #define mm2pt 2.83464566929 #define pt2mm 0.352777777778 +#ifndef __UNIX__ +const double M_PI = 3.14159265358979 ; +#endif +const double RAD2DEG = 180.0 / M_PI; + +//----------------------------------------------------------------------------- +// Local functions +//----------------------------------------------------------------------------- -long wxDC::m_macCurrentPortId = 1 ; +static inline double dmin(double a, double b) { return a < b ? a : b; } +static inline double dmax(double a, double b) { return a > b ? a : b; } +static inline double DegToRad(double deg) { return (deg * M_PI) / 180.0; } //----------------------------------------------------------------------------- // wxDC @@ -40,18 +58,11 @@ long wxDC::m_macCurrentPortId = 1 ; wxDC::wxDC() { m_ok = FALSE; - m_optimize = FALSE; - m_autoSetting = FALSE; m_colour = TRUE; - m_clipping = FALSE; m_mm_to_pix_x = mm2pt; m_mm_to_pix_y = mm2pt; - m_logicalOriginX = 0; - m_logicalOriginY = 0; - m_deviceOriginX = 0; - m_deviceOriginY = 0; m_internalDeviceOriginX = 0; m_internalDeviceOriginY = 0; m_externalDeviceOriginX = 0; @@ -64,28 +75,12 @@ wxDC::wxDC() m_scaleX = 1.0; m_scaleY = 1.0; - m_mappingMode = wxMM_TEXT; m_needComputeScaleX = FALSE; m_needComputeScaleY = FALSE; - m_signX = 1; // default x-axis left to right - m_signY = 1; // default y-axis top down - m_maxX = m_maxY = -100000; m_minY = m_minY = 100000; - m_logicalFunction = wxCOPY; -// m_textAlignment = wxALIGN_TOP_LEFT; - m_backgroundMode = wxTRANSPARENT; - - m_textForegroundColour = *wxBLACK; - m_textBackgroundColour = *wxWHITE; - m_pen = *wxBLACK_PEN; - m_font = *wxNORMAL_FONT; - m_brush = *wxTRANSPARENT_BRUSH; - m_backgroundBrush = *wxWHITE_BRUSH; - -// m_palette = wxAPP_COLOURMAP; m_macPort = NULL ; m_macMask = NULL ; m_ok = FALSE ; @@ -94,45 +89,47 @@ wxDC::wxDC() m_macBrushInstalled = false ; m_macPenInstalled = false ; - m_macPortId = 0 ; m_macLocalOrigin.h = m_macLocalOrigin.v = 0 ; m_macClipRect.left = -32000 ; m_macClipRect.top = -32000 ; m_macClipRect.right = 32000 ; m_macClipRect.bottom = 32000 ; - ::GetPort( &m_macOrigPort ) ; -}; + + m_pen = *wxBLACK_PEN; + m_font = *wxNORMAL_FONT; + m_brush = *wxWHITE_BRUSH; +} +wxMacPortSetter::wxMacPortSetter( const wxDC* dc ) : + m_ph( dc->m_macPort ) +{ + wxASSERT( dc->Ok() ) ; + + dc->MacSetupPort(&m_ph) ; +} + +wxMacPortSetter::~wxMacPortSetter() +{ +} wxDC::~wxDC(void) { - if ( !m_macPortHelper.IsCleared() ) +} +void wxDC::MacSetupPort(AGAPortHelper* help) const +{ +// help->Setup( m_macPort ) ; + ::SetOrigin(-m_macLocalOrigin.h, -m_macLocalOrigin.v); + + if ( m_clipping ) { - GrafPtr port ; - GetPort( &port ) ; - SetPort( m_macPortHelper.GetCurrentPort() ) ; - SetOrigin( 0 , 0 ) ; - SetPort( port ) ; + Rect clip = { m_clipY1 , m_clipX1 , m_clipY2 , m_clipX2 } ; + ::SectRect( &clip , &m_macClipRect , &clip ) ; + ::ClipRect( &clip ) ; } - /* - if ( m_macPort ) + else { - ::SetPort( m_macPort ) ; - ::SetOrigin( 0 , 0 ) ; - ::ClipRect( &m_macPort->portRect ) ; - ::PenNormal() ; - ::SetPort( m_macOrigPort ) ; + ::ClipRect(&m_macClipRect); } - */ - ++m_macCurrentPortId ; -}; -void wxDC::MacSetupPort() const -{ - AGAPortHelper* help = &m_macPortHelper ; - help->Setup( m_macPort ) ; - m_macPortId = ++m_macCurrentPortId ; - ::SetOrigin(-m_macLocalOrigin.h, -m_macLocalOrigin.v); - ::ClipRect(&m_macClipRect); m_macFontInstalled = false ; m_macBrushInstalled = false ; @@ -141,132 +138,141 @@ void wxDC::MacSetupPort() const void wxDC::DoDrawBitmap( const wxBitmap &bmp, wxCoord x, wxCoord y, bool useMask ) { - float scale = 1.0 ; - - if (!Ok()) - return; - MacVerifySetup() ; - - long xx1 = XLOG2DEV(x); - long yy1 = YLOG2DEV(y); - - { - wxBitmapRefData * bmap = (wxBitmapRefData*) ( bmp.GetRefData()) ; - - if ( bmap ) - { - if ( bmap->m_bitmapType == kMacBitmapTypePict ) - { - Rect bitmaprect = { 0 , 0 , bmap->m_height * scale , bmap->m_width * scale} ; - ::OffsetRect( &bitmaprect , xx1 , yy1 ) ; - ::DrawPicture( bmap->m_hPict , &bitmaprect ) ; - } - else if ( bmap->m_bitmapType == kMacBitmapTypeGrafWorld ) - { - if ( bmap->m_hBitmap ) - { - GWorldPtr bmapworld = bmap->m_hBitmap ; - PixMapHandle bmappixels ; - RGBColor white = { 0xFFFF, 0xFFFF,0xFFFF} ; - RGBColor black = { 0,0,0} ; - RGBForeColor( &black ) ; - RGBBackColor( &white ) ; - - bmappixels = GetGWorldPixMap( bmapworld ) ; - if ( LockPixels(bmappixels) ) - { - Rect source , dest ; - source.top = 0 ; - source.left = 0 ; - source.right = bmap->m_width ; - source.bottom = bmap->m_height ; - dest.top = YLOG2DEV(y) ; - dest.left = XLOG2DEV(x) ; - dest.bottom = YLOG2DEV(y + bmap->m_height * scale) ; - dest.right = XLOG2DEV(x + bmap->m_width * scale ) ; - - if ( useMask && bmp.GetMask() ) - { - if ( LockPixels( GetGWorldPixMap( bmp.GetMask()->GetMaskBitmap( ) ) ) ) - { - CopyMask( GetPortBitMapForCopyBits( bmapworld ) , GetPortBitMapForCopyBits( bmp.GetMask()->GetMaskBitmap( ) ) , - GetPortBitMapForCopyBits( m_macPort ) , - &source, &source , &dest ) ; - UnlockPixels( GetGWorldPixMap( bmp.GetMask()->GetMaskBitmap( ) ) ) ; - } - } - else - CopyBits( GetPortBitMapForCopyBits( bmapworld ) , GetPortBitMapForCopyBits( m_macPort ), - &source, &dest, srcCopy, NULL ) ; - - UnlockPixels( bmappixels ) ; - } - m_macPenInstalled = false ; - m_macBrushInstalled = false ; - m_macFontInstalled = false ; - } - } + wxCHECK_RET( Ok(), wxT("invalid window dc") ); + + wxCHECK_RET( bmp.Ok(), wxT("invalid bitmap") ); + + wxMacPortSetter helper(this) ; + + wxCoord xx = XLOG2DEV(x); + wxCoord yy = YLOG2DEV(y); + wxCoord w = bmp.GetWidth(); + wxCoord h = bmp.GetHeight(); + wxCoord ww = XLOG2DEVREL(w); + wxCoord hh = YLOG2DEVREL(h); + + // Set up drawing mode + short mode = (m_logicalFunction == wxCOPY ? srcCopy : + //m_logicalFunction == wxCLEAR ? WHITENESS : + //m_logicalFunction == wxSET ? BLACKNESS : + m_logicalFunction == wxINVERT ? hilite : + //m_logicalFunction == wxAND ? MERGECOPY : + m_logicalFunction == wxOR ? srcOr : + m_logicalFunction == wxSRC_INVERT ? notSrcCopy : + m_logicalFunction == wxXOR ? srcXor : + m_logicalFunction == wxOR_REVERSE ? notSrcOr : + //m_logicalFunction == wxAND_REVERSE ? SRCERASE : + //m_logicalFunction == wxSRC_OR ? srcOr : + //m_logicalFunction == wxSRC_AND ? SRCAND : + srcCopy ); + + if ( bmp.GetBitmapType() == kMacBitmapTypePict ) { + Rect bitmaprect = { 0 , 0 , hh, ww }; + ::OffsetRect( &bitmaprect, xx, yy ) ; + ::DrawPicture( bmp.GetPict(), &bitmaprect ) ; } - } + else if ( bmp.GetBitmapType() == kMacBitmapTypeGrafWorld ) + { + GWorldPtr bmapworld = bmp.GetHBITMAP(); + PixMapHandle bmappixels ; + + // Set foreground and background colours (for bitmaps depth = 1) + if(bmp.GetDepth() == 1) + { + RGBColor fore = m_textForegroundColour.GetPixel(); + RGBColor back = m_textBackgroundColour.GetPixel(); + RGBForeColor(&fore); + RGBBackColor(&back); + } + else + { + RGBColor white = { 0xFFFF, 0xFFFF,0xFFFF} ; + RGBColor black = { 0,0,0} ; + RGBForeColor( &black ) ; + RGBBackColor( &white ) ; + } + + bmappixels = GetGWorldPixMap( bmapworld ) ; + + wxCHECK_RET(LockPixels(bmappixels), + wxT("DoDrawBitmap: Unable to lock pixels")); + + Rect source = { 0, 0, h, w }; + Rect dest = { yy, xx, yy + hh, xx + ww }; + + if ( useMask && bmp.GetMask() ) + { + if( LockPixels(GetGWorldPixMap(bmp.GetMask()->GetMaskBitmap()))) + { + CopyDeepMask + ( + GetPortBitMapForCopyBits(bmapworld), + GetPortBitMapForCopyBits(bmp.GetMask()->GetMaskBitmap()), + GetPortBitMapForCopyBits( m_macPort ), + &source, &source, &dest, mode, NULL + ); + UnlockPixels(GetGWorldPixMap(bmp.GetMask()->GetMaskBitmap())); + } + } + else { + CopyBits( GetPortBitMapForCopyBits( bmapworld ), + GetPortBitMapForCopyBits( m_macPort ), + &source, &dest, mode, NULL ) ; + } + UnlockPixels( bmappixels ) ; + } + else if ( bmp.GetBitmapType() == kMacBitmapTypeIcon ) + { + Rect bitmaprect = { 0 , 0 , bmp.GetHeight(), bmp.GetWidth() } ; + OffsetRect( &bitmaprect, xx, yy ) ; + PlotCIconHandle( &bitmaprect , atNone , ttNone , bmp.GetHICON() ) ; + } + m_macPenInstalled = false ; + m_macBrushInstalled = false ; + m_macFontInstalled = false ; + } void wxDC::DoDrawIcon( const wxIcon &icon, wxCoord x, wxCoord y ) { - if (!Ok()) - return; - MacVerifySetup() ; - - long xx1 = XLOG2DEV(x); - long yy1 = YLOG2DEV(y); - - { - wxIconRefData * iconref = (wxIconRefData*) ( icon.GetRefData()) ; - - if ( iconref && iconref->m_ok && iconref->m_hIcon ) - { - Rect bitmaprect = { 0 , 0 , iconref->m_height , iconref->m_width } ; - OffsetRect( &bitmaprect , xx1 , yy1 ) ; - PlotCIconHandle( &bitmaprect , atNone , ttNone , iconref->m_hIcon ) ; - } - } -}; - + wxCHECK_RET(Ok(), wxT("Invalid dc wxDC::DoDrawIcon")); + + wxCHECK_RET(icon.Ok(), wxT("Invalid icon wxDC::DoDrawIcon")); + + DoDrawBitmap( icon , x , y , icon.GetMask() != NULL ) ; +} void wxDC::DoSetClippingRegion( wxCoord x, wxCoord y, wxCoord width, wxCoord height ) { - MacVerifySetup() ; - if( m_clipping ) - { - m_clipX1 = wxMax( m_clipX1 , x ) ; - m_clipY1 = wxMax( m_clipY1 ,y ); - m_clipX2 = wxMin( m_clipX2, (x + width)); - m_clipY2 = wxMin( m_clipY2,(y + height)); - - } - else - { - m_clipping = TRUE; - m_clipX1 = x; - m_clipY1 = y; - m_clipX2 = x + width; - m_clipY2 = y + height; - } - - long x1 = XLOG2DEV(m_clipX1); - long y1 = YLOG2DEV(m_clipY1); - long x2 = XLOG2DEV(m_clipX2); - long y2 = YLOG2DEV(m_clipY2); - - Rect clip = { y1 , x1 , y2 , x2 } ; - - ::ClipRect( &clip ) ; + wxCHECK_RET(Ok(), wxT("wxDC::DoSetClippingRegion Invalid DC")); + wxCoord xx, yy, ww, hh; + + xx = XLOG2DEV(x); + yy = YLOG2DEV(y); + ww = XLOG2DEVREL(width); + hh = YLOG2DEVREL(height); -}; + if( m_clipping ) + { + m_clipX1 = wxMax( m_clipX1 , xx ); + m_clipY1 = wxMax( m_clipY1 , yy ); + m_clipX2 = wxMin( m_clipX2, (xx + ww)); + m_clipY2 = wxMin( m_clipY2, (yy + hh)); + } + else + { + m_clipping = TRUE; + m_clipX1 = xx; + m_clipY1 = yy; + m_clipX2 = xx + ww; + m_clipY2 = yy + hh; + } +} void wxDC::DoSetClippingRegionAsRegion( const wxRegion ®ion ) { - wxCHECK_RET( Ok(), wxT("invalid window dc") ); + wxCHECK_RET( Ok(), wxT("invalid window dc") ) ; - MacVerifySetup() ; + wxMacPortSetter helper(this) ; if (region.Empty()) { DestroyClippingRegion(); @@ -278,20 +284,16 @@ void wxDC::DoSetClippingRegionAsRegion( const wxRegion ®ion ) wxDC::DoSetClippingRegion( xx, yy, ww, hh ); } -void wxDC::DestroyClippingRegion(void) +void wxDC::DestroyClippingRegion() { - MacVerifySetup() ; + wxMacPortSetter helper(this) ; m_clipping = FALSE; -// Rect clip = { -32000 , -32000 , 32000 , 32000 } ; - ::ClipRect(&m_macClipRect); -}; - +} void wxDC::DoGetSize( int* width, int* height ) const { *width = m_maxX-m_minX; *height = m_maxY-m_minY; -}; - +} void wxDC::DoGetSizeMM( int* width, int* height ) const { int w = 0; @@ -299,22 +301,19 @@ void wxDC::DoGetSizeMM( int* width, int* height ) const GetSize( &w, &h ); *width = long( double(w) / (m_scaleX*m_mm_to_pix_x) ); *height = long( double(h) / (m_scaleY*m_mm_to_pix_y) ); -}; - +} void wxDC::SetTextForeground( const wxColour &col ) { - if (!Ok()) return; - m_textForegroundColour = col; - m_macFontInstalled = false ; -}; - + wxCHECK_RET(Ok(), wxT("Invalid DC")); + m_textForegroundColour = col; + m_macFontInstalled = false ; +} void wxDC::SetTextBackground( const wxColour &col ) { - if (!Ok()) return; - m_textBackgroundColour = col; - m_macFontInstalled = false ; -}; - + wxCHECK_RET(Ok(), wxT("Invalid DC")); + m_textBackgroundColour = col; + m_macFontInstalled = false ; +} void wxDC::SetMapMode( int mode ) { switch (mode) @@ -335,64 +334,59 @@ void wxDC::SetMapMode( int mode ) case wxMM_TEXT: SetLogicalScale( 1.0, 1.0 ); break; - }; + } if (mode != wxMM_TEXT) { m_needComputeScaleX = TRUE; m_needComputeScaleY = TRUE; - }; -}; - + } +} void wxDC::SetUserScale( double x, double y ) { // allow negative ? -> no m_userScaleX = x; m_userScaleY = y; ComputeScaleAndOrigin(); -}; - +} void wxDC::SetLogicalScale( double x, double y ) { // allow negative ? m_logicalScaleX = x; m_logicalScaleY = y; ComputeScaleAndOrigin(); -}; - +} void wxDC::SetLogicalOrigin( wxCoord x, wxCoord y ) { m_logicalOriginX = x * m_signX; // is this still correct ? m_logicalOriginY = y * m_signY; ComputeScaleAndOrigin(); -}; - +} void wxDC::SetDeviceOrigin( wxCoord x, wxCoord y ) { m_externalDeviceOriginX = x; m_externalDeviceOriginY = y; ComputeScaleAndOrigin(); -}; -/* +} + +#if 0 void wxDC::SetInternalDeviceOrigin( long x, long y ) { m_internalDeviceOriginX = x; m_internalDeviceOriginY = y; ComputeScaleAndOrigin(); -}; - +} void wxDC::GetInternalDeviceOrigin( long *x, long *y ) { if (x) *x = m_internalDeviceOriginX; if (y) *y = m_internalDeviceOriginY; -}; -*/ +} +#endif void wxDC::SetAxisOrientation( bool xLeftRight, bool yBottomUp ) { m_signX = (xLeftRight ? 1 : -1); m_signY = (yBottomUp ? -1 : 1); ComputeScaleAndOrigin(); -}; -/* +}/* void wxDC::CalcBoundingBox( long x, long y ) { @@ -400,8 +394,7 @@ void wxDC::CalcBoundingBox( long x, long y ) if (y < m_minY) m_minY = y; if (x > m_maxX) m_maxX = x; if (y > m_maxY) m_maxY = y; -}; -*/ +}*/ wxSize wxDC::GetPPI() const { return wxSize(72, 72); @@ -412,7 +405,7 @@ int wxDC::GetDepth() const return wxDisplayDepth() ; } -void wxDC::ComputeScaleAndOrigin(void) +void wxDC::ComputeScaleAndOrigin() { // CMB: copy scale to see if it changes double origScaleX = m_scaleX; @@ -427,10 +420,14 @@ void wxDC::ComputeScaleAndOrigin(void) // CMB: if scale has changed call SetPen to recalulate the line width if (m_scaleX != origScaleX || m_scaleY != origScaleY) { - // TODO : set internal flags for recalc + // this is a bit artificial, but we need to force wxDC to think + // the pen has changed + wxPen* pen = & GetPen(); + wxPen tempPen; + m_pen = tempPen; + SetPen(* pen); } -}; - +} void wxDC::SetPalette( const wxPalette& palette ) { } @@ -442,38 +439,27 @@ void wxDC::SetBackgroundMode( int mode ) void wxDC::SetFont( const wxFont &font ) { - if (!Ok()) - return; - - MacVerifySetup() ; - - m_font = font; - m_macFontInstalled = false ; + wxCHECK_RET(Ok(), wxT("Invalid DC")); + + m_font = font; + m_macFontInstalled = false ; } void wxDC::SetPen( const wxPen &pen ) { - if (!Ok() ) - return; - - MacVerifySetup() ; + wxCHECK_RET(Ok(), wxT("Invalid DC")); if ( m_pen == pen ) return ; m_pen = pen; -/* - if (!m_pen.Ok()) - return; -*/ + m_macPenInstalled = false ; } void wxDC::SetBrush( const wxBrush &brush ) { - if (!Ok() ) - return; - MacVerifySetup() ; + wxCHECK_RET(Ok(), wxT("Invalid DC")); if (m_brush == brush) return; @@ -484,9 +470,7 @@ void wxDC::SetBrush( const wxBrush &brush ) void wxDC::SetBackground( const wxBrush &brush ) { - if (!Ok()) - return; - MacVerifySetup() ; + wxCHECK_RET(Ok(), wxT("Invalid DC")); if (m_backgroundBrush == brush) return; @@ -516,86 +500,248 @@ void wxDC::DoFloodFill( wxCoord x, wxCoord y, const wxColour& col, bool wxDC::DoGetPixel( wxCoord x, wxCoord y, wxColour *col ) const { - return true ; + wxCHECK_MSG( Ok(), false, wxT("wxDC::DoGetPixel Invalid DC") ); + wxMacPortSetter helper(this) ; + + RGBColor colour; + + GetCPixel( XLOG2DEV(x), YLOG2DEV(y), &colour ); + + // Convert from Mac colour to wx + col->Set( colour.red >> 8, + colour.green >> 8, + colour.blue >> 8); + + return true ; } void wxDC::DoDrawLine( wxCoord x1, wxCoord y1, wxCoord x2, wxCoord y2 ) { - if (!Ok()) - return; + wxCHECK_RET(Ok(), wxT("Invalid DC")); - MacVerifySetup() ; + wxMacPortSetter helper(this) ; if (m_pen.GetStyle() != wxTRANSPARENT) { MacInstallPen() ; - int offset = (m_pen.GetWidth() - 1) / 2 ; - long xx1 = XLOG2DEV(x1); - long yy1 = YLOG2DEV(y1); - long xx2 = XLOG2DEV(x2); - long yy2 = YLOG2DEV(y2); - - ::MoveTo(xx1 - offset ,yy1 - offset); - ::LineTo(xx2 - offset , yy2 - offset ); - }; + wxCoord offset = ( (m_pen.GetWidth() == 0 ? 1 : + m_pen.GetWidth() ) * (wxCoord)m_scaleX - 1) / 2; + + wxCoord xx1 = XLOG2DEV(x1) - offset; + wxCoord yy1 = YLOG2DEV(y1) - offset; + wxCoord xx2 = XLOG2DEV(x2) - offset; + wxCoord yy2 = YLOG2DEV(y2) - offset; + + if ((m_pen.GetCap() == wxCAP_ROUND) && + (m_pen.GetWidth() <= 1)) + { + // Implement LAST_NOT for MAC at least for + // orthogonal lines. RR. + if (xx1 == xx2) + { + if (yy1 < yy2) + yy2--; + if (yy1 > yy2) + yy2++; + } + if (yy1 == yy2) + { + if (xx1 < xx2) + xx2--; + if (xx1 > xx2) + xx2++; + } + } + + ::MoveTo(xx1, yy1); + ::LineTo(xx2, yy2); + } } void wxDC::DoCrossHair( wxCoord x, wxCoord y ) { + wxCHECK_RET( Ok(), wxT("wxDC::DoCrossHair Invalid window dc") ); + + if (m_pen.GetStyle() != wxTRANSPARENT) + { + int w = 0; + int h = 0; + GetSize( &w, &h ); + wxCoord xx = XLOG2DEV(x); + wxCoord yy = YLOG2DEV(y); + + MacInstallPen(); + ::MoveTo( 0, yy ); + ::LineTo( XLOG2DEVREL(w), yy ); + ::MoveTo( xx, 0 ); + ::LineTo( xx, YLOG2DEVREL(h) ); + } +} + +/* + * To draw arcs properly the angles need to be converted from the WX style: + * Angles start on the +ve X axis and go anti-clockwise (As you would draw on + * a normal axis on paper). + * TO + * the Mac style: + * Angles start on the +ve y axis and go clockwise. + * To achive this I work out which quadrant the angle lies in then map this to + * the equivalent quadrant on the Mac. (Sin and Cos values reveal which + * quadrant you are in). + */ +static double wxConvertWXangleToMACangle(double angle) +{ + double sin_a, cos_a; + + sin_a = sin(angle / RAD2DEG); + cos_a = cos(angle / RAD2DEG); + + if( (sin_a >= 0.0) && (cos_a >= 0.0) ) { + angle = acos(sin_a) * RAD2DEG; + } + else if( (sin_a >= 0.0) && (cos_a <= 0.0) ) { + sin_a *= -1; + angle = acos(sin_a) * RAD2DEG + 180; + } + else if( (sin_a <= 0.0) && (cos_a >= 0.0) ) { + angle = acos(sin_a) * RAD2DEG + 180; + } + else if( (sin_a < 0.0) && (cos_a < 0.0) ) { + sin_a *= -1; + angle = acos(sin_a) * RAD2DEG + 180; + } + return angle; } void wxDC::DoDrawArc( wxCoord x1, wxCoord y1, wxCoord x2, wxCoord y2, wxCoord xc, wxCoord yc ) { + wxCHECK_RET(Ok(), wxT("wxDC::DoDrawArc Invalid DC")); + + wxCoord xx1 = XLOG2DEV(x1); + wxCoord yy1 = YLOG2DEV(y1); + wxCoord xx2 = XLOG2DEV(x2); + wxCoord yy2 = YLOG2DEV(y2); + wxCoord xxc = XLOG2DEV(xc); + wxCoord yyc = YLOG2DEV(yc); + double dx = xx1 - xxc; + double dy = yy1 - yyc; + double radius = sqrt((double)(dx*dx+dy*dy)); + wxCoord rad = (wxCoord)radius; + double radius1, radius2; + + if (xx1 == xx2 && yy1 == yy2) + { + radius1 = 0.0; + radius2 = 360.0; + } + else if (radius == 0.0) + { + radius1 = radius2 = 0.0; + } + else + { + radius1 = (xx1 - xxc == 0) ? + (yy1 - yyc < 0) ? 90.0 : -90.0 : + -atan2(double(yy1-yyc), double(xx1-xxc)) * RAD2DEG; + radius2 = (xx2 - xxc == 0) ? + (yy2 - yyc < 0) ? 90.0 : -90.0 : + -atan2(double(yy2-yyc), double(xx2-xxc)) * RAD2DEG; + } + wxCoord alpha2 = wxCoord(radius2 - radius1); + wxCoord alpha1 = wxCoord(wxConvertWXangleToMACangle(radius1)); + if( (xx1 > xx2) || (yy1 > yy2) ) { + alpha2 *= -1; + } + + Rect r = { yyc - rad, xxc - rad, yyc + rad, xxc + rad }; + + if(m_brush.GetStyle() != wxTRANSPARENT) { + MacInstallBrush(); + PaintArc(&r, alpha1, alpha2); + } + if(m_pen.GetStyle() != wxTRANSPARENT) { + MacInstallPen(); + FrameArc(&r, alpha1, alpha2); + } } void wxDC::DoDrawEllipticArc( wxCoord x, wxCoord y, wxCoord w, wxCoord h, double sa, double ea ) { + wxCHECK_RET(Ok(), wxT("wxDC::DoDrawEllepticArc Invalid DC")); + + Rect r; + double angle = sa - ea; // Order important Mac in opposite direction to wx + + wxCoord xx = XLOG2DEV(x); + wxCoord yy = YLOG2DEV(y); + wxCoord ww = m_signX * XLOG2DEVREL(w); + wxCoord hh = m_signY * YLOG2DEVREL(h); + + // handle -ve width and/or height + if (ww < 0) { ww = -ww; xx = xx - ww; } + if (hh < 0) { hh = -hh; yy = yy - hh; } + + sa = wxConvertWXangleToMACangle(sa); + + r.top = yy; + r.left = xx; + r.bottom = yy + hh; + r.right = xx + ww; + + if(m_brush.GetStyle() != wxTRANSPARENT) { + MacInstallBrush(); + PaintArc(&r, (short)sa, (short)angle); + } + if(m_pen.GetStyle() != wxTRANSPARENT) { + MacInstallPen(); + FrameArc(&r, (short)sa, (short)angle); + } } void wxDC::DoDrawPoint( wxCoord x, wxCoord y ) { - if (!Ok()) - return; + wxCHECK_RET(Ok(), wxT("Invalid DC")); - MacVerifySetup() ; + wxMacPortSetter helper(this) ; if (m_pen.GetStyle() != wxTRANSPARENT) { MacInstallPen() ; - long xx1 = XLOG2DEV(x); - long yy1 = YLOG2DEV(y); + wxCoord xx1 = XLOG2DEV(x); + wxCoord yy1 = YLOG2DEV(y); ::MoveTo(xx1,yy1); ::LineTo(xx1+1, yy1+1); - }; + } } void wxDC::DoDrawLines(int n, wxPoint points[], wxCoord xoffset, wxCoord yoffset) { - if (!Ok()) - return; - MacVerifySetup() ; + wxCHECK_RET(Ok(), wxT("Invalid DC")); + wxMacPortSetter helper(this) ; if (m_pen.GetStyle() == wxTRANSPARENT) return; - MacInstallPen() ; + MacInstallPen() ; - int offset = (m_pen.GetWidth() - 1 ) / 2 ; - long x1, x2 , y1 , y2 ; + wxCoord offset = ( (m_pen.GetWidth() == 0 ? 1 : + m_pen.GetWidth() ) * (wxCoord)m_scaleX - 1) / 2 ; + + wxCoord x1, x2 , y1 , y2 ; x1 = XLOG2DEV(points[0].x + xoffset); - y1 = YLOG2DEV(points[0].y + yoffset); - ::MoveTo(x1 - offset ,y1 - offset ); + y1 = YLOG2DEV(points[0].y + yoffset); + ::MoveTo(x1 - offset, y1 - offset ); for (int i = 0; i < n-1; i++) { - long x2 = XLOG2DEV(points[i+1].x + xoffset); - long y2 = YLOG2DEV(points[i+1].y + yoffset); - ::LineTo(x2 - offset , y2 - offset ); + x2 = XLOG2DEV(points[i+1].x + xoffset); + y2 = YLOG2DEV(points[i+1].y + yoffset); + ::LineTo( x2 - offset, y2 - offset ); } } @@ -603,21 +749,20 @@ void wxDC::DoDrawPolygon(int n, wxPoint points[], wxCoord xoffset, wxCoord yoffset, int fillStyle ) { - if (!Ok()) - return; - MacVerifySetup() ; + wxCHECK_RET(Ok(), wxT("Invalid DC")); + wxMacPortSetter helper(this) ; PolyHandle polygon = OpenPoly() ; - long x1, x2 , y1 , y2 ; + wxCoord x1, x2 , y1 , y2 ; x1 = XLOG2DEV(points[0].x + xoffset); - y1 = YLOG2DEV(points[0].y + yoffset); - ::MoveTo(x1,y1); + y1 = YLOG2DEV(points[0].y + yoffset); + ::MoveTo(x1,y1); for (int i = 0; i < n-1; i++) { - long x2 = XLOG2DEV(points[i+1].x + xoffset); - long y2 = YLOG2DEV(points[i+1].y + yoffset); - ::LineTo(x2, y2); + x2 = XLOG2DEV(points[i+1].x + xoffset); + y2 = YLOG2DEV(points[i+1].y + yoffset); + ::LineTo(x2, y2); } ClosePoly() ; @@ -625,26 +770,25 @@ void wxDC::DoDrawPolygon(int n, wxPoint points[], { MacInstallBrush() ; ::PaintPoly( polygon ) ; - }; + } if (m_pen.GetStyle() != wxTRANSPARENT) { MacInstallPen() ; ::FramePoly( polygon ) ; - }; + } KillPoly( polygon ) ; } -void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height) +void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height) { - if (!Ok()) - return; - MacVerifySetup() ; - - long xx = XLOG2DEV(x); - long yy = YLOG2DEV(y); - long ww = m_signX * XLOG2DEVREL(width); - long hh = m_signY * YLOG2DEVREL(height); + wxCHECK_RET(Ok(), wxT("Invalid DC")); + wxMacPortSetter helper(this) ; + + wxCoord xx = XLOG2DEV(x); + wxCoord yy = YLOG2DEV(y); + wxCoord ww = m_signX * XLOG2DEVREL(width); + wxCoord hh = m_signY * YLOG2DEVREL(height); // CMB: draw nothing if transformed w or h is 0 if (ww == 0 || hh == 0) @@ -669,30 +813,29 @@ void wxDC::DoDrawRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height) { MacInstallBrush() ; ::PaintRect( &rect ) ; - }; + } if (m_pen.GetStyle() != wxTRANSPARENT) { MacInstallPen() ; ::FrameRect( &rect ) ; - }; + } } void wxDC::DoDrawRoundedRectangle(wxCoord x, wxCoord y, wxCoord width, wxCoord height, double radius) { - if (!Ok()) - return; - MacVerifySetup() ; + wxCHECK_RET(Ok(), wxT("Invalid DC")); + wxMacPortSetter helper(this) ; - if (radius < 0.0) - radius = - radius * ((width < height) ? width : height); + if (radius < 0.0) + radius = - radius * ((width < height) ? width : height); - long xx = XLOG2DEV(x); - long yy = YLOG2DEV(y); - long ww = m_signX * XLOG2DEVREL(width); - long hh = m_signY * YLOG2DEVREL(height); + wxCoord xx = XLOG2DEV(x); + wxCoord yy = YLOG2DEV(y); + wxCoord ww = m_signX * XLOG2DEVREL(width); + wxCoord hh = m_signY * YLOG2DEVREL(height); // CMB: draw nothing if transformed w or h is 0 if (ww == 0 || hh == 0) @@ -716,26 +859,25 @@ void wxDC::DoDrawRoundedRectangle(wxCoord x, wxCoord y, if (m_brush.GetStyle() != wxTRANSPARENT) { MacInstallBrush() ; - ::PaintRoundRect( &rect , radius * 2 , radius * 2 ) ; - }; + ::PaintRoundRect( &rect , int(radius * 2) , int(radius * 2) ) ; + } if (m_pen.GetStyle() != wxTRANSPARENT) { MacInstallPen() ; - ::FrameRoundRect( &rect , radius * 2 , radius * 2 ) ; - }; + ::FrameRoundRect( &rect , int(radius * 2) , int(radius * 2) ) ; + } } void wxDC::DoDrawEllipse(wxCoord x, wxCoord y, wxCoord width, wxCoord height) { - if (!Ok()) - return; - MacVerifySetup() ; + wxCHECK_RET(Ok(), wxT("Invalid DC")); + wxMacPortSetter helper(this) ; - long xx = XLOG2DEV(x); - long yy = YLOG2DEV(y); - long ww = m_signX * XLOG2DEVREL(width); - long hh = m_signY * YLOG2DEVREL(height); + wxCoord xx = XLOG2DEV(x); + wxCoord yy = YLOG2DEV(y); + wxCoord ww = m_signX * XLOG2DEVREL(width); + wxCoord hh = m_signY * YLOG2DEVREL(height); // CMB: draw nothing if transformed w or h is 0 if (ww == 0 || hh == 0) @@ -760,178 +902,13 @@ void wxDC::DoDrawEllipse(wxCoord x, wxCoord y, wxCoord width, wxCoord height) { MacInstallBrush() ; ::PaintOval( &rect ) ; - }; + } if (m_pen.GetStyle() != wxTRANSPARENT) { MacInstallPen() ; ::FrameOval( &rect ) ; - }; -} - -// ----------------------------------- spline code ---------------------------------------- - -static void wx_quadratic_spline(double a1, double b1, double a2, double b2, - double a3, double b3, double a4, double b4); -static void wx_clear_stack(void); -static int wx_spline_pop(double *x1, double *y1, double *x2, double *y2, double *x3, - double *y3, double *x4, double *y4); -static void wx_spline_push(double x1, double y1, double x2, double y2, double x3, double y3, - double x4, double y4); -static bool wx_spline_add_point(double x, double y); -static void wx_spline_draw_point_array(wxDC *dc); - -static wxList wx_spline_point_list; - -#define half(z1, z2) ((z1+z2)/2.0) -#define THRESHOLD 5 - -/* iterative version */ - -static void wx_quadratic_spline(double a1, double b1, double a2, double b2, double a3, double b3, double a4, - double b4) -{ - register double xmid, ymid; - double x1, y1, x2, y2, x3, y3, x4, y4; - - wx_clear_stack(); - wx_spline_push(a1, b1, a2, b2, a3, b3, a4, b4); - - while (wx_spline_pop(&x1, &y1, &x2, &y2, &x3, &y3, &x4, &y4)) { - xmid = (double)half(x2, x3); - ymid = (double)half(y2, y3); - if (fabs(x1 - xmid) < THRESHOLD && fabs(y1 - ymid) < THRESHOLD && - fabs(xmid - x4) < THRESHOLD && fabs(ymid - y4) < THRESHOLD) { - wx_spline_add_point( x1, y1 ); - wx_spline_add_point( xmid, ymid ); - } else { - wx_spline_push(xmid, ymid, (double)half(xmid, x3), (double)half(ymid, y3), - (double)half(x3, x4), (double)half(y3, y4), x4, y4); - wx_spline_push(x1, y1, (double)half(x1, x2), (double)half(y1, y2), - (double)half(x2, xmid), (double)half(y2, ymid), xmid, ymid); } - } -} - -/* utilities used by spline drawing routines */ - -typedef struct wx_spline_stack_struct { - double x1, y1, x2, y2, x3, y3, x4, y4; -} Stack; - -#define SPLINE_STACK_DEPTH 20 -static Stack wx_spline_stack[SPLINE_STACK_DEPTH]; -static Stack *wx_stack_top; -static int wx_stack_count; - -static void wx_clear_stack(void) -{ - wx_stack_top = wx_spline_stack; - wx_stack_count = 0; -} - -static void wx_spline_push(double x1, double y1, double x2, double y2, double x3, double y3, double x4, double y4) -{ - wx_stack_top->x1 = x1; - wx_stack_top->y1 = y1; - wx_stack_top->x2 = x2; - wx_stack_top->y2 = y2; - wx_stack_top->x3 = x3; - wx_stack_top->y3 = y3; - wx_stack_top->x4 = x4; - wx_stack_top->y4 = y4; - wx_stack_top++; - wx_stack_count++; -} - -static int wx_spline_pop(double *x1, double *y1, double *x2, double *y2, - double *x3, double *y3, double *x4, double *y4) -{ - if (wx_stack_count == 0) - return (0); - wx_stack_top--; - wx_stack_count--; - *x1 = wx_stack_top->x1; - *y1 = wx_stack_top->y1; - *x2 = wx_stack_top->x2; - *y2 = wx_stack_top->y2; - *x3 = wx_stack_top->x3; - *y3 = wx_stack_top->y3; - *x4 = wx_stack_top->x4; - *y4 = wx_stack_top->y4; - return (1); -} - -static bool wx_spline_add_point(double x, double y) -{ - wxPoint *point = new wxPoint ; - point->x = (int) x; - point->y = (int) y; - wx_spline_point_list.Append((wxObject*)point); - return TRUE; -} - -static void wx_spline_draw_point_array(wxDC *dc) -{ - dc->DrawLines(&wx_spline_point_list, 0, 0 ); - wxNode *node = wx_spline_point_list.First(); - while (node) - { - wxPoint *point = (wxPoint *)node->Data(); - delete point; - delete node; - node = wx_spline_point_list.First(); - } -} - -void wxDC::DoDrawSpline(wxList *points) -{ - wxPoint *p; - double cx1, cy1, cx2, cy2, cx3, cy3, cx4, cy4; - double x1, y1, x2, y2; - - wxNode *node = points->First(); - p = (wxPoint *)node->Data(); - - x1 = p->x; - y1 = p->y; - - node = node->Next(); - p = (wxPoint *)node->Data(); - - x2 = p->x; - y2 = p->y; - cx1 = (double)((x1 + x2) / 2); - cy1 = (double)((y1 + y2) / 2); - cx2 = (double)((cx1 + x2) / 2); - cy2 = (double)((cy1 + y2) / 2); - - wx_spline_add_point(x1, y1); - - while ((node = node->Next()) != NULL) - { - p = (wxPoint *)node->Data(); - x1 = x2; - y1 = y2; - x2 = p->x; - y2 = p->y; - cx4 = (double)(x1 + x2) / 2; - cy4 = (double)(y1 + y2) / 2; - cx3 = (double)(x1 + cx4) / 2; - cy3 = (double)(y1 + cy4) / 2; - - wx_quadratic_spline(cx1, cy1, cx2, cy2, cx3, cy3, cx4, cy4); - - cx1 = cx4; - cy1 = cy4; - cx2 = (double)(cx1 + x2) / 2; - cy2 = (double)(cy1 + y2) / 2; - } - - wx_spline_add_point( cx1, cy1 ); - wx_spline_add_point( x2, y2 ); - - wx_spline_draw_point_array( this ); } @@ -945,15 +922,18 @@ bool wxDC::CanDrawBitmap(void) const bool wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height, wxDC *source, wxCoord xsrc, wxCoord ysrc, int logical_func , bool useMask ) { - if (!Ok()) return FALSE; - MacVerifySetup() ; + wxCHECK_MSG(Ok(), false, wxT("wxDC::DoBlit Illegal dc")); + wxCHECK_MSG(source->Ok(), false, wxT("wxDC::DoBlit Illegal source DC")); + wxMacPortSetter helper(this) ; CGrafPtr sourcePort = (CGrafPtr) source->m_macPort ; PixMapHandle bmappixels = GetGWorldPixMap( sourcePort ) ; - RGBColor white = { 0xFFFF, 0xFFFF,0xFFFF} ; - RGBColor black = { 0,0,0} ; - RGBForeColor( &m_textForegroundColour.GetPixel() ) ; - RGBBackColor( &m_textBackgroundColour.GetPixel() ) ; + RGBColor white = { 0xFFFF, 0xFFFF,0xFFFF} ; + RGBColor black = { 0,0,0} ; + RGBColor forecolor = m_textForegroundColour.GetPixel(); + RGBColor backcolor = m_textBackgroundColour.GetPixel(); + RGBForeColor( &forecolor ) ; + RGBBackColor( &backcolor ) ; if ( LockPixels(bmappixels) ) { @@ -986,15 +966,17 @@ bool wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height, wxASSERT( mode == srcCopy ) ; if ( LockPixels( GetGWorldPixMap( source->m_macMask ) ) ) { - CopyMask( GetPortBitMapForCopyBits( sourcePort ) , GetPortBitMapForCopyBits( source->m_macMask ) , - GetPortBitMapForCopyBits( m_macPort ) , - &srcrect, &srcrect , &dstrect ) ; + CopyMask( GetPortBitMapForCopyBits( sourcePort ) , + GetPortBitMapForCopyBits( source->m_macMask ) , + GetPortBitMapForCopyBits( m_macPort ) , + &srcrect, &srcrect , &dstrect ) ; UnlockPixels( GetGWorldPixMap( source->m_macMask ) ) ; } } else { - CopyBits( GetPortBitMapForCopyBits( sourcePort ) , GetPortBitMapForCopyBits( m_macPort ) , + CopyBits( GetPortBitMapForCopyBits( sourcePort ) , + GetPortBitMapForCopyBits( m_macPort ) , &srcrect, &dstrect, mode, NULL ) ; } UnlockPixels( bmappixels ) ; @@ -1008,14 +990,107 @@ bool wxDC::DoBlit(wxCoord xdest, wxCoord ydest, wxCoord width, wxCoord height, } void wxDC::DoDrawRotatedText(const wxString& text, wxCoord x, wxCoord y, - double angle) + double angle) { + wxCHECK_RET( Ok(), wxT("wxDC::DoDrawRotatedText Invalid window dc") ); + + if (angle == 0.0) + { + DrawText(text, x, y); + return; + } + + MacInstallFont(); + + // the size of the text + wxCoord w, h; + GetTextExtent(text, &w, &h); + + // draw the string normally + wxBitmap src(w, h); + wxMemoryDC dc; + dc.SelectObject(src); + dc.SetFont(GetFont()); + dc.SetBackground(*wxWHITE_BRUSH); + dc.SetBrush(*wxBLACK_BRUSH); + dc.Clear(); + dc.DrawText(text, 0, 0); + dc.SetFont(wxNullFont); + dc.SelectObject(wxNullBitmap); + + wxMacPortSetter helper(this) ; + + // Calculate the size of the rotated bounding box. + double rad = DegToRad(angle); + double dx = cos(rad); + double dy = sin(rad); + + // the rectngle vertices are counted clockwise with the first one being at + // (0, 0) (or, rather, at (x, y)) + double x2 = w * dx; + double y2 = -w * dy; // y axis points to the bottom, hence minus + double x4 = h * dy; + double y4 = h * dx; + double x3 = x4 + x2; + double y3 = y4 + y2; + + // calc max and min + wxCoord maxX = (wxCoord)(dmax(x2, dmax(x3, x4)) + 0.5); + wxCoord maxY = (wxCoord)(dmax(y2, dmax(y3, y4)) + 0.5); + wxCoord minX = (wxCoord)(dmin(x2, dmin(x3, x4)) - 0.5); + wxCoord minY = (wxCoord)(dmin(y2, dmin(y3, y4)) - 0.5); + + // prepare to blit-with-rotate the bitmap to the DC + wxImage image(src); + + RGBColor colText = m_textForegroundColour.GetPixel(); + RGBColor colBack = m_textBackgroundColour.GetPixel(); + + unsigned char *data = image.GetData(); + + wxCoord dstX, dstY; + double r, angleOrig; + bool textPixel; + + // paint pixel by pixel + for ( wxCoord srcX = 0; srcX < w; srcX++ ) + { + for ( wxCoord srcY = 0; srcY < h; srcY++ ) + { + // transform source coords to dest coords + r = sqrt( (double)(srcX * srcX + srcY * srcY) ); + angleOrig = atan2((double)srcY, (double)srcX) - rad; + dstX = (wxCoord)(r * cos(angleOrig) + 0.5); + dstY = (wxCoord)(r * sin(angleOrig) + 0.5); + + // black pixel? + textPixel = data[(srcY*w + srcX)*3] == 0; + if ( textPixel || (m_backgroundMode == wxSOLID) ) + { + SetCPixel(XLOG2DEV(x + dstX), YLOG2DEV(y + dstY), + textPixel ? &colText : &colBack); + } + } + } + + // it would be better to draw with non underlined font and draw the line + // manually here (it would be more straight...) +#if 0 + if ( m_font.GetUnderlined() ) + { + ::MoveTo(XLOG2DEV(x + x4), YLOG2DEV(y + y4 + font->descent)); + ::LineTo(XLOG2DEV(x + x3), YLOG2DEV(y + y3 + font->descent)); + } +#endif // 0 + + // update the bounding box + CalcBoundingBox(x + minX, y + minY); + CalcBoundingBox(x + maxX, y + maxY); } void wxDC::DoDrawText(const wxString& strtext, wxCoord x, wxCoord y) { - if (!Ok()) - return; - MacVerifySetup() ; + wxCHECK_RET(Ok(), wxT("wxDC::DoDrawText Invalid DC")); + wxMacPortSetter helper(this) ; long xx = XLOG2DEV(x); long yy = YLOG2DEV(y); @@ -1080,10 +1155,9 @@ void wxDC::DoDrawText(const wxString& strtext, wxCoord x, wxCoord y) } } -bool wxDC::CanGetTextExtent(void) const +bool wxDC::CanGetTextExtent() const { - if ( !Ok() ) - return false ; + wxCHECK_MSG(Ok(), false, wxT("Invalid DC")); return true ; } @@ -1092,10 +1166,8 @@ void wxDC::DoGetTextExtent( const wxString &string, wxCoord *width, wxCoord *he wxCoord *descent, wxCoord *externalLeading , wxFont *theFont ) const { - if (!Ok()) - return; - - MacVerifySetup() ; + wxCHECK_RET(Ok(), wxT("Invalid DC")); + wxMacPortSetter helper(this) ; wxFont formerFont = m_font ; @@ -1174,10 +1246,9 @@ void wxDC::DoGetTextExtent( const wxString &string, wxCoord *width, wxCoord *he wxCoord wxDC::GetCharWidth(void) const { - if (!Ok()) - return 1; + wxCHECK_MSG(Ok(), 1, wxT("Invalid DC")); - MacVerifySetup() ; + wxMacPortSetter helper(this) ; MacInstallFont() ; @@ -1189,10 +1260,9 @@ wxCoord wxDC::GetCharWidth(void) const wxCoord wxDC::GetCharHeight(void) const { - if (!Ok()) - return 1; + wxCHECK_MSG(Ok(), 1, wxT("Invalid DC")); - MacVerifySetup() ; + wxMacPortSetter helper(this) ; MacInstallFont() ; @@ -1204,26 +1274,22 @@ wxCoord wxDC::GetCharHeight(void) const void wxDC::Clear(void) { - if (!Ok()) - return; - MacVerifySetup() ; + wxCHECK_RET(Ok(), wxT("Invalid DC")); + wxMacPortSetter helper(this) ; Rect rect = { -32767 , -32767 , 32767 , 32767 } ; if (m_backgroundBrush.GetStyle() != wxTRANSPARENT) { MacInstallBrush() ; ::EraseRect( &rect ) ; - }; + } } void wxDC::MacInstallFont() const { - if (!Ok()) - return; - MacVerifySetup() ; - - if ( m_macFontInstalled ) - return ; + wxCHECK_RET(Ok(), wxT("Invalid DC")); +// if ( m_macFontInstalled ) +// return ; Pattern blackColor ; wxFontRefData * font = (wxFontRefData*) m_font.GetRefData() ; @@ -1231,15 +1297,17 @@ void wxDC::MacInstallFont() const if ( font ) { ::TextFont( font->m_macFontNum ) ; - ::TextSize( m_scaleY * font->m_macFontSize ) ; + ::TextSize( short(m_scaleY * font->m_macFontSize) ) ; ::TextFace( font->m_macFontStyle ) ; m_macFontInstalled = true ; m_macBrushInstalled = false ; m_macPenInstalled = false ; - - ::RGBForeColor(&m_textForegroundColour.GetPixel() ); - ::RGBBackColor(&m_textBackgroundColour.GetPixel() ); + + RGBColor forecolor = m_textForegroundColour.GetPixel(); + RGBColor backcolor = m_textBackgroundColour.GetPixel(); + ::RGBForeColor( &forecolor ); + ::RGBBackColor( &backcolor ); } else { @@ -1247,7 +1315,7 @@ void wxDC::MacInstallFont() const GetFNum( "\pGeneva" , &fontnum ) ; ::TextFont( fontnum ) ; - ::TextSize( m_scaleY * 10 ) ; + ::TextSize( short(m_scaleY * 10) ) ; ::TextFace( 0 ) ; // todo reset after spacing changes - or store the current spacing somewhere @@ -1255,10 +1323,12 @@ void wxDC::MacInstallFont() const m_macFontInstalled = true ; m_macBrushInstalled = false ; m_macPenInstalled = false ; - ::RGBForeColor( &(m_textForegroundColour.GetPixel()) ); - ::RGBBackColor(&m_textBackgroundColour.GetPixel() ); - } + RGBColor forecolor = m_textForegroundColour.GetPixel(); + RGBColor backcolor = m_textBackgroundColour.GetPixel(); + ::RGBForeColor( &forecolor ); + ::RGBBackColor( &backcolor ); + } short mode = patCopy ; @@ -1336,26 +1406,32 @@ static void wxMacGetHatchPattern(int hatchStyle, Pattern *pattern) void wxDC::MacInstallPen() const { - if (!Ok()) - return; - MacVerifySetup() ; + wxCHECK_RET(Ok(), wxT("Invalid DC")); Pattern blackColor; - if ( m_macPenInstalled ) - return ; +// if ( m_macPenInstalled ) +// return ; - ::RGBForeColor(&m_pen.GetColour().GetPixel() ); - ::RGBBackColor(&m_backgroundBrush.GetColour().GetPixel() ); + RGBColor forecolor = m_pen.GetColour().GetPixel(); + RGBColor backcolor = m_backgroundBrush.GetColour().GetPixel(); + ::RGBForeColor( &forecolor ); + ::RGBBackColor( &backcolor ); ::PenNormal() ; - int penWidth = m_pen.GetWidth(); + int penWidth = m_pen.GetWidth() * (int) m_scaleX ; + + // null means only one pixel, at whatever resolution + if ( penWidth == 0 ) + penWidth = 1 ; ::PenSize(penWidth, penWidth); int penStyle = m_pen.GetStyle(); if (penStyle == wxSOLID) + { ::PenPat(GetQDGlobalsBlack(&blackColor)); + } else if (IS_HATCH(penStyle)) { Pattern pat ; @@ -1415,17 +1491,18 @@ void wxDC::MacInstallPen() const void wxDC::MacInstallBrush() const { - if (!Ok()) - return; - MacVerifySetup() ; + wxCHECK_RET(Ok(), wxT("Invalid DC")); + Pattern blackColor, whiteColor ; - if ( m_macBrushInstalled ) - return ; +// if ( m_macBrushInstalled ) +// return ; // foreground - ::RGBForeColor(&m_brush.GetColour().GetPixel() ); - ::RGBBackColor(&m_backgroundBrush.GetColour().GetPixel() ); + RGBColor forecolor = m_brush.GetColour().GetPixel(); + RGBColor backcolor = m_backgroundBrush.GetColour().GetPixel(); + ::RGBForeColor( &forecolor ); + ::RGBBackColor( &backcolor ); int brushStyle = m_brush.GetStyle(); if (brushStyle == wxSOLID)