don't create non-existing groups in HasEntry()

[wxWidgets.git] / src / mac / carbon / clipbrd.cpp

diff --git a/src/mac/carbon/clipbrd.cpp b/src/mac/carbon/clipbrd.cpp
index 7e48c9ec34ce8ee22e1f15c9e13e21e3985adf40..d0ea791e8b9a7509530637a118b2b23e64473557 100644 (file)
--- a/src/mac/carbon/clipbrd.cpp
+++ b/src/mac/carbon/clipbrd.cpp
@@ -1,7 +1,8 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        clipbrd.cpp
+// Name:        src/mac/carbon/clipbrd.cpp

 // Purpose:     Clipboard functionality
 // Purpose:     Clipboard functionality

-// Author:      Stefan Csomor
+// Author:      Stefan Csomor;
+//              Generalized clipboard implementation by Matthew Flatt

 // Modified by:
 // Created:     1998-01-01
 // RCS-ID:      $Id$
 // Modified by:
 // Created:     1998-01-01
 // RCS-ID:      $Id$


@@ -9,61 +10,63 @@
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 

-#ifdef __GNUG__
-#pragma implementation "clipbrd.h"
+#include "wx/wxprec.h"
+
+#if wxUSE_CLIPBOARD
+
+#include "wx/clipbrd.h"
+
+#ifndef WX_PRECOMP
+    #include "wx/intl.h"
+    #include "wx/log.h"
+    #include "wx/app.h"
+    #include "wx/utils.h"
+    #include "wx/frame.h"
+    #include "wx/bitmap.h"

 #endif
 
 #endif
 

-#include "wx/app.h"
-#include "wx/frame.h"
-#include "wx/bitmap.h"
-#include "wx/utils.h"

 #include "wx/metafile.h"
 #include "wx/metafile.h"

-#include "wx/clipbrd.h"
-#include "wx/intl.h"
-#include "wx/log.h"

 
 #ifndef __DARWIN__
 #include <Scrap.h>
 #endif
 
 #ifndef __DARWIN__
 #include <Scrap.h>
 #endif

+

 #include "wx/mac/uma.h"
 
 #define wxUSE_DATAOBJ 1
 
 #include <string.h>
 
 #include "wx/mac/uma.h"
 
 #define wxUSE_DATAOBJ 1
 
 #include <string.h>
 

+

 // the trace mask we use with wxLogTrace() - call
 // wxLog::AddTraceMask(TRACE_CLIPBOARD) to enable the trace messages from here
 // (there will be a *lot* of them!)
 // the trace mask we use with wxLogTrace() - call
 // wxLog::AddTraceMask(TRACE_CLIPBOARD) to enable the trace messages from here
 // (there will be a *lot* of them!)

-static const wxChar *TRACE_CLIPBOARD = _T("clipboard");
+static const wxChar *TRACE_CLIPBOARD = wxT("clipboard");
+

 
 

-void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
+void * wxGetClipboardData( wxDataFormat dataFormat, long *len )

 {
 {

-#if !TARGET_CARBON
-    OSErr err = noErr ;
-#else
-    OSStatus err = noErr ;
-#endif
-    void * data = NULL ;
+    OSStatus err = noErr;
+    void * data = NULL;

     Size byteCount;
 
     switch (dataFormat.GetType())
     {
     case wxDF_OEMTEXT:
         dataFormat = wxDF_TEXT;
     Size byteCount;
 
     switch (dataFormat.GetType())
     {
     case wxDF_OEMTEXT:
         dataFormat = wxDF_TEXT;

-        // fall through
+        break;

 
     case wxDF_TEXT:
 
     case wxDF_TEXT:

-        break;

     case wxDF_UNICODETEXT:
         break;
     case wxDF_UNICODETEXT:
         break;

-    case wxDF_BITMAP :
-    case wxDF_METAFILE :
-        break ;
+
+    case wxDF_BITMAP:
+    case wxDF_METAFILE:
+        break;
+

     default:
     default:

-        {
-            wxLogError(_("Unsupported clipboard format."));
-            return NULL;
-        }
+        // custom datatype
+        break;

     }
 
 #if TARGET_CARBON
     }
 
 #if TARGET_CARBON


@@ -74,128 +77,134 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
     {
         ScrapFlavorFlags    flavorFlags;
 
     {
         ScrapFlavorFlags    flavorFlags;
 

-        if (( err = GetScrapFlavorFlags( scrapRef, dataFormat.GetFormatId(), &flavorFlags )) == noErr)
+        err = GetScrapFlavorFlags( scrapRef, dataFormat.GetFormatId(), &flavorFlags );
+        if (err == noErr)

         {
         {

-            if (( err = GetScrapFlavorSize( scrapRef, dataFormat.GetFormatId(), &byteCount )) == noErr)
+            err = GetScrapFlavorSize( scrapRef, dataFormat.GetFormatId(), &byteCount );
+            if (err == noErr)

             {
             {

-                Size allocSize = byteCount ;
+                Size allocSize = byteCount;

                 if ( dataFormat.GetType() == wxDF_TEXT )
                 if ( dataFormat.GetType() == wxDF_TEXT )

-                    allocSize += 1 ;
+                    allocSize += 1;

                 else if ( dataFormat.GetType() == wxDF_UNICODETEXT )
                 else if ( dataFormat.GetType() == wxDF_UNICODETEXT )

-                    allocSize += 2 ;
+                    allocSize += 2;

 
 

-                data = new char[ allocSize ] ;
+                data = new char[ allocSize ];

 
                 if (( err = GetScrapFlavorData( scrapRef, dataFormat.GetFormatId(), &byteCount , data )) == noErr )
                 {
 
                 if (( err = GetScrapFlavorData( scrapRef, dataFormat.GetFormatId(), &byteCount , data )) == noErr )
                 {

-                    *len = allocSize ;
+                    *len = allocSize;

                     if ( dataFormat.GetType() == wxDF_TEXT )
                     if ( dataFormat.GetType() == wxDF_TEXT )

-                        ((char*)data)[byteCount] = 0 ;
+                        ((char*)data)[ byteCount ] = 0;

                     if ( dataFormat.GetType() == wxDF_UNICODETEXT )
                     if ( dataFormat.GetType() == wxDF_UNICODETEXT )

-                        ((wxChar*)data)[byteCount/2] = 0 ;
+                    {
+                        // "data" format is UTF16, so 2 bytes = one character
+                        // wxChar size depends on platform, so just clear last 2 bytes
+                        ((char*)data)[ byteCount + 0 ] =
+                        ((char*)data)[ byteCount + 1 ] = 0;
+                    }

                 }
                 else
                 {
                 }
                 else
                 {

-                    delete[] ((char *)data) ;
-                    data = NULL ;
+                    delete [] (char*)data;
+                    data = NULL;

                 }
             }
         }
     }
 
 #else
                 }
             }
         }
     }
 
 #else

-    long offset ;
-    Handle datahandle = NewHandle(0) ;
-    HLock( datahandle ) ;
-    GetScrap( datahandle , dataFormat.GetFormatId() , &offset ) ;
-    HUnlock( datahandle ) ;
+    long offset;
+    Handle datahandle = NewHandle( 0 );
+    HLock( datahandle );
+    err = (OSStatus)GetScrap( datahandle, dataFormat.GetFormatId(), &offset );
+    HUnlock( datahandle );

     if ( GetHandleSize( datahandle ) > 0 )
     {
     if ( GetHandleSize( datahandle ) > 0 )
     {

-        byteCount = GetHandleSize( datahandle ) ;
-        Size allocSize = byteCount ;
+        byteCount = GetHandleSize( datahandle );
+        Size allocSize = byteCount;

         if ( dataFormat.GetType() == wxDF_TEXT )
         if ( dataFormat.GetType() == wxDF_TEXT )

-            allocSize += 1 ;
+            allocSize += 1;

         else if ( dataFormat.GetType() == wxDF_UNICODETEXT )
         else if ( dataFormat.GetType() == wxDF_UNICODETEXT )

-            allocSize += 2 ;
+            allocSize += 2;

 
 

-        data = new char[ allocSize ] ;
+        data = new char[ allocSize ];

 
 

-        memcpy( (char*) data , (char*) *datahandle , byteCount ) ;
+        memcpy( (char*) data, (char*) *datahandle, byteCount );

         if ( dataFormat.GetType() == wxDF_TEXT )
         if ( dataFormat.GetType() == wxDF_TEXT )

-            ((char*)data)[byteCount] = 0 ;
-        if ( dataFormat.GetType() == wxDF_UNICODETEXT )
-            ((wxChar*)data)[byteCount/2] = 0 ;
-        *len = byteCount ;
+            ((char*)data)[ byteCount ] = 0;
+        else if ( dataFormat.GetType() == wxDF_UNICODETEXT )
+            ((wxChar*)data)[ byteCount / 2 ] = 0;
+        *len = byteCount;

     }
     }

-    DisposeHandle( datahandle ) ;
+
+    DisposeHandle( datahandle );

 #endif
 #endif

-    if ( err )
+
+    if (err != noErr)

     {
     {

-        wxLogSysError(_("Failed to get clipboard data."));
+        wxLogSysError(wxT("Failed to get clipboard data."));

 
 

-        return NULL ;
+        return NULL;

     }
 
     }
 

-    if ( dataFormat.GetType() == wxDF_TEXT )
-    {
-        wxMacConvertNewlines10To13( (char*) data ) ;
-    }
+    if (dataFormat.GetType() == wxDF_TEXT)
+        wxMacConvertNewlines10To13( (char*)data );

 
     return data;
 }
 
 
     return data;
 }
 

-
-/*
- * Generalized clipboard implementation by Matthew Flatt
- */
-

 IMPLEMENT_DYNAMIC_CLASS(wxClipboard, wxObject)
 
 wxClipboard::wxClipboard()
 {
 IMPLEMENT_DYNAMIC_CLASS(wxClipboard, wxObject)
 
 wxClipboard::wxClipboard()
 {

-    m_open = false ;
-    m_data = NULL ;
+    m_open = false;
+    m_data = NULL;

 }
 
 wxClipboard::~wxClipboard()
 {
 }
 
 wxClipboard::~wxClipboard()
 {

-    if (m_data)
+    if (m_data != NULL)

     {
         delete m_data;
     {
         delete m_data;

-        m_data = (wxDataObject*) NULL;
+        m_data = NULL;

     }
 }
 
 void wxClipboard::Clear()
 {
     }
 }
 
 void wxClipboard::Clear()
 {

-    if (m_data)
+    if (m_data != NULL)

     {
         delete m_data;
     {
         delete m_data;

-        m_data = (wxDataObject*) NULL;
+        m_data = NULL;

     }
     }

+

 #if TARGET_CARBON
 #if TARGET_CARBON

-    OSStatus err ;
-    err = ClearCurrentScrap( );
+    OSStatus err;
+    err = ClearCurrentScrap();

 #else
 #else

-    OSErr err ;
-    err = ZeroScrap( );
+    OSErr err;
+    err = ZeroScrap();

 #endif
 #endif

-    if ( err )
+
+    if (err != noErr)

     {
     {

-        wxLogSysError(_("Failed to empty the clipboard."));
+        wxLogSysError( wxT("Failed to empty the clipboard.") );

     }
 }
 
 bool wxClipboard::Flush()
 {
     }
 }
 
 bool wxClipboard::Flush()
 {

-    return FALSE;
+    return false;

 }
 
 bool wxClipboard::Open()
 {
 }
 
 bool wxClipboard::Open()
 {

-    wxCHECK_MSG( !m_open, FALSE, wxT("clipboard already open") );
-    m_open = true ;
-    return true ;
+    wxCHECK_MSG( !m_open, false, wxT("clipboard already open") );
+
+    m_open = true;
+
+    return true;

 }
 
 bool wxClipboard::IsOpened() const
 }
 
 bool wxClipboard::IsOpened() const


@@ -205,98 +214,117 @@ bool wxClipboard::IsOpened() const
 
 bool wxClipboard::SetData( wxDataObject *data )
 {
 
 bool wxClipboard::SetData( wxDataObject *data )
 {

-    wxCHECK_MSG( m_open, FALSE, wxT("clipboard not open") );
-
-    wxCHECK_MSG( data, FALSE, wxT("data is invalid") );
+    wxCHECK_MSG( m_open, false, wxT("clipboard not open") );
+    wxCHECK_MSG( data, false, wxT("data is invalid") );

 
     Clear();
 
     Clear();

-    // as we can only store one wxDataObject, this is the same in this
-    // implementation
+
+    // as we can only store one wxDataObject,
+    // this is the same in this implementation

     return AddData( data );
 }
 
 bool wxClipboard::AddData( wxDataObject *data )
 {
     return AddData( data );
 }
 
 bool wxClipboard::AddData( wxDataObject *data )
 {

-    wxCHECK_MSG( m_open, FALSE, wxT("clipboard not open") );
+    wxCHECK_MSG( m_open, false, wxT("clipboard not open") );
+    wxCHECK_MSG( data, false, wxT("data is invalid") );

 
 

-    wxCHECK_MSG( data, FALSE, wxT("data is invalid") );
-
-    /* we can only store one wxDataObject */
+    // we can only store one wxDataObject

     Clear();
 
     m_data = data;
 
     Clear();
 
     m_data = data;
 

-    /* get formats from wxDataObjects */
+    // get formats from wxDataObjects

     wxDataFormat *array = new wxDataFormat[ m_data->GetFormatCount() ];
     m_data->GetAllFormats( array );
 
     for (size_t i = 0; i < m_data->GetFormatCount(); i++)
     {
     wxDataFormat *array = new wxDataFormat[ m_data->GetFormatCount() ];
     m_data->GetAllFormats( array );
 
     for (size_t i = 0; i < m_data->GetFormatCount(); i++)
     {

-        wxLogTrace( TRACE_CLIPBOARD,
-                    wxT("wxClipboard now supports atom %s"),
-                    array[i].GetId().c_str() );
-
-        size_t sz = data->GetDataSize( array[i] ) ;
-        void* buf = malloc( sz + 1 ) ;
-        if ( buf )
-        {        
-            data->GetDataHere( array[i] , buf ) ;
-            OSType mactype = 0 ;
+        if (array[i].IsStandard())
+        {
+            wxLogTrace( TRACE_CLIPBOARD,
+                        wxT("wxClipboard now supports standard atom type %d"),
+                        array[i].GetType() );
+        }
+        else
+        {
+            wxLogTrace( TRACE_CLIPBOARD,
+                        wxT("wxClipboard now supports atom %s"),
+                        array[i].GetId().c_str() );
+        }
+
+        size_t sz = data->GetDataSize( array[ i ] );
+        void* buf = malloc( sz + 1 );
+        if ( buf != NULL )
+        {
+            // empty the buffer because in some case GetDataHere does not fill buf
+            memset( buf, 0, sz + 1 );
+            data->GetDataHere( array[ i ], buf );
+            OSType mactype = 0;

             switch ( array[i].GetType() )
             {
             switch ( array[i].GetType() )
             {

-               case wxDF_TEXT:
-               case wxDF_OEMTEXT:
-                    mactype = kScrapFlavorTypeText ;
-               break ;
-    #if wxUSE_UNICODE
-               case wxDF_UNICODETEXT :
-                    mactype = kScrapFlavorTypeUnicode ;
-               break ;
-    #endif
-    #if wxUSE_DRAG_AND_DROP
+            case wxDF_TEXT:
+            case wxDF_OEMTEXT:
+                mactype = kScrapFlavorTypeText;
+                sz -= 1;
+                break;
+
+#if wxUSE_UNICODE
+            case wxDF_UNICODETEXT:
+                mactype = kScrapFlavorTypeUnicode;
+                sz -= 2;
+                break;
+#endif
+
+#if wxUSE_DRAG_AND_DROP

             case wxDF_METAFILE:
             case wxDF_METAFILE:

-                    mactype = kScrapFlavorTypePicture ;
-               break ;
-    #endif
-               case wxDF_BITMAP:
-               case wxDF_DIB:
-                    mactype = kScrapFlavorTypePicture ;
-                    break ;
-               default:
-                    break ;
+                mactype = kScrapFlavorTypePicture;
+                break;
+#endif
+
+            case wxDF_BITMAP:
+            case wxDF_DIB:
+                mactype = kScrapFlavorTypePicture;
+                break;
+
+            default:
+                mactype = (OSType)(array[ i ].GetFormatId());
+                break;

             }
             }

-            UMAPutScrap( sz , mactype , buf ) ;
-            free( buf ) ;
+
+            UMAPutScrap( sz , mactype , buf );
+            free( buf );

         }
     }
 
         }
     }
 

-    delete[] array;
+    delete [] array;

 
 

-    return true ;
+    return true;

 }
 
 void wxClipboard::Close()
 {
     wxCHECK_RET( m_open, wxT("clipboard not open") );
 
 }
 
 void wxClipboard::Close()
 {
     wxCHECK_RET( m_open, wxT("clipboard not open") );
 

-    m_open = false ;
-    
-       // Get rid of cached object.  If this is not done copying from another application will
-       // only work once
+    m_open = false;
+
+    // Get rid of cached object.
+    // If this is not done, copying data from
+    // another application will only work once

     if (m_data)
     {
         delete m_data;
         m_data = (wxDataObject*) NULL;
     if (m_data)
     {
         delete m_data;
         m_data = (wxDataObject*) NULL;

-    }    
-           
+    }

 }
 
 bool wxClipboard::IsSupported( const wxDataFormat &dataFormat )
 {
 }
 
 bool wxClipboard::IsSupported( const wxDataFormat &dataFormat )
 {

-  if ( m_data )
-  {
-    return m_data->IsSupported( dataFormat ) ;
-  }
+    if ( m_data )
+        return m_data->IsSupported( dataFormat );
+
+    bool hasData = false;
+

 #if TARGET_CARBON
     OSStatus err = noErr;
     ScrapRef scrapRef;
 #if TARGET_CARBON
     OSStatus err = noErr;
     ScrapRef scrapRef;


@@ -304,98 +332,113 @@ bool wxClipboard::IsSupported( const wxDataFormat &dataFormat )
     err = GetCurrentScrap( &scrapRef );
     if ( err != noTypeErr && err != memFullErr )
     {
     err = GetCurrentScrap( &scrapRef );
     if ( err != noTypeErr && err != memFullErr )
     {

-        ScrapFlavorFlags    flavorFlags;
-        Size                byteCount;
+        ScrapFlavorFlags flavorFlags;
+        Size byteCount;

 
 

-        if (( err = GetScrapFlavorFlags( scrapRef, dataFormat.GetFormatId(), &flavorFlags )) == noErr)
+        err = GetScrapFlavorFlags( scrapRef, dataFormat.GetFormatId(), &flavorFlags );
+        if (err == noErr)
+        {
+            err = GetScrapFlavorSize( scrapRef, dataFormat.GetFormatId(), &byteCount );
+            if (err == noErr)
+                hasData = true;
+        }
+        else if ( dataFormat.GetType() == wxDF_UNICODETEXT )

         {
         {

-            if (( err = GetScrapFlavorSize( scrapRef, dataFormat.GetFormatId(), &byteCount )) == noErr)
+            err = GetScrapFlavorFlags( scrapRef, 'TEXT', &flavorFlags );
+            if (err == noErr)

             {
             {

-                return TRUE ;
+                err = GetScrapFlavorSize( scrapRef, 'TEXT', &byteCount );
+                if (err == noErr)
+                    hasData = true;

             }
         }
     }
             }
         }
     }

-    return FALSE;

 
 #else
 
 #else

-    long offset ;
-    Handle datahandle = NewHandle(0) ;
-    HLock( datahandle ) ;
-    GetScrap( datahandle , dataFormat.GetFormatId() , &offset ) ;
-    HUnlock( datahandle ) ;
-    bool hasData = GetHandleSize( datahandle ) > 0 ;
-    DisposeHandle( datahandle ) ;
-    return hasData ;
+
+    long offset = 0;
+    Handle datahandle = NewHandle( 0 );
+    HLock( datahandle );
+    GetScrap( datahandle, dataFormat.GetFormatId(), &offset );
+    HUnlock( datahandle );
+    hasData = GetHandleSize( datahandle ) > 0;
+    DisposeHandle( datahandle );

 #endif
 #endif

+
+    return hasData;

 }
 
 bool wxClipboard::GetData( wxDataObject& data )
 {
 }
 
 bool wxClipboard::GetData( wxDataObject& data )
 {

-    wxCHECK_MSG( m_open, FALSE, wxT("clipboard not open") );
+    wxCHECK_MSG( m_open, false, wxT("clipboard not open") );

 
 

-    size_t formatcount = data.GetFormatCount() + 1 ;
-    wxDataFormat *array = new wxDataFormat[ formatcount  ];
+    size_t formatcount = data.GetFormatCount() + 1;
+    wxDataFormat *array = new wxDataFormat[ formatcount ];

     array[0] = data.GetPreferredFormat();
     data.GetAllFormats( &array[1] );
 
     array[0] = data.GetPreferredFormat();
     data.GetAllFormats( &array[1] );
 

-    bool transferred = false ;
+    bool transferred = false;

 
     if ( m_data )
     {
 
     if ( m_data )
     {

-      for (size_t i = 0; !transferred && i < formatcount ; i++)
-      {
-          wxDataFormat format = array[i] ;
-          if ( m_data->IsSupported( format ) )
-          {
-            int size = m_data->GetDataSize( format );
-            transferred = true ;
-
-            if (size == 0)
-            {
-              data.SetData(format , 0 , 0 ) ;
-            }
-            else
+        for (size_t i = 0; !transferred && i < formatcount; i++)
+        {
+            wxDataFormat format = array[ i ];
+            if ( m_data->IsSupported( format ) )

             {
             {

-              char *d = new char[size];
-              m_data->GetDataHere( format , (void*) d );
-              data.SetData( format , size , d ) ;
-              delete[] d ;
+                int dataSize = m_data->GetDataSize( format );
+                transferred = true;
+
+                if (dataSize == 0)
+                {
+                    data.SetData( format, 0, 0 );
+                }
+                else
+                {
+                    char *d = new char[ dataSize ];
+                    m_data->GetDataHere( format, (void*)d );
+                    data.SetData( format, dataSize, d );
+                    delete [] d;
+                }

             }
             }

-          }
-       }
+        }

     }
     }

-    /* get formats from wxDataObjects */
+
+    // get formats from wxDataObjects

     if ( !transferred )
     {
     if ( !transferred )
     {

-      for (size_t i = 0; !transferred && i < formatcount ; i++)
-      {
-          wxDataFormat format = array[i] ;
-
-          switch ( format.GetType() )
-          {
-              case wxDF_TEXT :
-              case wxDF_OEMTEXT :
-              case wxDF_BITMAP :
-              case wxDF_METAFILE :
-              {
-                  long len ;
-                  char* s = (char*)wxGetClipboardData(format, &len );
-                  if ( s )
-                  {
-                    data.SetData( format , len , s ) ;
-                    delete [] s;
-
-                    transferred = true ;
-                  }
-              }
-                            break ;
-
-              default :
-                break ;
-          }
-       }
+        for (size_t i = 0; !transferred && i < formatcount; i++)
+        {
+            wxDataFormat format = array[ i ];
+
+            switch ( format.GetType() )
+            {
+                // NOTE: this is usable for all data types
+                case wxDF_TEXT:
+                case wxDF_UNICODETEXT:
+                case wxDF_OEMTEXT:
+                case wxDF_BITMAP:
+                case wxDF_METAFILE:
+                default:
+                {
+                    long len;
+                    char* s = (char*)wxGetClipboardData( format, &len );
+                    if (s != NULL)
+                    {
+                        data.SetData( format, len, s );
+                        delete [] s;
+
+                        transferred = true;
+                    }
+                }
+                break;
+            }
+        }

     }
 
     }
 

-    delete[] array ;
-    return transferred ;
+    delete [] array;
+
+    return transferred;

 }
 }

+
+#endif