cosmetic fixes to font encoding names

[wxWidgets.git] / src / mac / listbox.cpp

diff --git a/src/mac/listbox.cpp b/src/mac/listbox.cpp
index c518ad035ccb495731dd670ffaf0ce91a5874f3e..6564af7b7777ee1a33213e0a7cea0433eb16561a 100644 (file)
--- a/src/mac/listbox.cpp
+++ b/src/mac/listbox.cpp
@@ -21,11 +21,13 @@
 #include "wx/utils.h"
 #include "extldef.h"
 
 #include "wx/utils.h"
 #include "extldef.h"
 

+#if !USE_SHARED_LIBRARY

   IMPLEMENT_DYNAMIC_CLASS(wxListBox, wxControl)
 
 BEGIN_EVENT_TABLE(wxListBox, wxControl)
        EVT_SIZE( wxListBox::OnSize ) 
 END_EVENT_TABLE()
   IMPLEMENT_DYNAMIC_CLASS(wxListBox, wxControl)
 
 BEGIN_EVENT_TABLE(wxListBox, wxControl)
        EVT_SIZE( wxListBox::OnSize ) 
 END_EVENT_TABLE()

+#endif

 
 #include <wx/mac/uma.h>
 
 
 #include <wx/mac/uma.h>
 


@@ -41,6 +43,7 @@ wxListBox::wxListBox()
 {
   m_noItems = 0;
   m_selected = 0;
 {
   m_noItems = 0;
   m_selected = 0;

+  m_macList = NULL ;

 }
 
 bool wxListBox::Create(wxWindow *parent, wxWindowID id,
 }
 
 bool wxListBox::Create(wxWindow *parent, wxWindowID id,


@@ -56,8 +59,6 @@ bool wxListBox::Create(wxWindow *parent, wxWindowID id,
 
        Rect bounds ;
        Str255 title ;
 
        Rect bounds ;
        Str255 title ;

-       m_macHorizontalBorder = 5 ; // additional pixels around the real control
-       m_macVerticalBorder = 5 ;

        
        MacPreControlCreate( parent , id ,  "" , pos , size ,style, validator , name , &bounds , title ) ;
 
        
        MacPreControlCreate( parent , id ,  "" , pos , size ,style, validator , name , &bounds , title ) ;
 


@@ -67,6 +68,7 @@ bool wxListBox::Create(wxWindow *parent, wxWindowID id,
        long    result ;
        UMAGetControlData( m_macControl , kControlNoPart , kControlListBoxListHandleTag , sizeof( ListHandle ) , (char*) &m_macList  , &result ) ;
 
        long    result ;
        UMAGetControlData( m_macControl , kControlNoPart , kControlListBoxListHandleTag , sizeof( ListHandle ) , (char*) &m_macList  , &result ) ;
 

+       HLock( (Handle) m_macList ) ;

        NewExtLDEFInfo( m_macList , MacDrawStringCell , (long) this ) ;
        (**m_macList).selFlags = 0 ;
        if ( style & wxLB_MULTIPLE )
        NewExtLDEFInfo( m_macList , MacDrawStringCell , (long) this ) ;
        (**m_macList).selFlags = 0 ;
        if ( style & wxLB_MULTIPLE )


@@ -109,7 +111,11 @@ bool wxListBox::Create(wxWindow *parent, wxWindowID id,
 wxListBox::~wxListBox()
 {
        Free() ;
 wxListBox::~wxListBox()
 {
        Free() ;

-       DisposeExtLDEFInfo( m_macList ) ;
+       if ( m_macList )
+       {
+               DisposeExtLDEFInfo( m_macList ) ;
+               m_macList = NULL ;
+       }

 }
 
 void wxListBox::Free()
 }
 
 void wxListBox::Free()


@@ -156,7 +162,7 @@ void wxListBox::Delete(int N)
 #endif // wxUSE_OWNER_DRAWN/!wxUSE_OWNER_DRAWN
        m_stringArray.Remove( N ) ;
        m_dataArray.Remove( N ) ;
 #endif // wxUSE_OWNER_DRAWN/!wxUSE_OWNER_DRAWN
        m_stringArray.Remove( N ) ;
        m_dataArray.Remove( N ) ;

-       m_noItems --;
+       m_noItems --;

        
        MacDelete( N ) ;
 }
        
        MacDelete( N ) ;
 }


@@ -236,19 +242,29 @@ int wxListBox::FindString(const wxString& st) const
        {
                wxString search = s.Left( s.Length() - 1 ) ;
                int len = search.Length() ;
        {
                wxString search = s.Left( s.Length() - 1 ) ;
                int len = search.Length() ;

-    for ( int i = 0 ; i < m_noItems ; ++ i )
-    {
-       if ( equalstring( m_stringArray[i].Left( len ) , search , false , false ) )
-               return i ;
-    }
+       Str255 s1 , s2 ;
+           strcpy( (char*) s2 , search.c_str() ) ;
+           c2pstr( (char*) s2 ) ;
+       for ( int i = 0 ; i < m_noItems ; ++ i )
+           {
+               strcpy( (char*) s1 , m_stringArray[i].Left( len ).c_str() ) ;
+               c2pstr( (char*) s1 ) ;
+               if ( EqualString( s1 , s2 , false , false ) )
+                       return i ;
+           }

        }
        else
        {
        }
        else
        {

-    for ( int i = 0 ; i < m_noItems ; ++ i )
-    {
-       if ( equalstring( m_stringArray[i] , s , false , false ) )
-               return i ;
-    }
+       Str255 s1 , s2 ;
+           strcpy( (char*) s2 , s.c_str() ) ;
+           c2pstr( (char*) s2 ) ;
+           for ( int i = 0 ; i < m_noItems ; ++ i )
+           {
+               strcpy( (char*) s1 , m_stringArray[i].c_str() ) ;
+               c2pstr( (char*) s1 ) ;
+               if ( EqualString( s1 , s2 , false , false ) )
+                       return i ;
+           }

    }
    return -1;
 }
    }
    return -1;
 }


@@ -308,7 +324,7 @@ void wxListBox::DoSetItemClientData(int N, void *Client_data)
        if ( m_dataArray.GetCount() > N )
        {
        m_dataArray[N] = (char*) Client_data ;
        if ( m_dataArray.GetCount() > N )
        {
        m_dataArray[N] = (char*) Client_data ;

-       }
+    }

     else
     {
        m_dataArray.Add( (char*) Client_data ) ;
     else
     {
        m_dataArray.Add( (char*) Client_data ) ;


@@ -379,7 +395,14 @@ void wxListBox::DoInsertItems(const wxArrayString& items, int pos)
 
 void wxListBox::SetString(int N, const wxString& s)
 {
 
 void wxListBox::SetString(int N, const wxString& s)
 {

-       m_stringArray[N] = s ;
+       wxString str ;
+       if( wxApp::s_macDefaultEncodingIsPC )
+       {
+               str = wxMacMakeMacStringFromPC( s )  ;
+       }
+       else
+               str = s ;
+       m_stringArray[N] = str ;

        MacSet( N , s ) ;
 }
 
        MacSet( N , s ) ;
 }
 


@@ -539,7 +562,7 @@ void wxListBox::MacScrollTo( int n )
 void wxListBox::OnSize( const wxSizeEvent &event)
 {
        Point pt = (**m_macList).cellSize ;
 void wxListBox::OnSize( const wxSizeEvent &event)
 {
        Point pt = (**m_macList).cellSize ;

-       pt.h =  m_width - 15 /* scrollbar */ - m_macHorizontalBorder * 2 ;
+       pt.h =  m_width - 15  ;

        LCellSize( pt , m_macList ) ;
 }
 
        LCellSize( pt , m_macList ) ;
 }