solemnwarning/wxwidgets
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

Use wxDCClipper instead of manual DestroyClippingRegion()

Browse source 
No real changes, just avoid manual cleanup and use a RAII helper
instead.
This commit is contained in:
Vadim Zeitlin 2022-10-18 18:26:20 +02:00
parent 3d5078cefb
commit 6f6dde9121
1 changed files with 2 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/generic/treectlg.cpp
View file

@ -2647,19 +2647,18 @@ void wxGenericTreeCtrl::PaintItem(wxGenericTreeItem *item, wxDC& dc)
if ( state != wxTREE_ITEMSTATE_NONE )
{
dc.SetClippingRegion( item->GetX(), item->GetY(), state_w, total_h );
wxDCClipper clip(dc, item->GetX(), item->GetY(), state_w, total_h);
dc.DrawBitmap( m_imagesState.GetImageBitmapFor(this, state),
item->GetX(),
item->GetY() +
(total_h > state_h ? (total_h-state_h)/2
: 0),
true /* use mask */ );
dc.DestroyClippingRegion();
}
if ( image != NO_IMAGE )
{
dc.SetClippingRegion(item->GetX() + state_w, item->GetY(),
wxDCClipper clip(dc, item->GetX() + state_w, item->GetY(),
image_w, total_h);
dc.DrawBitmap( GetImageBitmapFor(this, image),
item->GetX() + state_w,
@ -2667,7 +2666,6 @@ void wxGenericTreeCtrl::PaintItem(wxGenericTreeItem *item, wxDC& dc)
(total_h > image_h ? (total_h-image_h)/2
: 0),
true /* use mask */ );
dc.DestroyClippingRegion();
}
dc.SetBackgroundMode(wxBRUSHSTYLE_TRANSPARENT);