wxStatusBarGeneric::wxStatusBarGeneric()
{
- m_statusWidths = (int *) NULL;
- m_statusStrings = (wxString *) NULL;
- m_nFields = 0;
m_borderX = wxTHICK_LINE_BORDER;
m_borderY = wxTHICK_LINE_BORDER;
}
# ifdef __WXMSW__
SetFont(wxNullFont);
# endif // MSW
-
- if ( m_statusStrings )
- delete[] m_statusStrings;
}
bool wxStatusBarGeneric::Create(wxWindow *parent,
- wxWindowID id,
- long style,
- const wxString& name)
+ wxWindowID id,
+ long style,
+ const wxString& name)
{
- m_statusWidths = (int *) NULL;
- m_statusStrings = (wxString *) NULL;
- m_nFields = 0;
+ // If create is ever meant to be re-entrant over the life of
+ // an object we should:
+ // m_statusStrings.Empty();
+
m_borderX = wxTHICK_LINE_BORDER;
m_borderY = wxTHICK_LINE_BORDER;
void wxStatusBarGeneric::SetFieldsCount(int number, const int *widths)
{
- if ( number != m_nFields )
- {
- m_nFields = number;
+ wxASSERT_MSG( number >= 0,
+ _T("Yes, number should be a size_t and less than no fields is silly.") );
- delete[] m_statusStrings;
- m_statusStrings = new wxString[number];
- }
+ // if( number > m_nFields )
+
+ int i;
+ for(i = m_nFields; i < number; ++i)
+ m_statusStrings.Add( wxEmptyString );
+
+ // if( number < m_nFields )
+
+ for (i = m_nFields - 1; i >= number; --i)
+ m_statusStrings.Remove(i);
+
+ m_nFields = number;
+
+ wxASSERT_MSG( m_nFields == (int)m_statusStrings.GetCount(),
+ _T("This really should never happen, can we do away with m_nFields here?") );
SetStatusWidths(number, widths);
}
wxRect rect;
GetFieldRect(number, rect);
-
+
Refresh( TRUE, &rect );
}
// delete the old widths in any case - this function may be used to reset
// the widths to the default (all equal)
- delete [] m_statusWidths;
+ // MBN: this is incompatible with at least wxMSW and wxMAC and not
+ // documented, but let's keep it for now
+ ReinitWidths();
if ( !widths_field )
{
// not an error, see the comment above
- m_statusWidths = (int *)NULL;
Refresh();
return;
}
- int i;
-
- // VZ: this doesn't do anything as is_variable is unused later
-#if 0
- // when one window (minimum) is variable (width <= 0)
- bool is_variable = FALSE;
- for (i = 0; i < m_nFields; i++)
- {
- if (widths_field[i] <= 0)
- is_variable = TRUE;
- }
-#endif // 0
-
- // set widths
- m_statusWidths = new int[n];
- for (i = 0; i < m_nFields; i++)
- {
- m_statusWidths[i] = widths_field[i];
- }
- Refresh();
+ wxStatusBarBase::SetStatusWidths(n, widths_field);
}
void wxStatusBarGeneric::OnPaint(wxPaintEvent& WXUNUSED(event) )
#ifdef __WXPM__
wxColour vColor;
- vColor.InitFromName("LIGHT GREY");
+ vColor = wxSystemSettings::GetColour(wxSYS_COLOUR_MENUBAR);
::WinFillRect(dc.m_hPS, &dc.m_vRclPaint, vColor.GetPixel());
#endif
dc.DrawLine(rect.x, rect.y,
rect.x + rect.width, rect.y);
#else
- // Right
+
+ dc.DrawLine(rect.x + rect.width, rect.height + 2,
+ rect.x, rect.height + 2);
dc.DrawLine(rect.x + rect.width, rect.y,
- rect.x + rect.width, rect.y + rect.height + 2);
+ rect.x + rect.width, rect.y + rect.height);
+
dc.SetPen(m_mediumShadowPen);
- dc.DrawLine(rect.x + rect.width + 1, rect.y,
- rect.x + rect.width + 1, rect.y + rect.height + 2);
- dc.DrawLine(rect.x + rect.width + 2, rect.y,
- rect.x + rect.width + 2, rect.y + rect.height + 2);
- // Top
- dc.DrawLine(rect.x + rect.width + 2, rect.y,
- rect.x - 2, rect.y);
- dc.DrawLine(rect.x + rect.width + 1, rect.y - 1,
- rect.x - 2, rect.y - 1);
- dc.SetPen(m_hilightPen);
- dc.DrawLine(rect.x + rect.width, rect.y - 2,
- rect.x - 2, rect.y - 2);
+ dc.DrawLine(rect.x, rect.y,
+ rect.x + rect.width, rect.y);
+ dc.DrawLine(rect.x, rect.y + rect.height,
+ rect.x, rect.y);
#endif
wxColour hilightColour(wxSystemSettings::GetColour(wxSYS_COLOUR_3DHILIGHT));
m_hilightPen = wxPen(hilightColour, 1, wxSOLID);
#elif defined(__WXPM__)
- m_mediumShadowPen = wxPen("DARK GREY", 1, wxSOLID);
+ m_mediumShadowPen = wxPen(wxColour(127, 127, 127), 1, wxSOLID);
m_hilightPen = wxPen("WHITE", 1, wxSOLID);
wxColour vColour;
}
#endif // wxUSE_STATUSBAR
+
+// vi:sts=4:sw=4:et