From: Stefan Csomor <csomor@advancedconcepts.ch>
Date: Wed, 29 Jan 2003 15:03:43 +0000 (+0000)
Subject: adaptions for cleaned up string conversions
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/95fd868c8f92ed40f3a4f16a1b3a38f7ee7d44ec

adaptions for cleaned up string conversions


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

diff --git a/src/mac/carbon/clipbrd.cpp b/src/mac/carbon/clipbrd.cpp
index 76952a442e..7059fae3c2 100644
--- a/src/mac/carbon/clipbrd.cpp
+++ b/src/mac/carbon/clipbrd.cpp
@@ -44,7 +44,8 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
 #else
     OSStatus err = noErr ;
 #endif
-  void * data = NULL ;
+  	void * data = NULL ;
+	Size byteCount;
     
     switch (dataFormat.GetType())
     {
@@ -68,7 +69,6 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
     if ( err != noTypeErr && err != memFullErr )    
     {
         ScrapFlavorFlags    flavorFlags;
-        Size                byteCount;
         
         if (( err = GetScrapFlavorFlags( scrapRef, dataFormat.GetFormatId(), &flavorFlags )) == noErr)
         {
@@ -101,7 +101,7 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
     HUnlock( datahandle ) ;
     if ( GetHandleSize( datahandle ) > 0 )
     {
-      long byteCount = GetHandleSize( datahandle ) ;
+      byteCount = GetHandleSize( datahandle ) ;
       if ( dataFormat.GetType() == wxDF_TEXT )  
         data = new char[ byteCount + 1] ;
       else
@@ -110,7 +110,7 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
       memcpy( (char*) data , (char*) *datahandle , byteCount ) ;
       if ( dataFormat.GetType() == wxDF_TEXT )  
           ((char*)data)[byteCount] = 0 ;
-      * len = byteCount ;
+      *len = byteCount ;
     }
     DisposeHandle( datahandle ) ;
 #endif
@@ -122,7 +122,7 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
     }
     if ( dataFormat.GetType() == wxDF_TEXT && wxApp::s_macDefaultEncodingIsPC )
     {
-      wxMacConvertToPC((char*)data) ;
+      wxMacConvertToPC((char*)data,(char*)data,byteCount) ;
     }
     return data;
 }
diff --git a/src/mac/carbon/fontenum.cpp b/src/mac/carbon/fontenum.cpp
index 43aeb9a7a8..4a91e5703f 100644
--- a/src/mac/carbon/fontenum.cpp
+++ b/src/mac/carbon/fontenum.cpp
@@ -105,7 +105,7 @@ void wxFontEnumeratorHelper::DoEnumerate()
 {
 	MenuHandle	menu ;
 	Str255		p_name ;
-	char        c_name[256] ;
+
 	short 		lines ;
 	
 	menu = NewMenu( 32000 , "\pFont" )  ;
@@ -115,12 +115,8 @@ void wxFontEnumeratorHelper::DoEnumerate()
 	for ( int i = 1 ; i < lines+1  ; i ++ )
 	{
 	    GetMenuItemText( menu , i , p_name ) ;
-#if TARGET_CARBON
-	    p2cstrcpy( c_name, p_name ) ;
-#else
-		p2cstr( p_name ) ;
-	    strcpy( c_name, (char *)p_name ) ;
-#endif
+	    wxString c_name = wxMacMakeStringFromPascal(p_name) ;
+
 	    /*
 	      
 	      if ( m_fixedOnly )
diff --git a/src/mac/clipbrd.cpp b/src/mac/clipbrd.cpp
index 76952a442e..7059fae3c2 100644
--- a/src/mac/clipbrd.cpp
+++ b/src/mac/clipbrd.cpp
@@ -44,7 +44,8 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
 #else
     OSStatus err = noErr ;
 #endif
-  void * data = NULL ;
+  	void * data = NULL ;
+	Size byteCount;
     
     switch (dataFormat.GetType())
     {
@@ -68,7 +69,6 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
     if ( err != noTypeErr && err != memFullErr )    
     {
         ScrapFlavorFlags    flavorFlags;
-        Size                byteCount;
         
         if (( err = GetScrapFlavorFlags( scrapRef, dataFormat.GetFormatId(), &flavorFlags )) == noErr)
         {
@@ -101,7 +101,7 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
     HUnlock( datahandle ) ;
     if ( GetHandleSize( datahandle ) > 0 )
     {
-      long byteCount = GetHandleSize( datahandle ) ;
+      byteCount = GetHandleSize( datahandle ) ;
       if ( dataFormat.GetType() == wxDF_TEXT )  
         data = new char[ byteCount + 1] ;
       else
@@ -110,7 +110,7 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
       memcpy( (char*) data , (char*) *datahandle , byteCount ) ;
       if ( dataFormat.GetType() == wxDF_TEXT )  
           ((char*)data)[byteCount] = 0 ;
-      * len = byteCount ;
+      *len = byteCount ;
     }
     DisposeHandle( datahandle ) ;
 #endif
@@ -122,7 +122,7 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
     }
     if ( dataFormat.GetType() == wxDF_TEXT && wxApp::s_macDefaultEncodingIsPC )
     {
-      wxMacConvertToPC((char*)data) ;
+      wxMacConvertToPC((char*)data,(char*)data,byteCount) ;
     }
     return data;
 }
diff --git a/src/mac/fontenum.cpp b/src/mac/fontenum.cpp
index 43aeb9a7a8..4a91e5703f 100644
--- a/src/mac/fontenum.cpp
+++ b/src/mac/fontenum.cpp
@@ -105,7 +105,7 @@ void wxFontEnumeratorHelper::DoEnumerate()
 {
 	MenuHandle	menu ;
 	Str255		p_name ;
-	char        c_name[256] ;
+
 	short 		lines ;
 	
 	menu = NewMenu( 32000 , "\pFont" )  ;
@@ -115,12 +115,8 @@ void wxFontEnumeratorHelper::DoEnumerate()
 	for ( int i = 1 ; i < lines+1  ; i ++ )
 	{
 	    GetMenuItemText( menu , i , p_name ) ;
-#if TARGET_CARBON
-	    p2cstrcpy( c_name, p_name ) ;
-#else
-		p2cstr( p_name ) ;
-	    strcpy( c_name, (char *)p_name ) ;
-#endif
+	    wxString c_name = wxMacMakeStringFromPascal(p_name) ;
+
 	    /*
 	      
 	      if ( m_fixedOnly )