Rewrote wxRadioBox (recompile)

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@713 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

diff --git a/include/wx/gtk/radiobox.h b/include/wx/gtk/radiobox.h
index 5fb8792ba24ecb406dced7085a381f895dcf56a8..61da2f7d2ac01bf5308a24ae70aac35f03eed9bf 100644 (file)
--- a/include/wx/gtk/radiobox.h
+++ b/include/wx/gtk/radiobox.h
@@ -55,6 +55,7 @@ class wxRadioBox: public wxControl
     {
       Create( parent, id, title, pos, size, n, choices, majorDim, style, val, name );
     }
     {
       Create( parent, id, title, pos, size, n, choices, majorDim, style, val, name );
     }

+    ~wxRadioBox(void);

     bool Create( wxWindow *parent, wxWindowID id, const wxString& title,
              const wxPoint& pos = wxDefaultPosition, const wxSize& size = wxDefaultSize,
              int n = 0, const wxString choices[] = (const wxString *) NULL,
     bool Create( wxWindow *parent, wxWindowID id, const wxString& title,
              const wxPoint& pos = wxDefaultPosition, const wxSize& size = wxDefaultSize,
              int n = 0, const wxString choices[] = (const wxString *) NULL,


@@ -86,14 +87,10 @@ class wxRadioBox: public wxControl
     
     bool IsOwnGtkWindow( GdkWindow *window );
     
     
     bool IsOwnGtkWindow( GdkWindow *window );
     

-    bool            m_alreadySent;
-    
-  private:
-  
-    GtkRadioButton *m_radio;
+    bool             m_alreadySent;
+    wxList           m_boxes;

     
   DECLARE_EVENT_TABLE()    
     
   DECLARE_EVENT_TABLE()    

-    

 };
 
 #endif // __GTKRADIOBOXH__
 };
 
 #endif // __GTKRADIOBOXH__

diff --git a/include/wx/gtk1/radiobox.h b/include/wx/gtk1/radiobox.h
index 5fb8792ba24ecb406dced7085a381f895dcf56a8..61da2f7d2ac01bf5308a24ae70aac35f03eed9bf 100644 (file)
--- a/include/wx/gtk1/radiobox.h
+++ b/include/wx/gtk1/radiobox.h
@@ -55,6 +55,7 @@ class wxRadioBox: public wxControl
     {
       Create( parent, id, title, pos, size, n, choices, majorDim, style, val, name );
     }
     {
       Create( parent, id, title, pos, size, n, choices, majorDim, style, val, name );
     }

+    ~wxRadioBox(void);

     bool Create( wxWindow *parent, wxWindowID id, const wxString& title,
              const wxPoint& pos = wxDefaultPosition, const wxSize& size = wxDefaultSize,
              int n = 0, const wxString choices[] = (const wxString *) NULL,
     bool Create( wxWindow *parent, wxWindowID id, const wxString& title,
              const wxPoint& pos = wxDefaultPosition, const wxSize& size = wxDefaultSize,
              int n = 0, const wxString choices[] = (const wxString *) NULL,


@@ -86,14 +87,10 @@ class wxRadioBox: public wxControl
     
     bool IsOwnGtkWindow( GdkWindow *window );
     
     
     bool IsOwnGtkWindow( GdkWindow *window );
     

-    bool            m_alreadySent;
-    
-  private:
-  
-    GtkRadioButton *m_radio;
+    bool             m_alreadySent;
+    wxList           m_boxes;

     
   DECLARE_EVENT_TABLE()    
     
   DECLARE_EVENT_TABLE()    

-    

 };
 
 #endif // __GTKRADIOBOXH__
 };
 
 #endif // __GTKRADIOBOXH__

diff --git a/src/gtk/radiobox.cpp b/src/gtk/radiobox.cpp
index ec26307409e22c4d89311c350befde294841c677..805ac0a2cac114aed6c4dc9d12ed8d32cad83f27 100644 (file)
--- a/src/gtk/radiobox.cpp
+++ b/src/gtk/radiobox.cpp
@@ -82,6 +82,8 @@ bool wxRadioBox::Create( wxWindow *parent, wxWindowID id, const wxString& title,
   int maxLen = 0;
   int height = 20;
   
   int maxLen = 0;
   int height = 20;
   

+  GtkRadioButton *m_radio = (GtkRadioButton*) NULL;
+  

 //  if (((m_style & wxRA_VERTICAL) == wxRA_VERTICAL) && (n > 0))
   if (n > 0)
   {
 //  if (((m_style & wxRA_VERTICAL) == wxRA_VERTICAL) && (n > 0))
   if (n > 0)
   {


@@ -92,6 +94,8 @@ bool wxRadioBox::Create( wxWindow *parent, wxWindowID id, const wxString& title,
       
       m_radio = GTK_RADIO_BUTTON( gtk_radio_button_new_with_label( radio_button_group, choices[i] ) );
       
       
       m_radio = GTK_RADIO_BUTTON( gtk_radio_button_new_with_label( radio_button_group, choices[i] ) );
       

+      m_boxes.Append( (wxObject*) m_radio );
+      

       ConnectWidget( GTK_WIDGET(m_radio) );
   
       if (!i) gtk_toggle_button_set_state( GTK_TOGGLE_BUTTON(m_radio), TRUE );
       ConnectWidget( GTK_WIDGET(m_radio) );
   
       if (!i) gtk_toggle_button_set_state( GTK_TOGGLE_BUTTON(m_radio), TRUE );


@@ -113,7 +117,7 @@ bool wxRadioBox::Create( wxWindow *parent, wxWindowID id, const wxString& title,
       
     }
   }
       
     }
   }

-
+  

   wxSize newSize = size;
   if (newSize.x == -1) newSize.x = maxLen+10;
   if (newSize.y == -1) newSize.y = height;
   wxSize newSize = size;
   if (newSize.x == -1) newSize.x = maxLen+10;
   if (newSize.y == -1) newSize.y = height;


@@ -121,11 +125,24 @@ bool wxRadioBox::Create( wxWindow *parent, wxWindowID id, const wxString& title,
   
   PostCreation();
   
   
   PostCreation();
   

+  SetLabel( title );
+  

   Show( TRUE );
     
   return TRUE;
 }
 
   Show( TRUE );
     
   return TRUE;
 }
 

+wxRadioBox::~wxRadioBox(void)
+{
+  wxNode *node = m_boxes.First();
+  while (node)
+  {
+    GtkWidget *button = GTK_WIDGET( node->Data() );
+    gtk_widget_destroy( button );
+    node = node->Next();
+  }
+}
+

 void wxRadioBox::OnSize( wxSizeEvent &event )
 {
   wxControl::OnSize( event );
 void wxRadioBox::OnSize( wxSizeEvent &event )
 {
   wxControl::OnSize( event );


@@ -133,16 +150,15 @@ void wxRadioBox::OnSize( wxSizeEvent &event )
   int x = m_x+5;
   int y = m_y+15;
   
   int x = m_x+5;
   int y = m_y+15;
   

-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkWidget *button = GTK_WIDGET( item->data );
+    GtkWidget *button = GTK_WIDGET( node->Data() );

     
     gtk_myfixed_move( GTK_MYFIXED(m_parent->m_wxwindow), button, x, y );
     
     gtk_myfixed_move( GTK_MYFIXED(m_parent->m_wxwindow), button, x, y );

-    

     y += 20;
     
     y += 20;
     

-    item = item->next;
+    node = node->Next();

   }
 }
 
   }
 }
 


@@ -150,12 +166,14 @@ bool wxRadioBox::Show( bool show )
 {
   wxWindow::Show( show );
 
 {
   wxWindow::Show( show );
 

-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkWidget *w = GTK_WIDGET( item->data );
-    if (show) gtk_widget_show( w ); else gtk_widget_hide( w );
-    item = item->next;
+    GtkWidget *button = GTK_WIDGET( node->Data() );
+    
+    if (show) gtk_widget_show( button ); else gtk_widget_hide( button );
+    
+    node = node->Next();

   }
 
   return TRUE;
   }
 
   return TRUE;


@@ -163,67 +181,67 @@ bool wxRadioBox::Show( bool show )
 
 int wxRadioBox::FindString( const wxString &s ) const
 {
 
 int wxRadioBox::FindString( const wxString &s ) const
 {

-  GSList *item = gtk_radio_button_group( m_radio );
-  
-  int count = g_slist_length(item)-1;
+  int count = 0;

   
   

-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkButton *b = GTK_BUTTON( item->data );
-    GtkLabel *l = GTK_LABEL( b->child );
-    if (s == l->label) return count;
-    count--;
-    item = item->next;
+    GtkButton *button = GTK_BUTTON( node->Data() );
+    
+    GtkLabel *label = GTK_LABEL( button->child );
+    if (s == label->label) return count;
+    count++;
+    
+    node = node->Next();

   }
   }

-
+  

   return -1;
 }
 
 void wxRadioBox::SetSelection( int n )
 {
   return -1;
 }
 
 void wxRadioBox::SetSelection( int n )
 {

-  GSList *item = gtk_radio_button_group( m_radio );
-  item = g_slist_nth( item, g_slist_length(item)-n-1 );
+  wxNode *node = m_boxes.Nth( n );

   
   

-  if (!item)
+  if (!node)

   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   
   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   

-  GtkToggleButton *button = GTK_TOGGLE_BUTTON( item->data );
+  GtkToggleButton *button = GTK_TOGGLE_BUTTON( node->Data() );

   
   gtk_toggle_button_set_state( button, 1 );
 }
 
 int wxRadioBox::GetSelection(void) const
 {
   
   gtk_toggle_button_set_state( button, 1 );
 }
 
 int wxRadioBox::GetSelection(void) const
 {

-  GSList *item = gtk_radio_button_group( m_radio );

   int count = 0;
   int count = 0;

-  int found = -1;
-  while (item)
+  
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkButton *button = GTK_BUTTON( item->data );
-    if (GTK_TOGGLE_BUTTON(button)->active) found = count;
+    GtkToggleButton *button = GTK_TOGGLE_BUTTON( node->Data() );
+    if (button->active) return count;

     count++;
     count++;

-    item = item->next;
+    node = node->Next();

   }
   
   }
   

-  return found != -1 ? count-found-1 : -1;
+  wxFAIL_MSG( "wxRadioBox none selected" );
+  
+  return -1;

 }
 
 wxString wxRadioBox::GetString( int n ) const
 {
 }
 
 wxString wxRadioBox::GetString( int n ) const
 {

-  GSList *item = gtk_radio_button_group( m_radio );
-  
-  item = g_slist_nth( item, g_slist_length(item)-n-1 );
+  wxNode *node = m_boxes.Nth( n );

   
   

-  if (!item)
+  if (!node)

   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return "";
   }
   
   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return "";
   }
   

-  GtkButton *button = GTK_BUTTON( item->data );
+  GtkButton *button = GTK_BUTTON( node->Data() );

   GtkLabel *label = GTK_LABEL( button->child );
   
   return wxString( label->label );
   GtkLabel *label = GTK_LABEL( button->child );
   
   return wxString( label->label );


@@ -243,16 +261,15 @@ void wxRadioBox::SetLabel( const wxString& label )
 
 void wxRadioBox::SetLabel( int item, const wxString& label )
 {
 
 void wxRadioBox::SetLabel( int item, const wxString& label )
 {

-  GSList *list = gtk_radio_button_group( m_radio );
-  list = g_slist_nth( list, g_slist_length(list)-item-1 );
+  wxNode *node = m_boxes.Nth( item );

   
   

-  if (!list)
+  if (!node)

   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   
   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   

-  GtkButton *button = GTK_BUTTON( list->data );
+  GtkButton *button = GTK_BUTTON( node->Data() );

   GtkLabel *g_label = GTK_LABEL( button->child );
   
   gtk_label_set( g_label, label );
   GtkLabel *g_label = GTK_LABEL( button->child );
   
   gtk_label_set( g_label, label );


@@ -267,30 +284,28 @@ void wxRadioBox::Enable( bool enable )
 {
   wxControl::Enable( enable );
   
 {
   wxControl::Enable( enable );
   

-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkButton *button = GTK_BUTTON( item->data );
+    GtkButton *button = GTK_BUTTON( node->Data() );

     GtkWidget *label = button->child;
     gtk_widget_set_sensitive( GTK_WIDGET(button), enable );
     gtk_widget_set_sensitive( label, enable );
     GtkWidget *label = button->child;
     gtk_widget_set_sensitive( GTK_WIDGET(button), enable );
     gtk_widget_set_sensitive( label, enable );

-    item = item->next;
+    node = node->Next();

   }
 }
 
 void wxRadioBox::Enable( int item, bool enable )
 {
   }
 }
 
 void wxRadioBox::Enable( int item, bool enable )
 {

-  GSList *list = gtk_radio_button_group( m_radio );
-  list = g_slist_nth( list, g_slist_length(list)-item-1 );
+  wxNode *node = m_boxes.Nth( item );

   
   

-  if (!list)
+  if (!node)

   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   
   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   

-  GtkButton *button = GTK_BUTTON( list->data );
-  
+  GtkButton *button = GTK_BUTTON( node->Data() );

   GtkWidget *label = button->child;
   gtk_widget_set_sensitive( GTK_WIDGET(button), enable );
   gtk_widget_set_sensitive( label, enable );
   GtkWidget *label = button->child;
   gtk_widget_set_sensitive( GTK_WIDGET(button), enable );
   gtk_widget_set_sensitive( label, enable );


@@ -298,16 +313,15 @@ void wxRadioBox::Enable( int item, bool enable )
 
 void wxRadioBox::Show( int item, bool show )
 {
 
 void wxRadioBox::Show( int item, bool show )
 {

-  GSList *list = gtk_radio_button_group( m_radio );
-  list = g_slist_nth( list, g_slist_length(list)-item-1 );
+  wxNode *node = m_boxes.Nth( item );

   
   

-  if (!list)
+  if (!node)

   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   
   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   

-  GtkWidget *button = GTK_WIDGET( list->data );
+  GtkWidget *button = GTK_WIDGET( node->Data() );

 
   if (show)
     gtk_widget_show( button );
 
   if (show)
     gtk_widget_show( button );


@@ -317,17 +331,19 @@ void wxRadioBox::Show( int item, bool show )
 
 wxString wxRadioBox::GetStringSelection(void) const
 {
 
 wxString wxRadioBox::GetStringSelection(void) const
 {

-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkButton *button = GTK_BUTTON( item->data );
-    if (GTK_TOGGLE_BUTTON(button)->active)
+    GtkToggleButton *button = GTK_TOGGLE_BUTTON( node->Data() );
+    if (button->active)

     {
     {

-      GtkLabel *label = GTK_LABEL( button->child );
+      GtkLabel *label = GTK_LABEL( GTK_BUTTON(button)->child );

       return label->label;
     }
       return label->label;
     }

-    item = item->next;
+    node = node->Next();

   }
   }

+  
+  wxFAIL_MSG( "wxRadioBox none selected" );

   return "";
 }
 
   return "";
 }
 


@@ -341,14 +357,7 @@ bool wxRadioBox::SetStringSelection( const wxString&s )
 
 int wxRadioBox::Number(void) const
 {
 
 int wxRadioBox::Number(void) const
 {

-  int count = 0;
-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
-  {
-    item = item->next;
-    count++;
-  }
-  return count;
+  return m_boxes.Number();

 }
 
 int wxRadioBox::GetNumberOfRowsOrCols(void) const
 }
 
 int wxRadioBox::GetNumberOfRowsOrCols(void) const


@@ -365,16 +374,16 @@ void wxRadioBox::SetFont( const wxFont &font )
 {
   wxWindow::SetFont( font );
    
 {
   wxWindow::SetFont( font );
    

-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkButton *button = GTK_BUTTON( item->data );
+    GtkButton *button = GTK_BUTTON( node->Data() );

     
     gtk_widget_set_style( button->child, 
       gtk_style_ref(
         gtk_widget_get_style( m_widget ) ) ); 
     
     
     gtk_widget_set_style( button->child, 
       gtk_style_ref(
         gtk_widget_get_style( m_widget ) ) ); 
     

-    item = item->next;
+    node = node->Next();

   }
 }
 
   }
 }
 


@@ -382,12 +391,14 @@ bool wxRadioBox::IsOwnGtkWindow( GdkWindow *window )
 {
   if (window == m_widget->window) return TRUE;
   
 {
   if (window == m_widget->window) return TRUE;
   

-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkWidget *button = GTK_WIDGET( item->data );
+    GtkWidget *button = GTK_WIDGET( node->Data() );
+    

     if (window == button->window) return TRUE;
     if (window == button->window) return TRUE;

-    item = item->next;
+    
+    node = node->Next();

   }
   
   return FALSE;
   }
   
   return FALSE;

diff --git a/src/gtk1/radiobox.cpp b/src/gtk1/radiobox.cpp
index ec26307409e22c4d89311c350befde294841c677..805ac0a2cac114aed6c4dc9d12ed8d32cad83f27 100644 (file)
--- a/src/gtk1/radiobox.cpp
+++ b/src/gtk1/radiobox.cpp
@@ -82,6 +82,8 @@ bool wxRadioBox::Create( wxWindow *parent, wxWindowID id, const wxString& title,
   int maxLen = 0;
   int height = 20;
   
   int maxLen = 0;
   int height = 20;
   

+  GtkRadioButton *m_radio = (GtkRadioButton*) NULL;
+  

 //  if (((m_style & wxRA_VERTICAL) == wxRA_VERTICAL) && (n > 0))
   if (n > 0)
   {
 //  if (((m_style & wxRA_VERTICAL) == wxRA_VERTICAL) && (n > 0))
   if (n > 0)
   {


@@ -92,6 +94,8 @@ bool wxRadioBox::Create( wxWindow *parent, wxWindowID id, const wxString& title,
       
       m_radio = GTK_RADIO_BUTTON( gtk_radio_button_new_with_label( radio_button_group, choices[i] ) );
       
       
       m_radio = GTK_RADIO_BUTTON( gtk_radio_button_new_with_label( radio_button_group, choices[i] ) );
       

+      m_boxes.Append( (wxObject*) m_radio );
+      

       ConnectWidget( GTK_WIDGET(m_radio) );
   
       if (!i) gtk_toggle_button_set_state( GTK_TOGGLE_BUTTON(m_radio), TRUE );
       ConnectWidget( GTK_WIDGET(m_radio) );
   
       if (!i) gtk_toggle_button_set_state( GTK_TOGGLE_BUTTON(m_radio), TRUE );


@@ -113,7 +117,7 @@ bool wxRadioBox::Create( wxWindow *parent, wxWindowID id, const wxString& title,
       
     }
   }
       
     }
   }

-
+  

   wxSize newSize = size;
   if (newSize.x == -1) newSize.x = maxLen+10;
   if (newSize.y == -1) newSize.y = height;
   wxSize newSize = size;
   if (newSize.x == -1) newSize.x = maxLen+10;
   if (newSize.y == -1) newSize.y = height;


@@ -121,11 +125,24 @@ bool wxRadioBox::Create( wxWindow *parent, wxWindowID id, const wxString& title,
   
   PostCreation();
   
   
   PostCreation();
   

+  SetLabel( title );
+  

   Show( TRUE );
     
   return TRUE;
 }
 
   Show( TRUE );
     
   return TRUE;
 }
 

+wxRadioBox::~wxRadioBox(void)
+{
+  wxNode *node = m_boxes.First();
+  while (node)
+  {
+    GtkWidget *button = GTK_WIDGET( node->Data() );
+    gtk_widget_destroy( button );
+    node = node->Next();
+  }
+}
+

 void wxRadioBox::OnSize( wxSizeEvent &event )
 {
   wxControl::OnSize( event );
 void wxRadioBox::OnSize( wxSizeEvent &event )
 {
   wxControl::OnSize( event );


@@ -133,16 +150,15 @@ void wxRadioBox::OnSize( wxSizeEvent &event )
   int x = m_x+5;
   int y = m_y+15;
   
   int x = m_x+5;
   int y = m_y+15;
   

-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkWidget *button = GTK_WIDGET( item->data );
+    GtkWidget *button = GTK_WIDGET( node->Data() );

     
     gtk_myfixed_move( GTK_MYFIXED(m_parent->m_wxwindow), button, x, y );
     
     gtk_myfixed_move( GTK_MYFIXED(m_parent->m_wxwindow), button, x, y );

-    

     y += 20;
     
     y += 20;
     

-    item = item->next;
+    node = node->Next();

   }
 }
 
   }
 }
 


@@ -150,12 +166,14 @@ bool wxRadioBox::Show( bool show )
 {
   wxWindow::Show( show );
 
 {
   wxWindow::Show( show );
 

-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkWidget *w = GTK_WIDGET( item->data );
-    if (show) gtk_widget_show( w ); else gtk_widget_hide( w );
-    item = item->next;
+    GtkWidget *button = GTK_WIDGET( node->Data() );
+    
+    if (show) gtk_widget_show( button ); else gtk_widget_hide( button );
+    
+    node = node->Next();

   }
 
   return TRUE;
   }
 
   return TRUE;


@@ -163,67 +181,67 @@ bool wxRadioBox::Show( bool show )
 
 int wxRadioBox::FindString( const wxString &s ) const
 {
 
 int wxRadioBox::FindString( const wxString &s ) const
 {

-  GSList *item = gtk_radio_button_group( m_radio );
-  
-  int count = g_slist_length(item)-1;
+  int count = 0;

   
   

-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkButton *b = GTK_BUTTON( item->data );
-    GtkLabel *l = GTK_LABEL( b->child );
-    if (s == l->label) return count;
-    count--;
-    item = item->next;
+    GtkButton *button = GTK_BUTTON( node->Data() );
+    
+    GtkLabel *label = GTK_LABEL( button->child );
+    if (s == label->label) return count;
+    count++;
+    
+    node = node->Next();

   }
   }

-
+  

   return -1;
 }
 
 void wxRadioBox::SetSelection( int n )
 {
   return -1;
 }
 
 void wxRadioBox::SetSelection( int n )
 {

-  GSList *item = gtk_radio_button_group( m_radio );
-  item = g_slist_nth( item, g_slist_length(item)-n-1 );
+  wxNode *node = m_boxes.Nth( n );

   
   

-  if (!item)
+  if (!node)

   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   
   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   

-  GtkToggleButton *button = GTK_TOGGLE_BUTTON( item->data );
+  GtkToggleButton *button = GTK_TOGGLE_BUTTON( node->Data() );

   
   gtk_toggle_button_set_state( button, 1 );
 }
 
 int wxRadioBox::GetSelection(void) const
 {
   
   gtk_toggle_button_set_state( button, 1 );
 }
 
 int wxRadioBox::GetSelection(void) const
 {

-  GSList *item = gtk_radio_button_group( m_radio );

   int count = 0;
   int count = 0;

-  int found = -1;
-  while (item)
+  
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkButton *button = GTK_BUTTON( item->data );
-    if (GTK_TOGGLE_BUTTON(button)->active) found = count;
+    GtkToggleButton *button = GTK_TOGGLE_BUTTON( node->Data() );
+    if (button->active) return count;

     count++;
     count++;

-    item = item->next;
+    node = node->Next();

   }
   
   }
   

-  return found != -1 ? count-found-1 : -1;
+  wxFAIL_MSG( "wxRadioBox none selected" );
+  
+  return -1;

 }
 
 wxString wxRadioBox::GetString( int n ) const
 {
 }
 
 wxString wxRadioBox::GetString( int n ) const
 {

-  GSList *item = gtk_radio_button_group( m_radio );
-  
-  item = g_slist_nth( item, g_slist_length(item)-n-1 );
+  wxNode *node = m_boxes.Nth( n );

   
   

-  if (!item)
+  if (!node)

   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return "";
   }
   
   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return "";
   }
   

-  GtkButton *button = GTK_BUTTON( item->data );
+  GtkButton *button = GTK_BUTTON( node->Data() );

   GtkLabel *label = GTK_LABEL( button->child );
   
   return wxString( label->label );
   GtkLabel *label = GTK_LABEL( button->child );
   
   return wxString( label->label );


@@ -243,16 +261,15 @@ void wxRadioBox::SetLabel( const wxString& label )
 
 void wxRadioBox::SetLabel( int item, const wxString& label )
 {
 
 void wxRadioBox::SetLabel( int item, const wxString& label )
 {

-  GSList *list = gtk_radio_button_group( m_radio );
-  list = g_slist_nth( list, g_slist_length(list)-item-1 );
+  wxNode *node = m_boxes.Nth( item );

   
   

-  if (!list)
+  if (!node)

   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   
   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   

-  GtkButton *button = GTK_BUTTON( list->data );
+  GtkButton *button = GTK_BUTTON( node->Data() );

   GtkLabel *g_label = GTK_LABEL( button->child );
   
   gtk_label_set( g_label, label );
   GtkLabel *g_label = GTK_LABEL( button->child );
   
   gtk_label_set( g_label, label );


@@ -267,30 +284,28 @@ void wxRadioBox::Enable( bool enable )
 {
   wxControl::Enable( enable );
   
 {
   wxControl::Enable( enable );
   

-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkButton *button = GTK_BUTTON( item->data );
+    GtkButton *button = GTK_BUTTON( node->Data() );

     GtkWidget *label = button->child;
     gtk_widget_set_sensitive( GTK_WIDGET(button), enable );
     gtk_widget_set_sensitive( label, enable );
     GtkWidget *label = button->child;
     gtk_widget_set_sensitive( GTK_WIDGET(button), enable );
     gtk_widget_set_sensitive( label, enable );

-    item = item->next;
+    node = node->Next();

   }
 }
 
 void wxRadioBox::Enable( int item, bool enable )
 {
   }
 }
 
 void wxRadioBox::Enable( int item, bool enable )
 {

-  GSList *list = gtk_radio_button_group( m_radio );
-  list = g_slist_nth( list, g_slist_length(list)-item-1 );
+  wxNode *node = m_boxes.Nth( item );

   
   

-  if (!list)
+  if (!node)

   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   
   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   

-  GtkButton *button = GTK_BUTTON( list->data );
-  
+  GtkButton *button = GTK_BUTTON( node->Data() );

   GtkWidget *label = button->child;
   gtk_widget_set_sensitive( GTK_WIDGET(button), enable );
   gtk_widget_set_sensitive( label, enable );
   GtkWidget *label = button->child;
   gtk_widget_set_sensitive( GTK_WIDGET(button), enable );
   gtk_widget_set_sensitive( label, enable );


@@ -298,16 +313,15 @@ void wxRadioBox::Enable( int item, bool enable )
 
 void wxRadioBox::Show( int item, bool show )
 {
 
 void wxRadioBox::Show( int item, bool show )
 {

-  GSList *list = gtk_radio_button_group( m_radio );
-  list = g_slist_nth( list, g_slist_length(list)-item-1 );
+  wxNode *node = m_boxes.Nth( item );

   
   

-  if (!list)
+  if (!node)

   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   
   {
     wxFAIL_MSG( "wxRadioBox wrong index" );
     return;
   }
   

-  GtkWidget *button = GTK_WIDGET( list->data );
+  GtkWidget *button = GTK_WIDGET( node->Data() );

 
   if (show)
     gtk_widget_show( button );
 
   if (show)
     gtk_widget_show( button );


@@ -317,17 +331,19 @@ void wxRadioBox::Show( int item, bool show )
 
 wxString wxRadioBox::GetStringSelection(void) const
 {
 
 wxString wxRadioBox::GetStringSelection(void) const
 {

-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkButton *button = GTK_BUTTON( item->data );
-    if (GTK_TOGGLE_BUTTON(button)->active)
+    GtkToggleButton *button = GTK_TOGGLE_BUTTON( node->Data() );
+    if (button->active)

     {
     {

-      GtkLabel *label = GTK_LABEL( button->child );
+      GtkLabel *label = GTK_LABEL( GTK_BUTTON(button)->child );

       return label->label;
     }
       return label->label;
     }

-    item = item->next;
+    node = node->Next();

   }
   }

+  
+  wxFAIL_MSG( "wxRadioBox none selected" );

   return "";
 }
 
   return "";
 }
 


@@ -341,14 +357,7 @@ bool wxRadioBox::SetStringSelection( const wxString&s )
 
 int wxRadioBox::Number(void) const
 {
 
 int wxRadioBox::Number(void) const
 {

-  int count = 0;
-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
-  {
-    item = item->next;
-    count++;
-  }
-  return count;
+  return m_boxes.Number();

 }
 
 int wxRadioBox::GetNumberOfRowsOrCols(void) const
 }
 
 int wxRadioBox::GetNumberOfRowsOrCols(void) const


@@ -365,16 +374,16 @@ void wxRadioBox::SetFont( const wxFont &font )
 {
   wxWindow::SetFont( font );
    
 {
   wxWindow::SetFont( font );
    

-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkButton *button = GTK_BUTTON( item->data );
+    GtkButton *button = GTK_BUTTON( node->Data() );

     
     gtk_widget_set_style( button->child, 
       gtk_style_ref(
         gtk_widget_get_style( m_widget ) ) ); 
     
     
     gtk_widget_set_style( button->child, 
       gtk_style_ref(
         gtk_widget_get_style( m_widget ) ) ); 
     

-    item = item->next;
+    node = node->Next();

   }
 }
 
   }
 }
 


@@ -382,12 +391,14 @@ bool wxRadioBox::IsOwnGtkWindow( GdkWindow *window )
 {
   if (window == m_widget->window) return TRUE;
   
 {
   if (window == m_widget->window) return TRUE;
   

-  GSList *item = gtk_radio_button_group( m_radio );
-  while (item)
+  wxNode *node = m_boxes.First();
+  while (node)

   {
   {

-    GtkWidget *button = GTK_WIDGET( item->data );
+    GtkWidget *button = GTK_WIDGET( node->Data() );
+    

     if (window == button->window) return TRUE;
     if (window == button->window) return TRUE;

-    item = item->next;
+    
+    node = node->Next();

   }
   
   return FALSE;
   }
   
   return FALSE;