remove unnecessary DoGetBestSize

[wxWidgets.git] / src / univ / radiobox.cpp

diff --git a/src/univ/radiobox.cpp b/src/univ/radiobox.cpp
index a9e5d1a34c147386088b265d1e925e1766c21d1f..ac606d1cd3bcf011dab6861d5b6adafe5c408a81 100644 (file)
--- a/src/univ/radiobox.cpp
+++ b/src/univ/radiobox.cpp
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        univ/radiobox.cpp
+// Name:        src/univ/radiobox.cpp

 // Purpose:     wxRadioBox implementation
 // Author:      Vadim Zeitlin
 // Modified by:
 // Purpose:     wxRadioBox implementation
 // Author:      Vadim Zeitlin
 // Modified by:


@@ -17,10 +17,6 @@
 // headers
 // ----------------------------------------------------------------------------
 
 // headers
 // ----------------------------------------------------------------------------
 

-#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
-    #pragma implementation "univradiobox.h"
-#endif
-

 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__


@@ -29,9 +25,10 @@
 
 #if wxUSE_RADIOBOX
 
 
 #if wxUSE_RADIOBOX
 

+#include "wx/radiobox.h"
+

 #ifndef WX_PRECOMP
     #include "wx/dcclient.h"
 #ifndef WX_PRECOMP
     #include "wx/dcclient.h"

-    #include "wx/radiobox.h"

     #include "wx/radiobut.h"
     #include "wx/validate.h"
     #include "wx/arrstr.h"
     #include "wx/radiobut.h"
     #include "wx/validate.h"
     #include "wx/arrstr.h"


@@ -99,7 +96,6 @@ IMPLEMENT_DYNAMIC_CLASS(wxRadioBox, wxControl)
 void wxRadioBox::Init()
 {
     m_selection = -1;
 void wxRadioBox::Init()
 {
     m_selection = -1;

-    m_majorDim = 0;

 }
 
 wxRadioBox::wxRadioBox(wxWindow *parent, wxWindowID id, const wxString& title,
 }
 
 wxRadioBox::wxRadioBox(wxWindow *parent, wxWindowID id, const wxString& title,


@@ -151,20 +147,20 @@ bool wxRadioBox::Create(wxWindow *parent,
     if ( !(style & (wxRA_LEFTTORIGHT | wxRA_TOPTOBOTTOM)) )
     {
         // horizontal radiobox use left to right layout
     if ( !(style & (wxRA_LEFTTORIGHT | wxRA_TOPTOBOTTOM)) )
     {
         // horizontal radiobox use left to right layout

-        if ( style & wxRA_HORIZONTAL )
+        if ( style & wxRA_SPECIFY_COLS )

         {
             style |= wxRA_LEFTTORIGHT;
         }
         {
             style |= wxRA_LEFTTORIGHT;
         }

-        else if ( style & wxRA_VERTICAL )
+        else if ( style & wxRA_SPECIFY_ROWS )

         {
             style |= wxRA_TOPTOBOTTOM;
         }
         else
         {
         {
             style |= wxRA_TOPTOBOTTOM;
         }
         else
         {

-            wxFAIL_MSG( _T("you must specify wxRA_XXX style!") );
+            wxFAIL_MSG( wxT("you must specify wxRA_XXX style!") );

 
             // use default
 
             // use default

-            style = wxRA_HORIZONTAL | wxRA_LEFTTORIGHT;
+            style = wxRA_SPECIFY_COLS | wxRA_LEFTTORIGHT;

         }
     }
 
         }
     }
 


@@ -178,7 +174,7 @@ bool wxRadioBox::Create(wxWindow *parent,
     Append(n, choices);
 
     // majorDim default value is 0 which means make one row/column
     Append(n, choices);
 
     // majorDim default value is 0 which means make one row/column

-    SetMajorDim(majorDim == 0 ? n : majorDim);
+    SetMajorDim(majorDim == 0 ? n : majorDim, style);

 
     if ( size == wxDefaultSize )
     {
 
     if ( size == wxDefaultSize )
     {


@@ -202,8 +198,8 @@ wxRadioBox::~wxRadioBox()
     // the buttons themselves: this must be done as the user code expects them
     // to disappear now and not some time later when they will be deleted by
     // our (common) parent
     // the buttons themselves: this must be done as the user code expects them
     // to disappear now and not some time later when they will be deleted by
     // our (common) parent

-    size_t count = m_buttons.GetCount();
-    for ( size_t n = 0; n < count; n++ )
+    unsigned int count = m_buttons.GetCount();
+    for ( unsigned int n = 0; n < count; n++ )

     {
         m_buttons[n]->PopEventHandler(true /* delete it */);
 
     {
         m_buttons[n]->PopEventHandler(true /* delete it */);
 


@@ -215,26 +211,6 @@ wxRadioBox::~wxRadioBox()
 // wxRadioBox init
 // ----------------------------------------------------------------------------
 
 // wxRadioBox init
 // ----------------------------------------------------------------------------
 

-void wxRadioBox::SetMajorDim(int majorDim)
-{
-    wxCHECK_RET( majorDim != 0, _T("major radiobox dimension can't be 0") );
-
-    m_majorDim = majorDim;
-
-    int minorDim = (GetCount() + m_majorDim - 1) / m_majorDim;
-
-    if ( GetWindowStyle() & wxRA_SPECIFY_COLS )
-    {
-        m_numCols = majorDim;
-        m_numRows = minorDim;
-    }
-    else // wxRA_SPECIFY_ROWS
-    {
-        m_numCols = minorDim;
-        m_numRows = majorDim;
-    }
-}
-

 void wxRadioBox::Append(int count, const wxString *choices)
 {
     if ( !count )
 void wxRadioBox::Append(int count, const wxString *choices)
 {
     if ( !count )


@@ -263,7 +239,7 @@ void wxRadioBox::Append(int count, const wxString *choices)
 
 void wxRadioBox::SetSelection(int n)
 {
 
 void wxRadioBox::SetSelection(int n)
 {

-    wxCHECK_RET( IsValid(n), _T("invalid index in wxRadioBox::SetSelection") );
+    wxCHECK_RET( IsValid(n), wxT("invalid index in wxRadioBox::SetSelection") );

 
     m_selection = n;
 
 
     m_selection = n;
 


@@ -283,7 +259,7 @@ int wxRadioBox::GetSelection() const
 
 void wxRadioBox::SendRadioEvent()
 {
 
 void wxRadioBox::SendRadioEvent()
 {

-    wxCHECK_RET( m_selection != -1, _T("no active radio button") );
+    wxCHECK_RET( m_selection != -1, wxT("no active radio button") );

 
     wxCommandEvent event(wxEVT_COMMAND_RADIOBOX_SELECTED, GetId());
     InitCommandEvent(event);
 
     wxCommandEvent event(wxEVT_COMMAND_RADIOBOX_SELECTED, GetId());
     InitCommandEvent(event);


@@ -296,7 +272,7 @@ void wxRadioBox::SendRadioEvent()
 void wxRadioBox::OnRadioButton(wxEvent& event)
 {
     int n = m_buttons.Index((wxRadioButton *)event.GetEventObject());
 void wxRadioBox::OnRadioButton(wxEvent& event)
 {
     int n = m_buttons.Index((wxRadioButton *)event.GetEventObject());

-    wxCHECK_RET( n != wxNOT_FOUND, _T("click from alien radio button") );
+    wxCHECK_RET( n != wxNOT_FOUND, wxT("click from alien radio button") );

 
     m_selection = n;
 
 
     m_selection = n;
 


@@ -307,33 +283,47 @@ void wxRadioBox::OnRadioButton(wxEvent& event)
 // methods forwarded to the buttons
 // ----------------------------------------------------------------------------
 
 // methods forwarded to the buttons
 // ----------------------------------------------------------------------------
 

-wxString wxRadioBox::GetString(int n) const
+wxString wxRadioBox::GetString(unsigned int n) const

 {
     wxCHECK_MSG( IsValid(n), wxEmptyString,
 {
     wxCHECK_MSG( IsValid(n), wxEmptyString,

-                 _T("invalid index in wxRadioBox::GetString") );
+                 wxT("invalid index in wxRadioBox::GetString") );

 
     return m_buttons[n]->GetLabel();
 }
 
 
     return m_buttons[n]->GetLabel();
 }
 

-void wxRadioBox::SetString(int n, const wxString& label)
+void wxRadioBox::SetString(unsigned int n, const wxString& label)

 {
 {

-    wxCHECK_RET( IsValid(n), _T("invalid index in wxRadioBox::SetString") );
+    wxCHECK_RET( IsValid(n), wxT("invalid index in wxRadioBox::SetString") );

 
     m_buttons[n]->SetLabel(label);
 }
 
 
     m_buttons[n]->SetLabel(label);
 }
 

-bool wxRadioBox::Enable(int n, bool enable)
+bool wxRadioBox::Enable(unsigned int n, bool enable)

 {
 {

-    wxCHECK_MSG( IsValid(n), false, _T("invalid index in wxRadioBox::Enable") );
+    wxCHECK_MSG( IsValid(n), false, wxT("invalid index in wxRadioBox::Enable") );

 
     return m_buttons[n]->Enable(enable);
 }
 
 
     return m_buttons[n]->Enable(enable);
 }
 

-void wxRadioBox::Show(int n, bool show)
+bool wxRadioBox::IsItemEnabled(unsigned int n) const
+{
+    wxCHECK_MSG( IsValid(n), false, wxT("invalid index in wxRadioBox::IsItemEnabled") );
+
+    return m_buttons[n]->IsEnabled();
+}
+
+bool wxRadioBox::Show(unsigned int n, bool show)
+{
+    wxCHECK_MSG( IsValid(n), false, wxT("invalid index in wxRadioBox::Show") );
+
+    return m_buttons[n]->Show(show);
+}
+
+bool wxRadioBox::IsItemShown(unsigned int n) const

 {
 {

-    wxCHECK_RET( IsValid(n), _T("invalid index in wxRadioBox::Show") );
+    wxCHECK_MSG( IsValid(n), false, wxT("invalid index in wxRadioBox::IsItemShown") );

 
 

-    m_buttons[n]->Show(show);
+    return m_buttons[n]->IsShown();

 }
 
 // ----------------------------------------------------------------------------
 }
 
 // ----------------------------------------------------------------------------


@@ -346,8 +336,8 @@ bool wxRadioBox::Enable(bool enable)
         return false;
 
     // also enable/disable the buttons
         return false;
 
     // also enable/disable the buttons

-    size_t count = m_buttons.GetCount();
-    for ( size_t n = 0; n < count; n++ )
+    const unsigned int count = m_buttons.GetCount();
+    for ( unsigned int n = 0; n < count; n++ )

     {
         Enable(n, enable);
     }
     {
         Enable(n, enable);
     }


@@ -361,8 +351,8 @@ bool wxRadioBox::Show(bool show)
         return false;
 
     // also show/hide the buttons
         return false;
 
     // also show/hide the buttons

-    size_t count = m_buttons.GetCount();
-    for ( size_t n = 0; n < count; n++ )
+    const unsigned int count = m_buttons.GetCount();
+    for ( unsigned int n = 0; n < count; n++ )

     {
         Show(n, show);
     }
     {
         Show(n, show);
     }


@@ -386,8 +376,8 @@ void wxRadioBox::DoSetToolTip(wxToolTip *tooltip)
     wxControl::DoSetToolTip(tooltip);
 
     // Also set them for all Radio Buttons
     wxControl::DoSetToolTip(tooltip);
 
     // Also set them for all Radio Buttons

-    size_t count = m_buttons.GetCount();
-    for ( size_t n = 0; n < count; n++ )
+    const unsigned int count = m_buttons.GetCount();
+    for ( unsigned int n = 0; n < count; n++ )

     {
         if (tooltip)
             m_buttons[n]->SetToolTip(tooltip->GetTip());
     {
         if (tooltip)
             m_buttons[n]->SetToolTip(tooltip->GetTip());


@@ -403,11 +393,11 @@ void wxRadioBox::DoSetToolTip(wxToolTip *tooltip)
 
 wxSize wxRadioBox::GetMaxButtonSize() const
 {
 
 wxSize wxRadioBox::GetMaxButtonSize() const
 {

-    int widthMax, heightMax, width, height;
+    int widthMax, heightMax, width = 0, height = 0;

     widthMax = heightMax = 0;
 
     widthMax = heightMax = 0;
 

-    int count = GetCount();
-    for ( int n = 0; n < count; n++ )
+    const unsigned int count = GetCount();
+    for ( unsigned int n = 0; n < count; n++ )

     {
         m_buttons[n]->GetBestSize(&width, &height);
 
     {
         m_buttons[n]->GetBestSize(&width, &height);
 


@@ -424,8 +414,8 @@ wxSize wxRadioBox::DoGetBestClientSize() const
 {
     wxSize sizeBtn = GetMaxButtonSize();
 
 {
     wxSize sizeBtn = GetMaxButtonSize();
 

-    sizeBtn.x *= m_numCols;
-    sizeBtn.y *= m_numRows;
+    sizeBtn.x *= GetColumnCount();
+    sizeBtn.y *= GetRowCount();

 
     // add a border around all buttons
     sizeBtn.x += 2*BOX_BORDER_X;
 
     // add a border around all buttons
     sizeBtn.x += 2*BOX_BORDER_X;


@@ -453,15 +443,15 @@ void wxRadioBox::DoMoveWindow(int x0, int y0, int width, int height)
     int x = x0,
         y = y0;
 
     int x = x0,
         y = y0;
 

-    int count = GetCount();
-    for ( int n = 0; n < count; n++ )
+    const unsigned int count = GetCount();
+    for ( unsigned int n = 0; n < count; n++ )

     {
         m_buttons[n]->SetSize(x, y, sizeBtn.x, sizeBtn.y);
 
         if ( GetWindowStyle() & wxRA_TOPTOBOTTOM )
         {
             // from top to bottom
     {
         m_buttons[n]->SetSize(x, y, sizeBtn.x, sizeBtn.y);
 
         if ( GetWindowStyle() & wxRA_TOPTOBOTTOM )
         {
             // from top to bottom

-            if ( (n + 1) % m_numRows )
+            if ( (n + 1) % GetRowCount() )

             {
                 // continue in this column
                 y += sizeBtn.y;
             {
                 // continue in this column
                 y += sizeBtn.y;


@@ -476,7 +466,7 @@ void wxRadioBox::DoMoveWindow(int x0, int y0, int width, int height)
         else // wxRA_LEFTTORIGHT: mirror the code above
         {
             // from left to right
         else // wxRA_LEFTTORIGHT: mirror the code above
         {
             // from left to right

-            if ( (n + 1) % m_numCols )
+            if ( (n + 1) % GetColumnCount() )

             {
                 // continue in this row
                 x += sizeBtn.x;
             {
                 // continue in this row
                 x += sizeBtn.x;


@@ -534,4 +524,3 @@ bool wxRadioBox::OnKeyDown(wxKeyEvent& event)
 }
 
 #endif // wxUSE_RADIOBOX
 }
 
 #endif // wxUSE_RADIOBOX

-