return NULL from the functions returning a pointer, not FALSE (patch 544557)

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

diff --git a/src/univ/radiobox.cpp b/src/univ/radiobox.cpp
index 342374f9d28710befe6fe18f2521ac98c16f137a..249fba0b252985e37445d3eed54f367c1842e2bd 100644 (file)
--- a/src/univ/radiobox.cpp
+++ b/src/univ/radiobox.cpp
@@ -303,12 +303,32 @@ void wxRadioBox::Show(int n, bool show)
 
 bool wxRadioBox::Enable(bool enable)
 {
-    return wxStaticBox::Enable(enable);
+    if ( !wxStaticBox::Enable(enable) )
+        return FALSE;
+
+    // also enable/disable the buttons
+    size_t count = m_buttons.GetCount();
+    for ( size_t n = 0; n < count; n++ )
+    {
+        Enable(n, enable);
+    }
+
+    return TRUE;
 }
 
 bool wxRadioBox::Show(bool show)
 {
-    return wxStaticBox::Show(show);
+    if ( !wxStaticBox::Show(show) )
+        return FALSE;
+
+    // also show/hide the buttons
+    size_t count = m_buttons.GetCount();
+    for ( size_t n = 0; n < count; n++ )
+    {
+        Show(n, show);
+    }
+
+    return TRUE;
 }
 
 wxString wxRadioBox::GetLabel() const
@@ -369,9 +389,10 @@ void wxRadioBox::DoMoveWindow(int x0, int y0, int width, int height)
 
     wxSize sizeBtn = GetMaxButtonSize();
     wxPoint ptOrigin = GetBoxAreaOrigin();
+    wxPoint clientOrigin = GetParent() ? GetParent()->GetClientAreaOrigin() : wxPoint(0,0);
 
-    x0 += ptOrigin.x + BOX_BORDER_X;
-    y0 += ptOrigin.y + BOX_BORDER_Y;
+    x0 += ptOrigin.x + BOX_BORDER_X - clientOrigin.x;
+    y0 += ptOrigin.y + BOX_BORDER_Y - clientOrigin.y;
 
     int x = x0,
         y = y0;