X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/1dd989e1bf4c3667d27b3c27a9cf64a66484ee90..ad653fa23069c5d9378247084f03c9a718c3ad62:/src/gtk/dataobj.cpp diff --git a/src/gtk/dataobj.cpp b/src/gtk/dataobj.cpp index 401dca0db0..db02431737 100644 --- a/src/gtk/dataobj.cpp +++ b/src/gtk/dataobj.cpp @@ -1,5 +1,5 @@ /////////////////////////////////////////////////////////////////////////////// -// Name: dataobj.cpp +// Name: src/gtk/dataobj.cpp // Purpose: wxDataObject class // Author: Robert Roebling // Id: $Id$ @@ -7,26 +7,33 @@ // Licence: wxWindows licence /////////////////////////////////////////////////////////////////////////////// -#ifdef __GNUG__ - #pragma implementation "dataobj.h" -#endif +// For compilers that support precompilation, includes "wx.h". +#include "wx/wxprec.h" + +#if wxUSE_DATAOBJ #include "wx/dataobj.h" -#include "wx/app.h" -#include "wx/debug.h" -#include "wx/mstream.h" -#include "wx/image.h" -#include "gdk/gdk.h" +#ifndef WX_PRECOMP + #include "wx/log.h" + #include "wx/app.h" + #include "wx/image.h" +#endif + +#include "wx/mstream.h" +#include "wx/uri.h" +#include "wx/gtk/private.h" //------------------------------------------------------------------------- // global data //------------------------------------------------------------------------- GdkAtom g_textAtom = 0; +GdkAtom g_altTextAtom = 0; GdkAtom g_pngAtom = 0; GdkAtom g_fileAtom = 0; +GdkAtom g_htmlAtom = 0; //------------------------------------------------------------------------- // wxDataFormat @@ -34,7 +41,14 @@ GdkAtom g_fileAtom = 0; wxDataFormat::wxDataFormat() { - PrepareFormats(); + // do *not* call PrepareFormats() from here for 2 reasons: + // + // 1. we will have time to do it later because some other Set function + // must be called before we really need them + // + // 2. doing so prevents us from declaring global wxDataFormats because + // calling PrepareFormats (and thus gdk_atom_intern) before GDK is + // initialised will result in a crash m_type = wxDF_INVALID; m_format = (GdkAtom) 0; } @@ -45,13 +59,7 @@ wxDataFormat::wxDataFormat( wxDataFormatId type ) SetType( type ); } -wxDataFormat::wxDataFormat( const wxChar *id ) -{ - PrepareFormats(); - SetId( id ); -} - -wxDataFormat::wxDataFormat( const wxString &id ) +void wxDataFormat::InitFromString( const wxString &id ) { PrepareFormats(); SetId( id ); @@ -65,10 +73,22 @@ wxDataFormat::wxDataFormat( NativeFormat format ) void wxDataFormat::SetType( wxDataFormatId type ) { + PrepareFormats(); + m_type = type; +#if wxUSE_UNICODE + if (m_type == wxDF_UNICODETEXT) + m_format = g_textAtom; + else if (m_type == wxDF_TEXT) + m_format = g_altTextAtom; +#else // !wxUSE_UNICODE + // notice that we don't map wxDF_UNICODETEXT to g_textAtom here, this + // would lead the code elsewhere to treat data objects with this format as + // containing UTF-8 data which is not true if (m_type == wxDF_TEXT) m_format = g_textAtom; +#endif // wxUSE_UNICODE/!wxUSE_UNICODE else if (m_type == wxDF_BITMAP) m_format = g_pngAtom; @@ -76,6 +96,9 @@ void wxDataFormat::SetType( wxDataFormatId type ) if (m_type == wxDF_FILENAME) m_format = g_fileAtom; else + if (m_type == wxDF_HTML) + m_format = g_htmlAtom; + else { wxFAIL_MSG( wxT("invalid dataformat") ); } @@ -88,15 +111,23 @@ wxDataFormatId wxDataFormat::GetType() const wxString wxDataFormat::GetId() const { - wxString ret( gdk_atom_name( m_format ) ); // this will convert from ascii to Unicode - return ret; + wxGtkString atom_name(gdk_atom_name(m_format)); + return wxString::FromAscii(atom_name); } void wxDataFormat::SetId( NativeFormat format ) { + PrepareFormats(); m_format = format; if (m_format == g_textAtom) +#if wxUSE_UNICODE + m_type = wxDF_UNICODETEXT; +#else + m_type = wxDF_TEXT; +#endif + else + if (m_format == g_altTextAtom) m_type = wxDF_TEXT; else if (m_format == g_pngAtom) @@ -104,53 +135,74 @@ void wxDataFormat::SetId( NativeFormat format ) else if (m_format == g_fileAtom) m_type = wxDF_FILENAME; + else + if (m_format == g_htmlAtom) + m_type = wxDF_HTML; else m_type = wxDF_PRIVATE; } -void wxDataFormat::SetId( const wxChar *id ) +void wxDataFormat::SetId( const wxString& id ) { + PrepareFormats(); m_type = wxDF_PRIVATE; - wxString tmp( id ); - m_format = gdk_atom_intern( wxMBSTRINGCAST tmp.mbc_str(), FALSE ); // what is the string cast for? + m_format = gdk_atom_intern( id.ToAscii(), FALSE ); } void wxDataFormat::PrepareFormats() { - if (!g_textAtom) - g_textAtom = gdk_atom_intern( "STRING", FALSE ); - if (!g_pngAtom) + // VZ: GNOME included in RedHat 6.1 uses the MIME types below and not the + // atoms STRING and file:ALL as the old code was, but normal X apps + // use STRING for text selection when transferring the data via + // clipboard, for example, so do use STRING for now (GNOME apps will + // probably support STRING as well for compatibility anyhow), but use + // text/uri-list for file dnd because compatibility is not important + // here (with whom?) + if (!g_textAtom) + { +#if wxUSE_UNICODE + g_textAtom = gdk_atom_intern( "UTF8_STRING", FALSE ); + g_altTextAtom = gdk_atom_intern( "STRING", FALSE ); +#else + g_textAtom = gdk_atom_intern( "STRING" /* "text/plain" */, FALSE ); +#endif + } + if (!g_pngAtom) g_pngAtom = gdk_atom_intern( "image/png", FALSE ); - if (!g_fileAtom) - g_fileAtom = gdk_atom_intern( "file:ALL", FALSE ); + if (!g_fileAtom) + g_fileAtom = gdk_atom_intern( "text/uri-list", FALSE ); + if (!g_htmlAtom) + g_htmlAtom = gdk_atom_intern( "text/html", FALSE ); } //------------------------------------------------------------------------- // wxDataObject //------------------------------------------------------------------------- -IMPLEMENT_ABSTRACT_CLASS( wxDataObject, wxObject ) - wxDataObject::wxDataObject() { } wxDataObject::~wxDataObject() { + // dtor is empty but needed for Darwin and AIX -- otherwise it doesn't link } -bool wxDataObject::IsSupportedFormat(const wxDataFormat& format) const +bool wxDataObject::IsSupportedFormat(const wxDataFormat& format, Direction dir) const { - size_t nFormatCount = GetFormatCount(); - if ( nFormatCount == 1 ) { + size_t nFormatCount = GetFormatCount(dir); + if ( nFormatCount == 1 ) + { return format == GetPreferredFormat(); } - else { + else + { wxDataFormat *formats = new wxDataFormat[nFormatCount]; - GetAllFormats(formats); + GetAllFormats(formats,dir); size_t n; - for ( n = 0; n < nFormatCount; n++ ) { + for ( n = 0; n < nFormatCount; n++ ) + { if ( formats[n] == format ) break; } @@ -162,231 +214,307 @@ bool wxDataObject::IsSupportedFormat(const wxDataFormat& format) const } } - // ---------------------------------------------------------------------------- // wxTextDataObject // ---------------------------------------------------------------------------- -IMPLEMENT_DYNAMIC_CLASS( wxTextDataObject, wxDataObject ) +#if wxUSE_UNICODE -wxTextDataObject::wxTextDataObject() -{ -} - -wxTextDataObject::wxTextDataObject(const wxString& strText) - : m_strText(strText) -{ -} - -size_t wxTextDataObject::GetDataSize(const wxDataFormat& format) const -{ - return m_strText.Len() + 1; // +1 for trailing '\0'of course -} - -bool wxTextDataObject::GetDataHere(const wxDataFormat& format, void *buf) const -{ - memcpy(buf, m_strText.c_str(), GetDataSize(format)); - return TRUE; +void +wxTextDataObject::GetAllFormats(wxDataFormat *formats, + wxDataObjectBase::Direction WXUNUSED(dir)) const +{ + *formats++ = GetPreferredFormat(); + *formats = g_altTextAtom; } -bool wxTextDataObject::SetData(const wxDataFormat& format, const void *buf) -{ - m_strText = (const wxChar *)buf; - return TRUE; -} +#endif // wxUSE_UNICODE // ---------------------------------------------------------------------------- // wxFileDataObject // ---------------------------------------------------------------------------- -IMPLEMENT_DYNAMIC_CLASS( wxFileDataObject, wxDataObject ) - -wxFileDataObject::wxFileDataObject() +bool wxFileDataObject::GetDataHere(void *buf) const { -} + wxString filenames; -void wxFileDataObject::AddFile( const wxString &file ) -{ - m_files += file; - m_files += (wxChar)0; -} + for (size_t i = 0; i < m_filenames.GetCount(); i++) + { + filenames += wxT("file:"); + filenames += m_filenames[i]; + filenames += wxT("\r\n"); + } -wxString wxFileDataObject::GetFiles() const -{ - return m_files; + memcpy( buf, filenames.mbc_str(), filenames.length() + 1 ); + + return true; } -bool wxFileDataObject::GetDataHere(const wxDataFormat& format, void *buf) const +size_t wxFileDataObject::GetDataSize() const { - if (format == wxDF_FILENAME) + size_t res = 0; + + for (size_t i = 0; i < m_filenames.GetCount(); i++) { - memcpy( buf, m_files.mbc_str(), m_files.Len() + 1 ); - return TRUE; + // This is junk in UTF-8 + res += m_filenames[i].length(); + res += 5 + 2; // "file:" (5) + "\r\n" (2) } - - return FALSE; + + return res + 1; } -size_t wxFileDataObject::GetDataSize(const wxDataFormat& format) const +bool wxFileDataObject::SetData(size_t WXUNUSED(size), const void *buf) { - if (format != wxDF_FILENAME) return 0; - - return m_files.Len() + 1; + // we get data in the text/uri-list format, i.e. as a sequence of URIs + // (filenames prefixed by "file:") delimited by "\r\n". size includes + // the trailing zero (in theory, not for Nautilus in early GNOME + // versions). + + m_filenames.Empty(); + + const gchar *nexttemp = (const gchar*) buf; + for ( ; ; ) + { + int len = 0; + const gchar *temp = nexttemp; + for (;;) + { + if (temp[len] == 0) + { + if (len > 0) + { + // if an app omits '\r''\n' + nexttemp = temp+len; + break; + } + + return true; + } + if (temp[len] == '\r') + { + if (temp[len+1] == '\n') + nexttemp = temp+len+2; + else + nexttemp = temp+len+1; + break; + } + len++; + } + + if (len == 0) + break; + + // required to give it a trailing zero + gchar *uri = g_strndup( temp, len ); + + gchar *fn = g_filename_from_uri( uri, NULL, NULL ); + + g_free( uri ); + + if (fn) + { + AddFile( wxConvFileName->cMB2WX( fn ) ); + g_free( fn ); + } + } + + return true; } -bool wxFileDataObject::SetData(const wxDataFormat& format, const void *buf) +void wxFileDataObject::AddFile( const wxString &filename ) { - if (format != wxDF_FILENAME) - return FALSE; - - m_files = (char*)(buf); // this is so ugly, I cannot look at it - - return TRUE; + m_filenames.Add( filename ); } // ---------------------------------------------------------------------------- // wxBitmapDataObject // ---------------------------------------------------------------------------- -IMPLEMENT_DYNAMIC_CLASS( wxBitmapDataObject, wxDataObject ) - wxBitmapDataObject::wxBitmapDataObject() { - m_pngData = (void*)NULL; - m_pngSize = 0; + Init(); } wxBitmapDataObject::wxBitmapDataObject( const wxBitmap& bitmap ) + : wxBitmapDataObjectBase(bitmap) { - m_pngData = (void*)NULL; - m_pngSize = 0; - m_bitmap = bitmap; + Init(); + DoConvertToPng(); } wxBitmapDataObject::~wxBitmapDataObject() { - if (m_pngData) - delete[] m_pngData; + Clear(); } void wxBitmapDataObject::SetBitmap( const wxBitmap &bitmap ) { - if (m_pngData) - delete[] m_pngData; - m_pngData = (void*)NULL; - m_pngSize = 0; - - m_bitmap = bitmap; - DoConvertToPng(); -} + ClearAll(); -size_t wxBitmapDataObject::GetDataSize(const wxDataFormat& format) const -{ - if (format != wxDF_BITMAP) return 0; - - return m_pngSize; + wxBitmapDataObjectBase::SetBitmap(bitmap); + + DoConvertToPng(); } -bool wxBitmapDataObject::GetDataHere(const wxDataFormat& format, void *buf) const +bool wxBitmapDataObject::GetDataHere(void *buf) const { - if (format != wxDF_BITMAP) return FALSE; - - if (m_pngSize > 0) + if ( !m_pngSize ) { - memcpy(buf, m_pngData, m_pngSize); - return TRUE; + wxFAIL_MSG( wxT("attempt to copy empty bitmap failed") ); + + return false; } - - return FALSE; -} -bool wxBitmapDataObject::SetData(const wxDataFormat& format, const void *buf) -{ - if (m_pngData) delete[] m_pngData; - m_pngData = (void*) NULL; - m_pngSize = 0; - m_bitmap = wxNullBitmap; - - return FALSE; + memcpy(buf, m_pngData, m_pngSize); + + return true; } -void wxBitmapDataObject::SetPngData(const void *buf, size_t size) +bool wxBitmapDataObject::SetData(size_t size, const void *buf) { - if (m_pngData) delete[] m_pngData; + Clear(); + + wxCHECK_MSG( wxImage::FindHandler(wxBITMAP_TYPE_PNG) != NULL, + false, wxT("You must call wxImage::AddHandler(new wxPNGHandler); to be able to use clipboard with bitmaps!") ); + m_pngSize = size; - m_pngData = (void*) new char[m_pngSize]; - - memcpy( m_pngData, buf, m_pngSize ); - - wxMemoryInputStream mstream( (char*) m_pngData, m_pngSize ); + m_pngData = malloc(m_pngSize); + + memcpy(m_pngData, buf, m_pngSize); + + wxMemoryInputStream mstream((char*) m_pngData, m_pngSize); wxImage image; - wxPNGHandler handler; - handler.LoadFile( &image, mstream ); - m_bitmap = image.ConvertToBitmap(); + if ( !image.LoadFile( mstream, wxBITMAP_TYPE_PNG ) ) + { + return false; + } + + m_bitmap = wxBitmap(image); + + return m_bitmap.IsOk(); } void wxBitmapDataObject::DoConvertToPng() { - if (!m_bitmap.Ok()) return; - - wxImage image( m_bitmap ); - wxPNGHandler handler; - + if ( !m_bitmap.IsOk() ) + return; + + wxCHECK_RET( wxImage::FindHandler(wxBITMAP_TYPE_PNG) != NULL, + wxT("You must call wxImage::AddHandler(new wxPNGHandler); to be able to use clipboard with bitmaps!") ); + + wxImage image = m_bitmap.ConvertToImage(); + wxCountingOutputStream count; - handler.SaveFile( &image, count ); + image.SaveFile(count, wxBITMAP_TYPE_PNG); + m_pngSize = count.GetSize() + 100; // sometimes the size seems to vary ??? - m_pngData = (void*) new char[m_pngSize]; - - wxMemoryOutputStream mstream( (char*) m_pngData, m_pngSize ); - handler.SaveFile( &image, mstream ); + m_pngData = malloc(m_pngSize); + + wxMemoryOutputStream mstream((char*) m_pngData, m_pngSize); + image.SaveFile(mstream, wxBITMAP_TYPE_PNG); } // ---------------------------------------------------------------------------- -// wxPrivateDataObject +// wxURLDataObject // ---------------------------------------------------------------------------- -IMPLEMENT_CLASS( wxPrivateDataObject, wxDataObject ) - -void wxPrivateDataObject::Free() +class wxTextURIListDataObject : public wxDataObjectSimple { - if ( m_data ) - free(m_data); -} +public: + wxTextURIListDataObject(const wxString& url) + : wxDataObjectSimple(wxDataFormat(g_fileAtom)), + m_url(url) + { + } -wxPrivateDataObject::wxPrivateDataObject() -{ - wxString id = wxT("application/"); - id += wxTheApp->GetAppName(); + const wxString& GetURL() const { return m_url; } + void SetURL(const wxString& url) { m_url = url; } - m_format.SetId( id ); - m_size = 0; - m_data = (void *)NULL; -} + virtual size_t GetDataSize() const + { + // It is not totally clear whether we should include "\r\n" at the end + // of the string if there is only one URL or not, but not doing it + // doesn't seem to create any problems, so keep things simple. + return strlen(m_url.utf8_str()) + 1; + } -void wxPrivateDataObject::SetData( const void *data, size_t size ) -{ - Free(); + virtual bool GetDataHere(void *buf) const + { + char* const dst = static_cast(buf); - m_size = size; - m_data = malloc(size); + strcpy(dst, m_url.utf8_str()); - memcpy( m_data, data, size ); -} + return true; + } + + virtual bool SetData(size_t len, const void *buf) + { + const char* const src = static_cast(buf); + + // The string might be "\r\n"-terminated but this is not necessarily + // the case (e.g. when dragging an URL from Firefox, it isn't). + if ( len > 1 && src[len - 1] == '\n' ) + { + if ( len > 2 && src[len - 2] == '\r' ) + len--; + + len--; + } + + m_url = wxString::FromUTF8(src, len); -void wxPrivateDataObject::WriteData( void *dest ) const + return true; + } + + // Must provide overloads to avoid hiding them (and warnings about it) + virtual size_t GetDataSize(const wxDataFormat&) const + { + return GetDataSize(); + } + virtual bool GetDataHere(const wxDataFormat&, void *buf) const + { + return GetDataHere(buf); + } + virtual bool SetData(const wxDataFormat&, size_t len, const void *buf) + { + return SetData(len, buf); + } + +private: + wxString m_url; +}; + +wxURLDataObject::wxURLDataObject(const wxString& url) : + m_dobjURIList(new wxTextURIListDataObject(url)), + m_dobjText(new wxTextDataObject(url)) { - WriteData( m_data, dest ); + // Use both URL-specific format and a plain text one to ensure that URLs + // can be pasted into any application. + Add(m_dobjURIList, true /* preferred */); + Add(m_dobjText); } -size_t wxPrivateDataObject::GetSize() const +void wxURLDataObject::SetURL(const wxString& url) { - return m_size; + m_dobjURIList->SetURL(url); + m_dobjText->SetText(url); } -void wxPrivateDataObject::WriteData( const void *data, void *dest ) const +wxString wxURLDataObject::GetURL() const { - memcpy( dest, data, GetSize() ); + if ( GetReceivedFormat() == g_fileAtom ) + { + // If we received the URL as an URI, use it. + return m_dobjURIList->GetURL(); + } + else // Otherwise we either got it as text or didn't get anything yet. + { + // In either case using the text format should be fine. + return m_dobjText->GetText(); + } } +#endif // wxUSE_DATAOBJ