Corrected some things in, and some thing revealed by

[wxWidgets.git] / src / gtk / control.cpp

diff --git a/src/gtk/control.cpp b/src/gtk/control.cpp
index 3c7f89d95b8af92647bf133ba04f7b71d9a93bdd..d8f6b14c9d4072fee1fd8a99d6ce8b5b2d99f783 100644 (file)
--- a/src/gtk/control.cpp
+++ b/src/gtk/control.cpp
@@ -2,9 +2,8 @@
 // Name:        control.cpp
 // Purpose:
 // Author:      Robert Roebling
-// Created:     01/02/97
-// Id:
-// Copyright:   (c) 1998 Robert Roebling, Julian Smart and Markus Holzem
+// Id:          $Id$
+// Copyright:   (c) 1998 Robert Roebling, Julian Smart and Vadim Zeitlin
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
@@ -14,47 +13,76 @@
 
 #include "wx/control.h"
 
+#include <gtk/gtk.h>
+
 //-----------------------------------------------------------------------------
 // wxControl
 //-----------------------------------------------------------------------------
 
-IMPLEMENT_DYNAMIC_CLASS(wxControl,wxWindow)
+IMPLEMENT_DYNAMIC_CLASS(wxControl, wxWindow)
 
-wxControl::wxControl(void)
+wxControl::wxControl()
 {
-  m_needParent = TRUE;
-};
+    m_needParent = TRUE;
+}
 
-wxControl::wxControl( wxWindow *parent, wxWindowID id,
-      const wxPoint &pos, const wxSize &size,
-      long style, const wxString &name ) :
-  wxWindow( parent, id, pos, size, style, name )
+wxControl::wxControl( wxWindow *parent,
+                      wxWindowID id,
+                      const wxPoint &pos,
+                      const wxSize &size,
+                      long style,
+                      const wxString &name )
 {
-};
+    (void)Create(parent, id, pos, size, style, name);
+}
 
-void wxControl::Command( wxCommandEvent &WXUNUSED(event) )
+#if wxUSE_VALIDATORS
+wxControl::wxControl( wxWindow *parent,
+                      wxWindowID id,
+                      const wxPoint &pos,
+                      const wxSize &size,
+                      long style,
+                      const wxValidator& validator,
+                      const wxString &name)
 {
-};
+    (void)Create(parent, id, pos, size, style, name);
+    SetValidator(validator);
+}
+#endif
 
 void wxControl::SetLabel( const wxString &label )
 {
-  for ( const char *pc = label; *pc != '\0'; pc++ ) {
-    if ( *pc == '&' ) {
-      pc++; // skip it
+    m_label.Empty();
+    for ( const wxChar *pc = label; *pc != wxT('\0'); pc++ )
+    {
+        if ( *pc == wxT('&') )
+        {
+            pc++; // skip it
 #if 0 // it would be unused anyhow for now - kbd interface not done yet
-      if ( *pc != '&' )
-        m_chAccel = *pc;
+            if ( *pc != wxT('&') ) m_chAccel = *pc;
 #endif
+        }
+        m_label << *pc;
     }
+}
+
+wxString wxControl::GetLabel() const
+{
+    return m_label;
+}
 
-    m_label << *pc;
-  }
-};
 
-wxString wxControl::GetLabel(void) const
+wxSize wxControl::DoGetBestSize() const
 {
-  return m_label;
-};
+    // Do not return any arbitrary default value...
+    wxASSERT_MSG( m_widget, wxT("DoGetBestSize called before creation") );
 
+    GtkRequisition req;
+    req.width = 2;
+    req.height = 2;
+    (* GTK_WIDGET_CLASS( GTK_OBJECT(m_widget)->klass )->size_request )
+        (m_widget, &req );
 
+    return wxSize(req.width, req.height);
+}