missing commit

[wxWidgets.git] / src / msw / clipbrd.cpp

diff --git a/src/msw/clipbrd.cpp b/src/msw/clipbrd.cpp
index bfb991866d677280273bb248a9f79863a783915b..bd929dcac3fe3168d363439b49e2de783f86f3ee 100644 (file)
--- a/src/msw/clipbrd.cpp
+++ b/src/msw/clipbrd.cpp
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        clipbrd.cpp
+// Name:        src/msw/clipbrd.cpp

 // Purpose:     Clipboard functionality
 // Author:      Julian Smart
 // Modified by:
 // Purpose:     Clipboard functionality
 // Author:      Julian Smart
 // Modified by:


@@ -17,10 +17,6 @@
 // headers
 // ---------------------------------------------------------------------------
 
 // headers
 // ---------------------------------------------------------------------------
 

-#ifdef __GNUG__
-    #pragma implementation "clipbrd.h"
-#endif
-

 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 


@@ -28,12 +24,10 @@
     #pragma hdrstop
 #endif
 
     #pragma hdrstop
 #endif
 

-#ifndef WX_PRECOMP
-    #include "wx/setup.h"
-#endif
-

 #if wxUSE_CLIPBOARD
 
 #if wxUSE_CLIPBOARD
 

+#include "wx/clipbrd.h"
+

 #ifndef WX_PRECOMP
     #include "wx/object.h"
     #include "wx/event.h"
 #ifndef WX_PRECOMP
     #include "wx/object.h"
     #include "wx/event.h"


@@ -42,33 +36,29 @@
     #include "wx/bitmap.h"
     #include "wx/utils.h"
     #include "wx/intl.h"
     #include "wx/bitmap.h"
     #include "wx/utils.h"
     #include "wx/intl.h"

+    #include "wx/log.h"
+    #include "wx/dataobj.h"

 #endif
 
 #if wxUSE_METAFILE
     #include "wx/metafile.h"
 #endif
 
 #endif
 
 #if wxUSE_METAFILE
     #include "wx/metafile.h"
 #endif
 

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

 
 #include <string.h>
 
 #include <string.h>

-#include <windows.h>

 
 #include "wx/msw/private.h"
 
 #include "wx/msw/private.h"

+#include "wx/msw/ole/oleutils.h"

 
 

-#ifndef __WXMICROWIN__
-#include "wx/msw/dib.h"
+#if wxUSE_WXDIB
+    #include "wx/msw/dib.h"

 #endif
 
 // wxDataObject is tied to OLE/drag and drop implementation, therefore so are
 // the functions using wxDataObject in wxClipboard
 //#define wxUSE_DATAOBJ wxUSE_DRAG_AND_DROP
 
 #endif
 
 // wxDataObject is tied to OLE/drag and drop implementation, therefore so are
 // the functions using wxDataObject in wxClipboard
 //#define wxUSE_DATAOBJ wxUSE_DRAG_AND_DROP
 

-#if wxUSE_DATAOBJ
-    #include "wx/dataobj.h"
-#endif
-
-#if wxUSE_OLE
+#if wxUSE_OLE && !defined(__WXWINCE__)

     // use OLE clipboard
     #define wxUSE_OLE_CLIPBOARD 1
 #else // !wxUSE_DATAOBJ
     // use OLE clipboard
     #define wxUSE_OLE_CLIPBOARD 1
 #else // !wxUSE_DATAOBJ


@@ -80,10 +70,6 @@
     #include <ole2.h>
 #endif // wxUSE_OLE_CLIPBOARD
 
     #include <ole2.h>
 #endif // wxUSE_OLE_CLIPBOARD
 

-#ifdef __WIN16__
-    #define memcpy hmemcpy
-#endif
-

 // ===========================================================================
 // implementation
 // ===========================================================================
 // ===========================================================================
 // implementation
 // ===========================================================================


@@ -92,11 +78,12 @@
 // old-style clipboard functions using Windows API
 // ---------------------------------------------------------------------------
 
 // old-style clipboard functions using Windows API
 // ---------------------------------------------------------------------------
 

-static bool gs_wxClipboardIsOpen = FALSE;
+static bool gs_wxClipboardIsOpen = false;
+static int gs_htmlcfid = 0;

 
 bool wxOpenClipboard()
 {
 
 bool wxOpenClipboard()
 {

-    wxCHECK_MSG( !gs_wxClipboardIsOpen, TRUE, wxT("clipboard already opened.") );
+    wxCHECK_MSG( !gs_wxClipboardIsOpen, true, wxT("clipboard already opened.") );

 
     wxWindow *win = wxTheApp->GetTopWindow();
     if ( win )
 
     wxWindow *win = wxTheApp->GetTopWindow();
     if ( win )


@@ -104,32 +91,34 @@ bool wxOpenClipboard()
         gs_wxClipboardIsOpen = ::OpenClipboard((HWND)win->GetHWND()) != 0;
 
         if ( !gs_wxClipboardIsOpen )
         gs_wxClipboardIsOpen = ::OpenClipboard((HWND)win->GetHWND()) != 0;
 
         if ( !gs_wxClipboardIsOpen )

+        {

             wxLogSysError(_("Failed to open the clipboard."));
             wxLogSysError(_("Failed to open the clipboard."));

+        }

 
         return gs_wxClipboardIsOpen;
     }
     else
     {
 
         return gs_wxClipboardIsOpen;
     }
     else
     {

-        wxLogDebug(wxT("Can not open clipboard without a main window."));
+        wxLogDebug(wxT("Cannot open clipboard without a main window."));

 
 

-        return FALSE;
+        return false;

     }
 }
 
 bool wxCloseClipboard()
 {
     }
 }
 
 bool wxCloseClipboard()
 {

-    wxCHECK_MSG( gs_wxClipboardIsOpen, FALSE, wxT("clipboard is not opened") );
+    wxCHECK_MSG( gs_wxClipboardIsOpen, false, wxT("clipboard is not opened") );

 
 

-    gs_wxClipboardIsOpen = FALSE;
+    gs_wxClipboardIsOpen = false;

 
     if ( ::CloseClipboard() == 0 )
     {
         wxLogSysError(_("Failed to close the clipboard."));
 
 
     if ( ::CloseClipboard() == 0 )
     {
         wxLogSysError(_("Failed to close the clipboard."));
 

-        return FALSE;
+        return false;

     }
 
     }
 

-    return TRUE;
+    return true;

 }
 
 bool wxEmptyClipboard()
 }
 
 bool wxEmptyClipboard()


@@ -138,10 +127,10 @@ bool wxEmptyClipboard()
     {
         wxLogSysError(_("Failed to empty the clipboard."));
 
     {
         wxLogSysError(_("Failed to empty the clipboard."));
 

-        return FALSE;
+        return false;

     }
 
     }
 

-    return TRUE;
+    return true;

 }
 
 bool wxIsClipboardOpened()
 }
 
 bool wxIsClipboardOpened()


@@ -151,26 +140,30 @@ bool wxIsClipboardOpened()
 
 bool wxIsClipboardFormatAvailable(wxDataFormat dataFormat)
 {
 
 bool wxIsClipboardFormatAvailable(wxDataFormat dataFormat)
 {

-    if ( ::IsClipboardFormatAvailable(dataFormat) )
+    wxDataFormat::NativeFormat cf = dataFormat.GetFormatId();
+    if (cf == wxDF_HTML)
+        cf = gs_htmlcfid;
+
+    if ( ::IsClipboardFormatAvailable(cf) )

     {
         // ok from the first try
     {
         // ok from the first try

-        return TRUE;
+        return true;

     }
 
     // for several standard formats, we can convert from some other ones too
     }
 
     // for several standard formats, we can convert from some other ones too

-    switch ( dataFormat.GetFormatId() )
+    switch ( cf )

     {
         // for bitmaps, DIBs will also do
         case CF_BITMAP:
             return ::IsClipboardFormatAvailable(CF_DIB) != 0;
 
     {
         // for bitmaps, DIBs will also do
         case CF_BITMAP:
             return ::IsClipboardFormatAvailable(CF_DIB) != 0;
 

-#if wxUSE_ENH_METAFILE && !defined(__WIN16__)
+#if wxUSE_ENH_METAFILE && !defined(__WXWINCE__)

         case CF_METAFILEPICT:
             return ::IsClipboardFormatAvailable(CF_ENHMETAFILE) != 0;
 #endif // wxUSE_ENH_METAFILE
 
         default:
         case CF_METAFILEPICT:
             return ::IsClipboardFormatAvailable(CF_ENHMETAFILE) != 0;
 #endif // wxUSE_ENH_METAFILE
 
         default:

-            return FALSE;
+            return false;

     }
 }
 
     }
 }
 


@@ -199,7 +192,7 @@ bool wxSetClipboardData(wxDataFormat dataFormat,
                     SelectObject(hdcSrc, old);
                     DeleteDC(hdcMem);
                     DeleteDC(hdcSrc);
                     SelectObject(hdcSrc, old);
                     DeleteDC(hdcMem);
                     DeleteDC(hdcSrc);

-                    return FALSE;
+                    return false;

                 }
 
                 HBITMAP old1 = (HBITMAP) SelectObject(hdcMem, hBitmap);
                 }
 
                 HBITMAP old1 = (HBITMAP) SelectObject(hdcMem, hBitmap);


@@ -219,22 +212,27 @@ bool wxSetClipboardData(wxDataFormat dataFormat,
                 break;
             }
 
                 break;
             }
 

+#if wxUSE_WXDIB

         case wxDF_DIB:
             {
                 wxBitmap *bitmap = (wxBitmap *)data;
 
         case wxDF_DIB:
             {
                 wxBitmap *bitmap = (wxBitmap *)data;
 

-                HGLOBAL hDIB = wxDIB::ConvertFromBitmap(GetHbitmapOf(*bitmap));
-                if ( hDIB )
+                if ( bitmap && bitmap->IsOk() )

                 {
                 {

-                    handle = ::SetClipboardData(CF_DIB, hDIB);
+                    wxDIB dib(*bitmap);
+                    if ( dib.IsOk() )
+                    {
+                        handle = ::SetClipboardData(CF_DIB, dib.Detach());
+                    }

                 }
                 break;
             }
                 }
                 break;
             }

+#endif

 
     // VZ: I'm told that this code works, but it doesn't seem to work for me
 
     // VZ: I'm told that this code works, but it doesn't seem to work for me

-    //     and, anyhow, I'd be highly surprized if it did. So I leave it here
+    //     and, anyhow, I'd be highly surprised if it did. So I leave it here

     //     but IMNSHO it is completely broken.
     //     but IMNSHO it is completely broken.

-#if wxUSE_METAFILE && !defined(wxMETAFILE_IS_ENH)
+#if wxUSE_METAFILE && !defined(wxMETAFILE_IS_ENH) && !defined(__WXWINCE__)

         case wxDF_METAFILE:
             {
                 wxMetafile *wxMF = (wxMetafile *)data;
         case wxDF_METAFILE:
             {
                 wxMetafile *wxMF = (wxMetafile *)data;


@@ -253,7 +251,7 @@ bool wxSetClipboardData(wxDataFormat dataFormat,
             }
 #endif // wxUSE_METAFILE
 
             }
 #endif // wxUSE_METAFILE
 

-#if wxUSE_ENH_METAFILE && !defined(__WIN16__)
+#if wxUSE_ENH_METAFILE && !defined(__WXWINCE__)

         case wxDF_ENHMETAFILE:
             {
                 wxEnhMetaFile *emf = (wxEnhMetaFile *)data;
         case wxDF_ENHMETAFILE:
             {
                 wxEnhMetaFile *emf = (wxEnhMetaFile *)data;


@@ -272,7 +270,7 @@ bool wxSetClipboardData(wxDataFormat dataFormat,
         default:
             {
                 wxLogError(_("Unsupported clipboard format."));
         default:
             {
                 wxLogError(_("Unsupported clipboard format."));

-                return FALSE;
+                return false;

             }
 
         case wxDF_OEMTEXT:
             }
 
         case wxDF_OEMTEXT:


@@ -299,20 +297,16 @@ bool wxSetClipboardData(wxDataFormat dataFormat,
                 handle = SetClipboardData(dataFormat, hGlobalMemory);
                 break;
             }
                 handle = SetClipboardData(dataFormat, hGlobalMemory);
                 break;
             }

-            // Only tested with non-Unicode, Visual C++ 6.0 so far
-#if defined(__VISUALC__) && !defined(UNICODE)
+            // Only tested with Visual C++ 6.0 so far
+#if defined(__VISUALC__)

         case wxDF_HTML:
             {
                 char* html = (char *)data;
         case wxDF_HTML:
             {
                 char* html = (char *)data;

-                
+

                 // Create temporary buffer for HTML header...
                 char *buf = new char [400 + strlen(html)];
                 // Create temporary buffer for HTML header...
                 char *buf = new char [400 + strlen(html)];

-                if(!buf) return FALSE;
-                
-                // Get clipboard id for HTML format...
-                static int cfid = 0;
-                if(!cfid) cfid = RegisterClipboardFormat(wxT("HTML Format"));
-                
+                if(!buf) return false;
+

                 // Create a template string for the HTML header...
                 strcpy(buf,
                     "Version:0.9\r\n"
                 // Create a template string for the HTML header...
                 strcpy(buf,
                     "Version:0.9\r\n"


@@ -322,7 +316,7 @@ bool wxSetClipboardData(wxDataFormat dataFormat,
                     "EndFragment:00000000\r\n"
                     "<html><body>\r\n"
                     "<!--StartFragment -->\r\n");
                     "EndFragment:00000000\r\n"
                     "<html><body>\r\n"
                     "<!--StartFragment -->\r\n");

-                
+

                 // Append the HTML...
                 strcat(buf, html);
                 strcat(buf, "\r\n");
                 // Append the HTML...
                 strcat(buf, html);
                 strcat(buf, "\r\n");


@@ -331,43 +325,43 @@ bool wxSetClipboardData(wxDataFormat dataFormat,
                     "<!--EndFragment-->\r\n"
                     "</body>\r\n"
                     "</html>");
                     "<!--EndFragment-->\r\n"
                     "</body>\r\n"
                     "</html>");

-                
+

                 // Now go back, calculate all the lengths, and write out the
                 // necessary header information. Note, wsprintf() truncates the
                 // string when you overwrite it so you follow up with code to replace
                 // the 0 appended at the end with a '\r'...
                 char *ptr = strstr(buf, "StartHTML");
                 // Now go back, calculate all the lengths, and write out the
                 // necessary header information. Note, wsprintf() truncates the
                 // string when you overwrite it so you follow up with code to replace
                 // the 0 appended at the end with a '\r'...
                 char *ptr = strstr(buf, "StartHTML");

-                wsprintf(ptr+10, "%08u", strstr(buf, "<html>") - buf);
+                sprintf(ptr+10, "%08u", (unsigned)(strstr(buf, "<html>") - buf));

                 *(ptr+10+8) = '\r';
                 *(ptr+10+8) = '\r';

-                
+

                 ptr = strstr(buf, "EndHTML");
                 ptr = strstr(buf, "EndHTML");

-                wsprintf(ptr+8, "%08u", strlen(buf));
+                sprintf(ptr+8, "%08u", (unsigned)strlen(buf));

                 *(ptr+8+8) = '\r';
                 *(ptr+8+8) = '\r';

-                
+

                 ptr = strstr(buf, "StartFragment");
                 ptr = strstr(buf, "StartFragment");

-                wsprintf(ptr+14, "%08u", strstr(buf, "<!--StartFrag") - buf);
+                sprintf(ptr+14, "%08u", (unsigned)(strstr(buf, "<!--StartFrag") - buf));

                 *(ptr+14+8) = '\r';
                 *(ptr+14+8) = '\r';

-                
+

                 ptr = strstr(buf, "EndFragment");
                 ptr = strstr(buf, "EndFragment");

-                wsprintf(ptr+12, "%08u", strstr(buf, "<!--EndFrag") - buf);
+                sprintf(ptr+12, "%08u", (unsigned)(strstr(buf, "<!--EndFrag") - buf));

                 *(ptr+12+8) = '\r';
                 *(ptr+12+8) = '\r';

-                
+

                 // Now you have everything in place ready to put on the
                 // clipboard.
                 // Now you have everything in place ready to put on the
                 // clipboard.

-                
+

                 // Allocate global memory for transfer...
                 HGLOBAL hText = GlobalAlloc(GMEM_MOVEABLE |GMEM_DDESHARE, strlen(buf)+4);
                 // Allocate global memory for transfer...
                 HGLOBAL hText = GlobalAlloc(GMEM_MOVEABLE |GMEM_DDESHARE, strlen(buf)+4);

-                
+

                 // Put your string in the global memory...
                 ptr = (char *)GlobalLock(hText);
                 strcpy(ptr, buf);
                 GlobalUnlock(hText);
                 // Put your string in the global memory...
                 ptr = (char *)GlobalLock(hText);
                 strcpy(ptr, buf);
                 GlobalUnlock(hText);

-                
-                handle = ::SetClipboardData(cfid, hText);
-                
+
+                handle = ::SetClipboardData(gs_htmlcfid, hText);
+

                 // Free memory...
                 GlobalFree(hText);
                 // Free memory...
                 GlobalFree(hText);

-                
+

                 // Clean up...
                 delete [] buf;
                 break;
                 // Clean up...
                 delete [] buf;
                 break;


@@ -379,10 +373,10 @@ bool wxSetClipboardData(wxDataFormat dataFormat,
     {
         wxLogSysError(_("Failed to set clipboard data."));
 
     {
         wxLogSysError(_("Failed to set clipboard data."));
 

-        return FALSE;
+        return false;

     }
 
     }
 

-    return TRUE;
+    return true;

 }
 
 void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
 }
 
 void *wxGetClipboardData(wxDataFormat dataFormat, long *len)


@@ -391,6 +385,7 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
 
     switch ( dataFormat )
     {
 
     switch ( dataFormat )
     {

+#ifndef __WXWINCE__

         case wxDF_BITMAP:
             {
                 BITMAP bm;
         case wxDF_BITMAP:
             {
                 BITMAP bm;


@@ -432,13 +427,10 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
                 wxBM->SetWidth(bm.bmWidth);
                 wxBM->SetHeight(bm.bmHeight);
                 wxBM->SetDepth(bm.bmPlanes);
                 wxBM->SetWidth(bm.bmWidth);
                 wxBM->SetHeight(bm.bmHeight);
                 wxBM->SetDepth(bm.bmPlanes);

-#if WXWIN_COMPATIBILITY_2
-                wxBM->SetOk(TRUE);
-#endif // WXWIN_COMPATIBILITY_2

                 retval = wxBM;
                 break;
             }
                 retval = wxBM;
                 break;
             }

-
+#endif

         case wxDF_METAFILE:
         case CF_SYLK:
         case CF_DIF:
         case wxDF_METAFILE:
         case CF_SYLK:
         case CF_DIF:


@@ -466,11 +458,11 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
                 if (!s)
                     break;
 
                 if (!s)
                     break;
 

-                LPSTR lpGlobalMemory = (LPSTR)::GlobalLock(hGlobalMemory);
+                LPSTR lpGlobalMemory = (LPSTR) GlobalLock(hGlobalMemory);

 
                 memcpy(s, lpGlobalMemory, hsize);
 
 
                 memcpy(s, lpGlobalMemory, hsize);
 

-                ::GlobalUnlock(hGlobalMemory);
+                GlobalUnlock(hGlobalMemory);

 
                 retval = s;
                 break;
 
                 retval = s;
                 break;


@@ -490,11 +482,11 @@ void *wxGetClipboardData(wxDataFormat dataFormat, long *len)
                 if ( !buf )
                     break;
 
                 if ( !buf )
                     break;
 

-                LPSTR lpGlobalMemory = (LPSTR)::GlobalLock(hGlobalMemory);
+                LPSTR lpGlobalMemory = (LPSTR) GlobalLock(hGlobalMemory);

 
                 memcpy(buf, lpGlobalMemory, size);
 
 
                 memcpy(buf, lpGlobalMemory, size);
 

-                ::GlobalUnlock(hGlobalMemory);
+                GlobalUnlock(hGlobalMemory);

 
                 retval = buf;
                 break;
 
                 retval = buf;
                 break;


@@ -534,54 +526,85 @@ IMPLEMENT_DYNAMIC_CLASS(wxClipboard, wxObject)
 
 wxClipboard::wxClipboard()
 {
 
 wxClipboard::wxClipboard()
 {

-    m_clearOnExit = FALSE;
-    m_isOpened = FALSE;
+#if wxUSE_OLE_CLIPBOARD
+    wxOleInitialize();
+#endif
+
+    m_lastDataObject = NULL;
+    m_isOpened = false;

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

-    if ( m_clearOnExit )
+    if ( m_lastDataObject )

     {
         Clear();
     }
     {
         Clear();
     }

+
+#if wxUSE_OLE_CLIPBOARD
+    wxOleUninitialize();
+#endif

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

+    if ( IsUsingPrimarySelection() )
+        return;
+

 #if wxUSE_OLE_CLIPBOARD
 #if wxUSE_OLE_CLIPBOARD

-    if ( FAILED(OleSetClipboard(NULL)) )
+    if (m_lastDataObject)

     {
     {

-        wxLogLastError(wxT("OleSetClipboard(NULL)"));
+        // don't touch data set by other applications
+        HRESULT hr = OleIsCurrentClipboard(m_lastDataObject);
+        if (S_OK == hr)
+        {
+            hr = OleSetClipboard(NULL);
+            if ( FAILED(hr) )
+            {
+                wxLogApiError(wxT("OleSetClipboard(NULL)"), hr);
+            }
+        }
+        m_lastDataObject = NULL;

     }
     }

-#endif
+#endif // wxUSE_OLE_CLIPBOARD

 }
 
 bool wxClipboard::Flush()
 {
 #if wxUSE_OLE_CLIPBOARD
 }
 
 bool wxClipboard::Flush()
 {
 #if wxUSE_OLE_CLIPBOARD

-    if ( FAILED(OleFlushClipboard()) )
-    {
-        wxLogLastError(wxT("OleFlushClipboard"));
-
-        return FALSE;
-    }
-    else
+    if (m_lastDataObject)

     {
     {

-        m_clearOnExit = FALSE;
+        // don't touch data set by other applications
+        HRESULT hr = OleIsCurrentClipboard(m_lastDataObject);
+        m_lastDataObject = NULL;
+        if (S_OK == hr)
+        {
+            hr = OleFlushClipboard();
+            if ( FAILED(hr) )
+            {
+                wxLogApiError(wxT("OleFlushClipboard"), hr);

 
 

-        return TRUE;
+                return false;
+            }
+            return true;
+        }

     }
     }

+    return false;

 #else // !wxUSE_OLE_CLIPBOARD
 #else // !wxUSE_OLE_CLIPBOARD

-    return FALSE;
+    return false;

 #endif // wxUSE_OLE_CLIPBOARD/!wxUSE_OLE_CLIPBOARD
 }
 
 bool wxClipboard::Open()
 {
 #endif // wxUSE_OLE_CLIPBOARD/!wxUSE_OLE_CLIPBOARD
 }
 
 bool wxClipboard::Open()
 {

+    // Get clipboard id for HTML format...
+    if(!gs_htmlcfid)
+        gs_htmlcfid = RegisterClipboardFormat(wxT("HTML Format"));
+

     // OLE opens clipboard for us
     // OLE opens clipboard for us

-    m_isOpened = TRUE;
+    m_isOpened = true;

 #if wxUSE_OLE_CLIPBOARD
 #if wxUSE_OLE_CLIPBOARD

-    return TRUE;
+    return true;

 #else
     return wxOpenClipboard();
 #endif
 #else
     return wxOpenClipboard();
 #endif


@@ -598,6 +621,9 @@ bool wxClipboard::IsOpened() const
 
 bool wxClipboard::SetData( wxDataObject *data )
 {
 
 bool wxClipboard::SetData( wxDataObject *data )
 {

+    if ( IsUsingPrimarySelection() )
+        return false;
+

 #if !wxUSE_OLE_CLIPBOARD
     (void)wxEmptyClipboard();
 #endif // wxUSE_OLE_CLIPBOARD
 #if !wxUSE_OLE_CLIPBOARD
     (void)wxEmptyClipboard();
 #endif // wxUSE_OLE_CLIPBOARD


@@ -605,12 +631,15 @@ bool wxClipboard::SetData( wxDataObject *data )
     if ( data )
         return AddData(data);
     else
     if ( data )
         return AddData(data);
     else

-        return TRUE;
+        return true;

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

-    wxCHECK_MSG( data, FALSE, wxT("data is invalid") );
+    if ( IsUsingPrimarySelection() )
+        return false;
+
+    wxCHECK_MSG( data, false, wxT("data is invalid") );

 
 #if wxUSE_OLE_CLIPBOARD
     HRESULT hr = OleSetClipboard(data->GetInterface());
 
 #if wxUSE_OLE_CLIPBOARD
     HRESULT hr = OleSetClipboard(data->GetInterface());


@@ -620,9 +649,15 @@ bool wxClipboard::AddData( wxDataObject *data )
 
         // don't free anything in this case
 
 
         // don't free anything in this case
 

-        return FALSE;
+        return false;

     }
 
     }
 

+    // we have to call either OleSetClipboard(NULL) or OleFlushClipboard() when
+    // using OLE clipboard when the app terminates - by default, we call
+    // OleSetClipboard(NULL) which won't waste RAM, but the app can call
+    // wxClipboard::Flush() to change this
+    m_lastDataObject = data->GetInterface();
+

     // we have a problem here because we should delete wxDataObject, but we
     // can't do it because IDataObject which we just gave to the clipboard
     // would try to use it when it will need the data. IDataObject is ref
     // we have a problem here because we should delete wxDataObject, but we
     // can't do it because IDataObject which we just gave to the clipboard
     // would try to use it when it will need the data. IDataObject is ref


@@ -630,15 +665,9 @@ bool wxClipboard::AddData( wxDataObject *data )
     // and tell it to delete wxDataObject when it is deleted itself.
     data->SetAutoDelete();
 
     // and tell it to delete wxDataObject when it is deleted itself.
     data->SetAutoDelete();
 

-    // we have to call either OleSetClipboard(NULL) or OleFlushClipboard() when
-    // using OLE clipboard when the app terminates - by default, we call
-    // OleSetClipboard(NULL) which won't waste RAM, but the app can call
-    // wxClipboard::Flush() to chaneg this
-    m_clearOnExit = TRUE;
-
-    return TRUE;
+    return true;

 #elif wxUSE_DATAOBJ
 #elif wxUSE_DATAOBJ

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

 
     wxDataFormat format = data->GetPreferredFormat();
 
 
     wxDataFormat format = data->GetPreferredFormat();
 


@@ -665,8 +694,8 @@ bool wxClipboard::AddData( wxDataObject *data )
         {
 #if 1
             // TODO
         {
 #if 1
             // TODO

-            wxLogError("Not implemented because wxMetafileDataObject does not contain width and height values.");
-            return FALSE;
+            wxLogError(wxT("Not implemented because wxMetafileDataObject does not contain width and height values."));
+            return false;

 #else
             wxMetafileDataObject* metaFileDataObject =
                 (wxMetafileDataObject*) data;
 #else
             wxMetafileDataObject* metaFileDataObject =
                 (wxMetafileDataObject*) data;


@@ -683,31 +712,34 @@ bool wxClipboard::AddData( wxDataObject *data )
 // This didn't compile, of course
 //            return wxSetClipboardData(data);
             // TODO
 // This didn't compile, of course
 //            return wxSetClipboardData(data);
             // TODO

-            wxLogError("Not implemented.");
-            return FALSE;
+            wxLogError(wxT("Not implemented."));
+            return false;

         }
     }
 #else // !wxUSE_DATAOBJ
         }
     }
 #else // !wxUSE_DATAOBJ

-    return FALSE;
+    return false;

 #endif // wxUSE_DATAOBJ/!wxUSE_DATAOBJ
 }
 
 void wxClipboard::Close()
 {
 #endif // wxUSE_DATAOBJ/!wxUSE_DATAOBJ
 }
 
 void wxClipboard::Close()
 {

-    m_isOpened = FALSE;
+    m_isOpened = false;

     // OLE closes clipboard for us
 #if !wxUSE_OLE_CLIPBOARD
     wxCloseClipboard();
 #endif
 }
 
     // OLE closes clipboard for us
 #if !wxUSE_OLE_CLIPBOARD
     wxCloseClipboard();
 #endif
 }
 

-bool wxClipboard::IsSupported( wxDataFormat format )
+bool wxClipboard::IsSupported( const wxDataFormat& format )

 {
 {

-    return wxIsClipboardFormatAvailable(format);
+    return !IsUsingPrimarySelection() && wxIsClipboardFormatAvailable(format);

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

+    if ( IsUsingPrimarySelection() )
+        return false;
+

 #if wxUSE_OLE_CLIPBOARD
     IDataObject *pDataObject = NULL;
     HRESULT hr = OleGetClipboard(&pDataObject);
 #if wxUSE_OLE_CLIPBOARD
     IDataObject *pDataObject = NULL;
     HRESULT hr = OleGetClipboard(&pDataObject);


@@ -715,7 +747,7 @@ bool wxClipboard::GetData( wxDataObject& data )
     {
         wxLogSysError(hr, _("Failed to get data from the clipboard"));
 
     {
         wxLogSysError(hr, _("Failed to get data from the clipboard"));
 

-        return FALSE;
+        return false;

     }
 
     // build the list of supported formats
     }
 
     // build the list of supported formats


@@ -735,9 +767,16 @@ bool wxClipboard::GetData( wxDataObject& data )
 
     data.GetAllFormats(formats, wxDataObject::Set);
 
 
     data.GetAllFormats(formats, wxDataObject::Set);
 

+    // get the data for the given formats
+    FORMATETC formatEtc;
+    CLIPFORMAT cf;
+    bool result = false;
+
+    // enumerate all explicit formats on the clipboard.
+    // note that this does not include implicit / synthetic (automatically
+    // converted) formats.
+#if wxDEBUG_LEVEL >= 2

     // get the format enumerator
     // get the format enumerator

-    bool result = FALSE;
-    wxArrayInt supportedFormats;

     IEnumFORMATETC *pEnumFormatEtc = NULL;
     hr = pDataObject->EnumFormatEtc(DATADIR_GET, &pEnumFormatEtc);
     if ( FAILED(hr) || !pEnumFormatEtc )
     IEnumFORMATETC *pEnumFormatEtc = NULL;
     hr = pDataObject->EnumFormatEtc(DATADIR_GET, &pEnumFormatEtc);
     if ( FAILED(hr) || !pEnumFormatEtc )


@@ -748,7 +787,6 @@ bool wxClipboard::GetData( wxDataObject& data )
     else
     {
         // ask for the supported formats and see if there are any we support
     else
     {
         // ask for the supported formats and see if there are any we support

-        FORMATETC formatEtc;

         for ( ;; )
         {
             ULONG nCount;
         for ( ;; )
         {
             ULONG nCount;


@@ -761,108 +799,101 @@ bool wxClipboard::GetData( wxDataObject& data )
                 break;
             }
 
                 break;
             }
 

-            CLIPFORMAT cf = formatEtc.cfFormat;
+            cf = formatEtc.cfFormat;

 
 

-#ifdef __WXDEBUG__

             wxLogTrace(wxTRACE_OleCalls,
                        wxT("Object on the clipboard supports format %s."),
                        wxDataObject::GetFormatName(cf));
             wxLogTrace(wxTRACE_OleCalls,
                        wxT("Object on the clipboard supports format %s."),
                        wxDataObject::GetFormatName(cf));

-#endif // Debug
-
-            // is supported?
-            for ( size_t n = 0; n < nFormats; n++ )
-            {
-                if ( formats[n].GetFormatId() == cf )
-                {
-                    if ( supportedFormats.Index(cf) == wxNOT_FOUND )
-                    {
-                        supportedFormats.Add(cf);
-                    }
-                }
-            }

         }
 
         pEnumFormatEtc->Release();
     }
         }
 
         pEnumFormatEtc->Release();
     }

+#endif // wxDEBUG_LEVEL >= 2

 
 

-    if ( formats != &format )
+    STGMEDIUM medium;
+    // stop at the first valid format found on the clipboard
+    for ( size_t n = 0; !result && (n < nFormats); n++ )

     {
     {

-        delete [] formats;
-    }
-    //else: we didn't allocate any memory
+        // convert to NativeFormat Id
+        cf = formats[n].GetFormatId();

 
 

-    if ( !supportedFormats.IsEmpty() )
-    {
-        FORMATETC formatEtc;
+        if (cf == wxDF_HTML)
+            cf = gs_htmlcfid;
+        // if the format is not available, try the next one
+        // this test includes implicit / sythetic formats
+        if ( !::IsClipboardFormatAvailable(cf) )
+            continue;
+
+        formatEtc.cfFormat = cf;

         formatEtc.ptd      = NULL;
         formatEtc.dwAspect = DVASPECT_CONTENT;
         formatEtc.lindex   = -1;
 
         formatEtc.ptd      = NULL;
         formatEtc.dwAspect = DVASPECT_CONTENT;
         formatEtc.lindex   = -1;
 

-        size_t nSupportedFormats = supportedFormats.GetCount();
-        for ( size_t n = 0; !result && (n < nSupportedFormats); n++ )
+        // use the appropriate tymed
+        switch ( formatEtc.cfFormat )

         {
         {

-            STGMEDIUM medium;
-            formatEtc.cfFormat = supportedFormats[n];
+            case CF_BITMAP:
+                formatEtc.tymed = TYMED_GDI;
+                break;

 
 

-            // use the appropriate tymed
-            switch ( formatEtc.cfFormat )
-            {
-                case CF_BITMAP:
-                    formatEtc.tymed = TYMED_GDI;
-                    break;
+#ifndef __WXWINCE__
+            case CF_METAFILEPICT:
+                formatEtc.tymed = TYMED_MFPICT;
+                break;

 
 

-                case CF_METAFILEPICT:
-                    formatEtc.tymed = TYMED_MFPICT;
-                    break;
+            case CF_ENHMETAFILE:
+                formatEtc.tymed = TYMED_ENHMF;
+                break;
+#endif

 
 

-                case CF_ENHMETAFILE:
-                    formatEtc.tymed = TYMED_ENHMF;
-                    break;
+            default:
+                formatEtc.tymed = TYMED_HGLOBAL;
+        }

 
 

-                default:
-                    formatEtc.tymed = TYMED_HGLOBAL;
+        // try to get data
+        hr = pDataObject->GetData(&formatEtc, &medium);
+        if ( FAILED(hr) )
+        {
+            // try other tymed for GDI objects
+            if ( formatEtc.cfFormat == CF_BITMAP )
+            {
+                formatEtc.tymed = TYMED_HGLOBAL;
+                hr = pDataObject->GetData(&formatEtc, &medium);

             }
             }

+        }

 
 

-            // try to get data
-            hr = pDataObject->GetData(&formatEtc, &medium);
+        if ( SUCCEEDED(hr) )
+        {
+            // pass the data to the data object
+            hr = data.GetInterface()->SetData(&formatEtc, &medium, true);

             if ( FAILED(hr) )
             {
             if ( FAILED(hr) )
             {

-                // try other tymed for GDI objects
-                if ( formatEtc.cfFormat == CF_BITMAP )
-                {
-                    formatEtc.tymed = TYMED_HGLOBAL;
-                    hr = pDataObject->GetData(&formatEtc, &medium);
-                }
-            }
+                wxLogDebug(wxT("Failed to set data in wxIDataObject"));

 
 

-            if ( SUCCEEDED(hr) )
+                // IDataObject only takes the ownership of data if it
+                // successfully got it - which is not the case here
+                ReleaseStgMedium(&medium);
+            }
+            else

             {
             {

-                // pass the data to the data object
-                hr = data.GetInterface()->SetData(&formatEtc, &medium, TRUE);
-                if ( FAILED(hr) )
-                {
-                    wxLogDebug(wxT("Failed to set data in wxIDataObject"));
-
-                    // IDataObject only takes the ownership of data if it
-                    // successfully got it - which is not the case here
-                    ReleaseStgMedium(&medium);
-                }
-                else
-                {
-                    result = TRUE;
-                }
+                result = true;

             }
             }

-            //else: unsupported tymed?

         }
         }

+        //else: unsupported tymed?

     }
     }

-    //else: unsupported format
+
+    if ( formats != &format )
+    {
+        delete [] formats;
+    }
+    //else: we didn't allocate any memory

 
     // clean up and return
     pDataObject->Release();
 
     return result;
 #elif wxUSE_DATAOBJ
 
     // clean up and return
     pDataObject->Release();
 
     return result;
 #elif wxUSE_DATAOBJ

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

 
     wxDataFormat format = data.GetPreferredFormat();
     switch ( format )
 
     wxDataFormat format = data.GetPreferredFormat();
     switch ( format )


@@ -873,12 +904,12 @@ bool wxClipboard::GetData( wxDataObject& data )
             wxTextDataObject& textDataObject = (wxTextDataObject &)data;
             char* s = (char*)wxGetClipboardData(format);
             if ( !s )
             wxTextDataObject& textDataObject = (wxTextDataObject &)data;
             char* s = (char*)wxGetClipboardData(format);
             if ( !s )

-                return FALSE;
+                return false;

 
 

-            textDataObject.SetText(s);
+            textDataObject.SetText(wxString::FromAscii(s));

             delete [] s;
 
             delete [] s;
 

-            return TRUE;
+            return true;

         }
 
         case wxDF_BITMAP:
         }
 
         case wxDF_BITMAP:


@@ -887,12 +918,12 @@ bool wxClipboard::GetData( wxDataObject& data )
             wxBitmapDataObject& bitmapDataObject = (wxBitmapDataObject &)data;
             wxBitmap* bitmap = (wxBitmap *)wxGetClipboardData(data.GetPreferredFormat());
             if ( !bitmap )
             wxBitmapDataObject& bitmapDataObject = (wxBitmapDataObject &)data;
             wxBitmap* bitmap = (wxBitmap *)wxGetClipboardData(data.GetPreferredFormat());
             if ( !bitmap )

-                return FALSE;
+                return false;

 
             bitmapDataObject.SetBitmap(*bitmap);
             delete bitmap;
 
 
             bitmapDataObject.SetBitmap(*bitmap);
             delete bitmap;
 

-            return TRUE;
+            return true;

         }
 #if wxUSE_METAFILE
         case wxDF_METAFILE:
         }
 #if wxUSE_METAFILE
         case wxDF_METAFILE:


@@ -900,21 +931,20 @@ bool wxClipboard::GetData( wxDataObject& data )
             wxMetafileDataObject& metaFileDataObject = (wxMetafileDataObject &)data;
             wxMetafile* metaFile = (wxMetafile *)wxGetClipboardData(wxDF_METAFILE);
             if ( !metaFile )
             wxMetafileDataObject& metaFileDataObject = (wxMetafileDataObject &)data;
             wxMetafile* metaFile = (wxMetafile *)wxGetClipboardData(wxDF_METAFILE);
             if ( !metaFile )

-                return FALSE;
+                return false;

 
             metaFileDataObject.SetMetafile(*metaFile);
             delete metaFile;
 
 
             metaFileDataObject.SetMetafile(*metaFile);
             delete metaFile;
 

-            return TRUE;
+            return true;

         }
 #endif // wxUSE_METAFILE
     }
         }
 #endif // wxUSE_METAFILE
     }

-    return FALSE;
+    return false;

 #else // !wxUSE_DATAOBJ
     wxFAIL_MSG( wxT("no clipboard implementation") );
 #else // !wxUSE_DATAOBJ
     wxFAIL_MSG( wxT("no clipboard implementation") );

-    return FALSE;
+    return false;

 #endif // wxUSE_OLE_CLIPBOARD/wxUSE_DATAOBJ
 }
 
 #endif // wxUSE_CLIPBOARD
 #endif // wxUSE_OLE_CLIPBOARD/wxUSE_DATAOBJ
 }
 
 #endif // wxUSE_CLIPBOARD

-