mirror of
https://github.com/cookiengineer/audacity
synced 2025-10-21 14:02:57 +02:00
Improve comments surrounding my recent slider checkin.
This commit is contained in:
@@ -600,6 +600,11 @@ void LWSlider::Draw()
|
||||
//
|
||||
// Get the thumb slider bitmap
|
||||
//
|
||||
// AD: Setting the mThumbBitmap pointer requires caution, because
|
||||
// ownership of the object pointed to varies. If we've allocated
|
||||
// mThumbBitmap we must delete it first, and we must set
|
||||
// mThumbBitmapAllocated according to whether we have.
|
||||
//
|
||||
|
||||
if (mEnabled && mOrientation == wxHORIZONTAL)
|
||||
{
|
||||
@@ -645,9 +650,6 @@ void LWSlider::Draw()
|
||||
new wxMask(wxBitmap(SliderThumb_VerticalAlpha), *wxBLACK));
|
||||
}
|
||||
|
||||
// mThumbBitmap = new wxBitmap( SliderThumb );
|
||||
// mThumbBitmap->SetMask( new wxMask( wxBitmap( SliderThumbAlpha ), *wxBLACK ) );
|
||||
|
||||
//
|
||||
// Now the background bitmap
|
||||
//
|
||||
|
@@ -213,9 +213,9 @@ class LWSlider
|
||||
wxBitmap *mBitmap;
|
||||
wxBitmap *mThumbBitmap;
|
||||
|
||||
// True if this object owns *mThumbBitmap (sometimes mThumbBitmap points to
|
||||
// an object we shouldn't delete) -- once we get theming totally right this
|
||||
// should go away
|
||||
// AD: True if this object owns *mThumbBitmap (sometimes mThumbBitmap points
|
||||
// to an object we shouldn't delete) -- once we get theming totally right
|
||||
// this should go away
|
||||
bool mThumbBitmapAllocated;
|
||||
|
||||
wxString mName;
|
||||
|
Reference in New Issue
Block a user