added assert to check that notebook page does have notebook as parent; removed duplic...

[wxWidgets.git] / src / motif / combobox.cpp

diff --git a/src/motif/combobox.cpp b/src/motif/combobox.cpp
index 36e322073f2c621e40ec0622533fa35e6f5adf14..fb3e685bf18551e56a69ec3a09897063993a3b66 100644 (file)
--- a/src/motif/combobox.cpp
+++ b/src/motif/combobox.cpp
@@ -13,10 +13,12 @@
 #pragma implementation "combobox.h"
 #endif
 
 #pragma implementation "combobox.h"
 #endif
 

-#include "wx/combobox.h"
+#include "wx/setup.h"

 
 #if wxUSE_COMBOBOX
 
 
 #if wxUSE_COMBOBOX
 

+#include "wx/combobox.h"
+

 #ifdef __VMS__
 #pragma message disable nosimpint
 #endif
 #ifdef __VMS__
 #pragma message disable nosimpint
 #endif


@@ -24,6 +26,10 @@
 #ifdef __VMS__
 #pragma message enable nosimpint
 #endif
 #ifdef __VMS__
 #pragma message enable nosimpint
 #endif

+
+// use the old, GPL'd combobox
+#if (XmVersion < 2000)
+

 #include "xmcombo/xmcombo.h"
 
 #include "wx/motif/private.h"
 #include "xmcombo/xmcombo.h"
 
 #include "wx/motif/private.h"


@@ -120,10 +126,16 @@ void wxComboBox::SetValue(const wxString& value)
 {
     m_inSetValue = TRUE;
     if( !value.empty() )
 {
     m_inSetValue = TRUE;
     if( !value.empty() )

-        XmComboBoxSetString( (Widget)m_mainWidget, (char*)value.c_str() );
+        XmComboBoxSetString( (Widget)m_mainWidget,
+                             wxConstCast(value.c_str(), char) );

     m_inSetValue = FALSE;
 }
 
     m_inSetValue = FALSE;
 }
 

+void wxComboBox::SetString(int n, const wxString& s)
+{
+    wxFAIL_MSG( wxT("wxComboBox::SetString only implemented for Motif 2.0") );
+}
+

 int wxComboBox::DoAppend(const wxString& item)
 {
     wxXmString str( item.c_str() );
 int wxComboBox::DoAppend(const wxString& item)
 {
     wxXmString str( item.c_str() );


@@ -242,8 +254,9 @@ long wxComboBox::GetLastPosition() const
 
 void wxComboBox::Replace(long from, long to, const wxString& value)
 {
 
 void wxComboBox::Replace(long from, long to, const wxString& value)
 {

-    XmComboBoxReplace ((Widget) m_mainWidget, (XmTextPosition) from, (XmTextPosition) to,
-        (char*) (const char*) value);
+    XmComboBoxReplace ((Widget) m_mainWidget, (XmTextPosition) from,
+                       (XmTextPosition) to,
+                       wxConstCast(value.c_str(), char));

 }
 
 void wxComboBox::Remove(long from, long to)
 }
 
 void wxComboBox::Remove(long from, long to)


@@ -269,7 +282,8 @@ void  wxComboBoxCallback (Widget WXUNUSED(w), XtPointer clientData,
     case XmCR_SINGLE_SELECT:
     case XmCR_BROWSE_SELECT:
         {
     case XmCR_SINGLE_SELECT:
     case XmCR_BROWSE_SELECT:
         {

-            wxCommandEvent event (wxEVT_COMMAND_COMBOBOX_SELECTED, item->GetId());
+            wxCommandEvent event (wxEVT_COMMAND_COMBOBOX_SELECTED,
+                                  item->GetId());

             event.m_commandInt = cbs->index - 1;
             event.m_commandString = item->GetString (event.m_commandInt);
             if ( item->HasClientObjectData() )
             event.m_commandInt = cbs->index - 1;
             event.m_commandString = item->GetString (event.m_commandInt);
             if ( item->HasClientObjectData() )


@@ -326,5 +340,6 @@ wxSize wxComboBox::DoGetBestSize() const
         return wxWindow::DoGetBestSize();
 }
 
         return wxWindow::DoGetBestSize();
 }
 

-#endif
+#endif // XmVersion < 2000

 
 

+#endif // wxUSE_COMBOBOX