X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/6c9a19aabab3a878b565e6c2a5f2a3824277c4dc..1ab440bc203fc700744039abdcff12bdd6ac6f0a:/src/msw/dcclient.cpp?ds=inline diff --git a/src/msw/dcclient.cpp b/src/msw/dcclient.cpp index c03897cf2a..05685aa39f 100644 --- a/src/msw/dcclient.cpp +++ b/src/msw/dcclient.cpp @@ -1,5 +1,5 @@ ///////////////////////////////////////////////////////////////////////////// -// Name: dcclient.cpp +// Name: src/msw/dcclient.cpp // Purpose: wxClientDC class // Author: Julian Smart // Modified by: @@ -17,10 +17,6 @@ // headers // ---------------------------------------------------------------------------- -#ifdef __GNUG__ - #pragma implementation "dcclient.h" -#endif - // For compilers that support precompilation, includes "wx.h". #include "wx/wxprec.h" @@ -28,13 +24,15 @@ #pragma hdrstop #endif -#include "wx/string.h" -#include "wx/log.h" -#include "wx/window.h" +#include "wx/dcclient.h" -#include "wx/msw/private.h" +#ifndef WX_PRECOMP + #include "wx/string.h" + #include "wx/log.h" + #include "wx/window.h" +#endif -#include "wx/dcclient.h" +#include "wx/msw/private.h" // ---------------------------------------------------------------------------- // array/list types @@ -56,7 +54,7 @@ struct WXDLLEXPORT wxPaintDCInfo #include "wx/arrimpl.cpp" -WX_DEFINE_OBJARRAY(wxArrayDCInfo); +WX_DEFINE_OBJARRAY(wxArrayDCInfo) // ---------------------------------------------------------------------------- // macros @@ -65,6 +63,7 @@ WX_DEFINE_OBJARRAY(wxArrayDCInfo); IMPLEMENT_DYNAMIC_CLASS(wxWindowDC, wxDC) IMPLEMENT_DYNAMIC_CLASS(wxClientDC, wxWindowDC) IMPLEMENT_DYNAMIC_CLASS(wxPaintDC, wxClientDC) +IMPLEMENT_CLASS(wxPaintDCEx, wxPaintDC) // ---------------------------------------------------------------------------- // global variables @@ -74,8 +73,8 @@ static PAINTSTRUCT g_paintStruct; #ifdef __WXDEBUG__ // a global variable which we check to verify that wxPaintDC are only - // created in resopnse to WM_PAINT message - doing this from elsewhere is a - // common programming error among wxWindows programmers and might lead to + // created in response to WM_PAINT message - doing this from elsewhere is a + // common programming error among wxWidgets programmers and might lead to // very subtle and difficult to debug refresh/repaint bugs. int g_isPainting = 0; #endif // __WXDEBUG__ @@ -155,7 +154,11 @@ void wxClientDC::InitDC() // in wxUniv build we must manually do some DC adjustments usually // performed by Windows for us -#ifdef __WXUNIVERSAL__ + // + // we also need to take the menu/toolbar manually into account under + // Windows CE because they're just another control there, not anything + // special as usually under Windows +#if defined(__WXUNIVERSAL__) || defined(__WXWINCE__) wxPoint ptOrigin = m_canvas->GetClientAreaOrigin(); if ( ptOrigin.x || ptOrigin.y ) { @@ -164,8 +167,8 @@ void wxClientDC::InitDC() } // clip the DC to avoid overwriting the non client area - SetClippingRegion(wxPoint(0, 0), m_canvas->GetClientSize()); -#endif // __WXUNIVERSAL__ + SetClippingRegion(wxPoint(0,0), m_canvas->GetClientSize()); +#endif // __WXUNIVERSAL__ || __WXWINCE__ } wxClientDC::~wxClientDC() @@ -231,7 +234,7 @@ wxPaintDC::wxPaintDC(wxWindow *canvas) else // not in cache, create a new one { m_hDC = (WXHDC)::BeginPaint(GetHwndOf(m_canvas), &g_paintStruct); - if (m_hDC) + if (m_hDC) ms_cache.Add(new wxPaintDCInfo(m_canvas, this)); } @@ -252,7 +255,7 @@ wxPaintDC::~wxPaintDC() wxCHECK_RET( info, wxT("existing DC should have a cache entry") ); - if ( !--info->count ) + if ( --info->count == 0 ) { ::EndPaint(GetHwndOf(m_canvas), &g_paintStruct); @@ -292,11 +295,10 @@ wxPaintDCInfo *wxPaintDC::FindInCache(size_t *index) const // find the entry for this DC in the cache (keyed by the window) WXHDC wxPaintDC::FindDCInCache(wxWindow* win) { - wxPaintDCInfo *info = NULL; size_t nCache = ms_cache.GetCount(); for ( size_t n = 0; n < nCache; n++ ) { - info = &ms_cache[n]; + wxPaintDCInfo *info = &ms_cache[n]; if ( info->hwnd == win->GetHWND() ) { return info->hdc; @@ -305,3 +307,51 @@ WXHDC wxPaintDC::FindDCInCache(wxWindow* win) return 0; } +/* + * wxPaintDCEx + */ + +// TODO: don't duplicate wxPaintDC code here!! + +wxPaintDCEx::wxPaintDCEx(wxWindow *canvas, WXHDC dc) : saveState(0) +{ + wxCHECK_RET( dc, wxT("wxPaintDCEx requires an existing device context") ); + + m_canvas = canvas; + + wxPaintDCInfo *info = FindInCache(); + if ( info ) + { + m_hDC = info->hdc; + info->count++; + } + else // not in cache, create a new one + { + m_hDC = dc; + ms_cache.Add(new wxPaintDCInfo(m_canvas, this)); + saveState = SaveDC((HDC) dc); + } +} + +wxPaintDCEx::~wxPaintDCEx() +{ + size_t index; + wxPaintDCInfo *info = FindInCache(&index); + + wxCHECK_RET( info, wxT("existing DC should have a cache entry") ); + + if ( --info->count == 0 ) + { + RestoreDC((HDC) m_hDC, saveState); + ms_cache.RemoveAt(index); + + // Reduce the number of bogus reports of non-freed memory + // at app exit + if (ms_cache.IsEmpty()) + ms_cache.Clear(); + } + //else: cached DC entry is still in use + + // prevent the base class dtor from ReleaseDC()ing it again + m_hDC = 0; +}