correct hhp2cached path

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

diff --git a/src/mac/carbon/dnd.cpp b/src/mac/carbon/dnd.cpp
index 22ced39ede9fbf79b9fbd1cc2cf95cd94d7cc111..961eb6da0f9617a531eccf2110cb00f74e78940e 100644 (file)
--- a/src/mac/carbon/dnd.cpp
+++ b/src/mac/carbon/dnd.cpp
@@ -1,6 +1,6 @@
 ///////////////////////////////////////////////////////////////////////////////
 ///////////////////////////////////////////////////////////////////////////////

-// Name:        dnd.cpp
-// Purpose:     wxDropTarget, wxDropSource, wxDataObject implementation
+// Name:        src/mac/carbon/dnd.cpp
+// Purpose:     wxDropTarget, wxDropSource implementations

 // Author:      Stefan Csomor
 // Modified by:
 // Created:     1998-01-01
 // Author:      Stefan Csomor
 // Modified by:
 // Created:     1998-01-01


@@ -9,35 +9,36 @@
 // Licence:     wxWindows licence
 ///////////////////////////////////////////////////////////////////////////////
 
 // Licence:     wxWindows licence
 ///////////////////////////////////////////////////////////////////////////////
 

-#ifdef __GNUG__
-#pragma implementation "dnd.h"
-#endif
-
-#include "wx/defs.h"
+#include "wx/wxprec.h"

 
 #if wxUSE_DRAG_AND_DROP
 
 #include "wx/dnd.h"
 
 #if wxUSE_DRAG_AND_DROP
 
 #include "wx/dnd.h"

-#include "wx/window.h"
-#include "wx/toplevel.h"
-#include "wx/app.h"
-#include "wx/gdicmn.h"
+
+#ifndef WX_PRECOMP
+    #include "wx/app.h"
+    #include "wx/toplevel.h"
+    #include "wx/gdicmn.h"
+#endif // WX_PRECOMP
+

 #include "wx/mac/private.h"
 
 // ----------------------------------------------------------------------------
 #include "wx/mac/private.h"
 
 // ----------------------------------------------------------------------------

-// global
+// globals

 // ----------------------------------------------------------------------------
 
 // ----------------------------------------------------------------------------
 

-void wxMacEnsureTrackingHandlersInstalled() ;
-
-typedef struct 
+typedef struct

 {
 {

-    wxWindow* m_currentTargetWindow ;
-    wxDropTarget* m_currentTarget ;
-    wxDropSource* m_currentSource ;
-} MacTrackingGlobals ;
+    wxWindow *m_currentTargetWindow;
+    wxDropTarget *m_currentTarget;
+    wxDropSource *m_currentSource;
+    wxDragResult m_result;
+    int m_flags;
+} MacTrackingGlobals;

 
 

-MacTrackingGlobals gTrackingGlobals ; 
+MacTrackingGlobals gTrackingGlobals;
+
+void wxMacEnsureTrackingHandlersInstalled();

 
 //----------------------------------------------------------------------------
 // wxDropTarget
 
 //----------------------------------------------------------------------------
 // wxDropTarget


@@ -46,29 +47,29 @@ MacTrackingGlobals gTrackingGlobals ;
 wxDropTarget::wxDropTarget( wxDataObject *data )
             : wxDropTargetBase( data )
 {
 wxDropTarget::wxDropTarget( wxDataObject *data )
             : wxDropTargetBase( data )
 {

-    wxMacEnsureTrackingHandlersInstalled() ;
+    wxMacEnsureTrackingHandlersInstalled();

 }
 
 }
 

-wxDragResult wxDropTarget::OnDragOver( wxCoord WXUNUSED(x),
-                                       wxCoord WXUNUSED(y),
-                                       wxDragResult def )
+wxDragResult wxDropTarget::OnDragOver(
+    wxCoord WXUNUSED(x), wxCoord WXUNUSED(y),
+    wxDragResult def )

 {
 {

-

     return CurrentDragHasSupportedFormat() ? def : wxDragNone;
 }
 
 bool wxDropTarget::OnDrop( wxCoord WXUNUSED(x), wxCoord WXUNUSED(y) )
 {
     return CurrentDragHasSupportedFormat() ? def : wxDragNone;
 }
 
 bool wxDropTarget::OnDrop( wxCoord WXUNUSED(x), wxCoord WXUNUSED(y) )
 {

-    if (!m_dataObject)
-        return FALSE;
+    if (m_dataObject == NULL)
+        return false;

 
 

-    return CurrentDragHasSupportedFormat() ;
+    return CurrentDragHasSupportedFormat();

 }
 
 }
 

-wxDragResult wxDropTarget::OnData( wxCoord WXUNUSED(x), wxCoord WXUNUSED(y),
-                                   wxDragResult def )
+wxDragResult wxDropTarget::OnData(
+    wxCoord WXUNUSED(x), wxCoord WXUNUSED(y),
+    wxDragResult def )

 {
 {

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

         return wxDragNone;
 
     if (!CurrentDragHasSupportedFormat())
         return wxDragNone;
 
     if (!CurrentDragHasSupportedFormat())


@@ -77,162 +78,103 @@ wxDragResult wxDropTarget::OnData( wxCoord WXUNUSED(x), wxCoord WXUNUSED(y),
     return GetData() ? def : wxDragNone;
 }
 
     return GetData() ? def : wxDragNone;
 }
 

-bool wxDropTarget::CurrentDragHasSupportedFormat() 
+bool wxDropTarget::CurrentDragHasSupportedFormat()

 {
 {

-    bool supported = false ;
+    bool supported = false;
+    if (m_dataObject == NULL)
+        return false;
+

     if ( gTrackingGlobals.m_currentSource != NULL )
     {
     if ( gTrackingGlobals.m_currentSource != NULL )
     {

-        wxDataObject* data = gTrackingGlobals.m_currentSource->GetDataObject() ;
-        
+        wxDataObject* data = gTrackingGlobals.m_currentSource->GetDataObject();
+

         if ( data )
         {
         if ( data )
         {

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

             data->GetAllFormats( array );
             data->GetAllFormats( array );

-            for (size_t i = 0; !supported && i < formatcount ; i++)
+            for (size_t i = 0; !supported && i < formatcount; i++)

             {
             {

-                wxDataFormat format = array[i] ;
-                if ( m_dataObject->IsSupported( format ) ) 
+                wxDataFormat format = array[i];
+                if ( m_dataObject->IsSupported( format ) )

                 {
                 {

-                    supported = true ;
-                    break ;
+                    supported = true;
+                    break;

                 }
             }
                 }
             }

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

         }
     }
         }
     }

+

     if ( !supported )
     {
     if ( !supported )
     {

-        UInt16 items ;
-        OSErr result;
-        CountDragItems((DragReference)m_currentDrag, &items);
-        for (UInt16 index = 1; index <= items && supported == false ; ++index) 
+        PasteboardRef   pasteboard;
+
+        if ( GetDragPasteboard( (DragReference)m_currentDrag, &pasteboard ) == noErr )

         {
         {

-            ItemReference theItem;
-            FlavorType theType ;
-            UInt16 flavors = 0 ;
-            GetDragItemReferenceNumber((DragReference)m_currentDrag, index, &theItem);
-            CountDragItemFlavors( (DragReference)m_currentDrag, theItem , &flavors ) ;
-            for ( UInt16 flavor = 1 ; flavor <= flavors ; ++flavor )
-            {
-                result = GetFlavorType((DragReference)m_currentDrag, theItem, flavor , &theType);
-                if ( m_dataObject->IsSupportedFormat( wxDataFormat( theType ) ) )
-                {
-                    supported = true ;
-                    break ;
-                }
-            }
+            supported = m_dataObject->HasDataInPasteboard( pasteboard );

         }
     }
         }
     }

-    return supported ;   
+
+    return supported;

 }
 
 bool wxDropTarget::GetData()
 {
 }
 
 bool wxDropTarget::GetData()
 {

-    if (!m_dataObject)
-        return FALSE;
-    
+    if (m_dataObject == NULL)
+        return false;
+

     if ( !CurrentDragHasSupportedFormat() )
     if ( !CurrentDragHasSupportedFormat() )

-        return FALSE ;
-    
-    bool transferred = false ;   
+        return false;
+
+    bool transferred = false;

     if ( gTrackingGlobals.m_currentSource != NULL )
     {
     if ( gTrackingGlobals.m_currentSource != NULL )
     {

-        wxDataObject* data = gTrackingGlobals.m_currentSource->GetDataObject() ;
-        
-        if ( data )
+        wxDataObject* data = gTrackingGlobals.m_currentSource->GetDataObject();
+
+        if (data != NULL)

         {
         {

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

             data->GetAllFormats( array );
             data->GetAllFormats( array );

-            for (size_t i = 0; !transferred && i < formatcount ; i++)
+            for (size_t i = 0; !transferred && i < formatcount; i++)

             {
             {

-                wxDataFormat format = array[i] ;
-                if ( m_dataObject->IsSupported( format ) ) 
+                wxDataFormat format = array[i];
+                if ( m_dataObject->IsSupported( format ) )

                 {
                     int size = data->GetDataSize( format );
                 {
                     int size = data->GetDataSize( format );

-                    transferred = true ;
-                    
-                    if (size == 0) 
+                    transferred = true;
+
+                    if (size == 0)

                     {
                     {

-                        m_dataObject->SetData(format , 0 , 0 ) ;
+                        m_dataObject->SetData( format, 0, 0 );

                     }
                     else
                     {
                         char *d = new char[size];
                     }
                     else
                     {
                         char *d = new char[size];

-                        data->GetDataHere( format , (void*) d );
-                        m_dataObject->SetData( format , size , d ) ;
-                        delete[] d ;
+                        data->GetDataHere( format, (void*)d );
+                        m_dataObject->SetData( format, size, d );
+                        delete [] d;

                     }
                 }
             }
                     }
                 }
             }

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

         }
     }
         }
     }

+

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

-        UInt16 items ;
-        OSErr result;
-        bool firstFileAdded = false ;
-        CountDragItems((DragReference)m_currentDrag, &items);
-        for (UInt16 index = 1; index <= items; ++index) 
+        PasteboardRef   pasteboard;
+
+        if ( GetDragPasteboard(  (DragReference)m_currentDrag, &pasteboard ) == noErr )

         {
         {

-            ItemReference theItem;
-            FlavorType theType ;
-            UInt16 flavors = 0 ;
-            GetDragItemReferenceNumber((DragReference)m_currentDrag, index, &theItem);
-            CountDragItemFlavors( (DragReference)m_currentDrag, theItem , &flavors ) ;
-            for ( UInt16 flavor = 1 ; flavor <= flavors ; ++flavor )
-            {
-                result = GetFlavorType((DragReference)m_currentDrag, theItem, flavor , &theType);
-                wxDataFormat format(theType) ;
-                if ( m_dataObject->IsSupportedFormat( format ) )
-                {
-                    FlavorFlags theFlags;
-                    result = GetFlavorFlags((DragReference)m_currentDrag, theItem, theType, &theFlags);
-                    if (result == noErr) 
-                    {
-                        Size dataSize ;
-                        Ptr theData ;
-                        GetFlavorDataSize((DragReference)m_currentDrag, theItem, theType, &dataSize);
-                        if ( theType == 'TEXT' )
-                        {
-                            // this increment is only valid for allocating, on the next GetFlavorData
-                            // call it is reset again to the original value
-                            dataSize++ ;
-                        }
-                        theData = new char[dataSize];
-                        GetFlavorData((DragReference)m_currentDrag, theItem, theType, (void*) theData, &dataSize, 0L); 
-                        if( theType == 'TEXT' )
-                        {
-                            theData[dataSize]=0 ; 
-                            wxString convert( theData , wxConvLocal ) ;    
-                            m_dataObject->SetData( format, convert.Length() * sizeof(wxChar), (const wxChar*) convert );
-                        }
-                        else if ( theType == kDragFlavorTypeHFS )
-                        {
-                            HFSFlavor* theFile = (HFSFlavor*) theData ;
-                            wxString name = wxMacFSSpec2MacFilename( &theFile->fileSpec ) ;
-                            if (  firstFileAdded )
-                                ((wxFileDataObject*)m_dataObject)->AddFile( name ) ;
-                            else
-                            {
-                                ((wxFileDataObject*)m_dataObject)->SetData( 0 , name.c_str() ) ;
-                                firstFileAdded = true ;    
-                            }
-                        }
-                        else
-                        {
-                            m_dataObject->SetData( format, dataSize, theData );
-                        }
-                        delete[] theData;
-                    }
-                    break ;
-                }
-            }
+            transferred = m_dataObject->GetFromPasteboard( pasteboard );

         }
     }
         }
     }

-    return TRUE ;   
+
+    return transferred;

 }
 
 //-------------------------------------------------------------------------
 }
 
 //-------------------------------------------------------------------------


@@ -248,7 +190,8 @@ wxDropSource::wxDropSource(wxWindow *win,
                            const wxCursor &cursorStop)
             : wxDropSourceBase(cursorCopy, cursorMove, cursorStop)
 {
                            const wxCursor &cursorStop)
             : wxDropSourceBase(cursorCopy, cursorMove, cursorStop)
 {

-    wxMacEnsureTrackingHandlersInstalled() ;
+    wxMacEnsureTrackingHandlersInstalled();
+

     m_window = win;
 }
 
     m_window = win;
 }
 


@@ -259,7 +202,8 @@ wxDropSource::wxDropSource(wxDataObject& data,
                            const wxCursor &cursorStop)
             : wxDropSourceBase(cursorCopy, cursorMove, cursorStop)
 {
                            const wxCursor &cursorStop)
             : wxDropSourceBase(cursorCopy, cursorMove, cursorStop)
 {

-    wxMacEnsureTrackingHandlersInstalled() ;
+    wxMacEnsureTrackingHandlersInstalled();
+

     SetData( data );
     m_window = win;
 }
     SetData( data );
     m_window = win;
 }


@@ -268,208 +212,197 @@ wxDropSource::~wxDropSource()
 {
 }
 
 {
 }
 

+OSStatus wxMacPromiseKeeper(PasteboardRef WXUNUSED(inPasteboard),
+                            PasteboardItemID WXUNUSED(inItem),
+                            CFStringRef WXUNUSED(inFlavorType),
+                            void * WXUNUSED(inContext))
+{
+    OSStatus  err = noErr;
+
+    // we might add promises here later, inContext is the wxDropSource*

 
 

-wxDragResult wxDropSource::DoDragDrop(int WXUNUSED(flags))
+    return err;
+}
+
+wxDragResult wxDropSource::DoDragDrop(int flags)

 {
     wxASSERT_MSG( m_data, wxT("Drop source: no data") );
 {
     wxASSERT_MSG( m_data, wxT("Drop source: no data") );

-    
-    if (!m_data)
-        return (wxDragResult) wxDragNone;
-    
-    if (m_data->GetFormatCount() == 0)
-        return (wxDragResult) wxDragNone;
-    
-    OSErr result;
+
+    if ((m_data == NULL) || (m_data->GetFormatCount() == 0))
+        return (wxDragResult)wxDragNone;
+

     DragReference theDrag;
     RgnHandle dragRegion;
     DragReference theDrag;
     RgnHandle dragRegion;

-    if ((result = NewDrag(&theDrag)))
+    OSStatus err = noErr;
+    PasteboardRef   pasteboard;
+
+    // add data to drag
+
+    err = PasteboardCreate( kPasteboardUniqueName, &pasteboard );
+    if ( err != noErr )
+        return wxDragNone;
+
+    // we add a dummy promise keeper because of strange messages when linking against carbon debug
+    err = PasteboardSetPromiseKeeper( pasteboard, wxMacPromiseKeeper, this );
+    if ( err != noErr )

     {
     {

-        return wxDragNone ;
+        CFRelease( pasteboard );
+        return wxDragNone;

     }
     }

-    // add data to drag
-    size_t formatCount = m_data->GetFormatCount() ;
-    wxDataFormat *formats = new wxDataFormat[formatCount] ;
-    m_data->GetAllFormats( formats ) ;
-    ItemReference theItem = 1 ;
-    for ( size_t i = 0 ; i < formatCount ; ++i )
+
+    err = PasteboardClear( pasteboard );
+    if ( err != noErr )

     {
     {

-        size_t dataSize = m_data->GetDataSize( formats[i] ) ;
-        Ptr dataPtr = new char[dataSize] ;
-        m_data->GetDataHere( formats[i] , dataPtr ) ;
-        OSType type = formats[i].GetFormatId() ;
-        if ( type == 'TEXT' )
-        {
-            dataSize-- ;
-            dataPtr[ dataSize ] = 0 ;
-            wxString st( (wxChar*) dataPtr ) ;
-            wxCharBuffer buf = st.mb_str( wxConvLocal) ;
-            AddDragItemFlavor(theDrag, theItem, type , buf.data(), strlen(buf), 0);
-        }
-        else if (type == kDragFlavorTypeHFS )
-        {
-            HFSFlavor  theFlavor ;
-            OSErr err = noErr;
-            CInfoPBRec cat;
-            
-            wxMacFilename2FSSpec( dataPtr , &theFlavor.fileSpec ) ;
-            
-            cat.hFileInfo.ioNamePtr = theFlavor.fileSpec.name;
-            cat.hFileInfo.ioVRefNum = theFlavor.fileSpec.vRefNum;
-            cat.hFileInfo.ioDirID = theFlavor.fileSpec.parID;
-            cat.hFileInfo.ioFDirIndex = 0;
-            err = PBGetCatInfoSync(&cat);
-            if (err == noErr )
-            {
-                theFlavor.fdFlags = cat.hFileInfo.ioFlFndrInfo.fdFlags;
-                if (theFlavor.fileSpec.parID == fsRtParID) {
-                    theFlavor.fileCreator = 'MACS';
-                    theFlavor.fileType = 'disk';
-                } else if ((cat.hFileInfo.ioFlAttrib & ioDirMask) != 0) {
-                    theFlavor.fileCreator = 'MACS';
-                    theFlavor.fileType = 'fold';
-                } else {
-                    theFlavor.fileCreator = cat.hFileInfo.ioFlFndrInfo.fdCreator;
-                    theFlavor.fileType = cat.hFileInfo.ioFlFndrInfo.fdType;
-                }
-                AddDragItemFlavor(theDrag, theItem, type , &theFlavor, sizeof(theFlavor), 0);  
-            }    
-        }
-        else
-        {
-            AddDragItemFlavor(theDrag, theItem, type , dataPtr, dataSize, 0);      
-        }
-        delete[] dataPtr ;
+        CFRelease( pasteboard );
+        return wxDragNone;

     }
     }

-    delete[] formats ;
-    
+    PasteboardSynchronize( pasteboard );
+
+    m_data->AddToPasteboard( pasteboard, 1 );
+
+    if (NewDragWithPasteboard( pasteboard , &theDrag) != noErr)
+    {
+        CFRelease( pasteboard );
+        return wxDragNone;
+    }
+

     dragRegion = NewRgn();
     dragRegion = NewRgn();

-    RgnHandle tempRgn = NewRgn() ;
-    
-    EventRecord* ev = NULL ;
-#if !TARGET_CARBON // TODO
-    ev = (EventRecord*) wxTheApp->MacGetCurrentEvent() ;
-#else
-    EventRecord rec ;
-    ev = &rec ;
-    wxMacConvertEventToRecord( (EventRef) wxTheApp->MacGetCurrentEvent() , &rec ) ;
-#endif
-    const short dragRegionOuterBoundary = 10 ;
-    const short dragRegionInnerBoundary = 9 ;
-    
-    SetRectRgn( dragRegion , ev->where.h - dragRegionOuterBoundary , 
-        ev->where.v  - dragRegionOuterBoundary ,
-        ev->where.h + dragRegionOuterBoundary , 
-        ev->where.v + dragRegionOuterBoundary ) ;
-    
-    SetRectRgn( tempRgn , ev->where.h - dragRegionInnerBoundary , 
-        ev->where.v  - dragRegionInnerBoundary ,
-        ev->where.h + dragRegionInnerBoundary , 
-        ev->where.v + dragRegionInnerBoundary ) ;
-    
-    DiffRgn( dragRegion , tempRgn , dragRegion ) ;
-    DisposeRgn( tempRgn ) ;    
-    
-    // TODO:work with promises in order to return data only when drag
-    // was successfully completed
-    
-    gTrackingGlobals.m_currentSource = this ;
-    result = TrackDrag(theDrag, ev , dragRegion);
-    DisposeRgn(dragRegion);
-    DisposeDrag(theDrag);
-    gTrackingGlobals.m_currentSource = NULL ;
-    
-    KeyMap keymap;
-    GetKeys(keymap);
-    bool optionDown = keymap[1] & 4;
-    wxDragResult dndresult = optionDown ? wxDragCopy : wxDragMove;
-    return dndresult;
+    RgnHandle tempRgn = NewRgn();
+
+    EventRecord rec;
+    ConvertEventRefToEventRecord(  (EventRef) wxTheApp->MacGetCurrentEvent(), &rec );
+
+    const short dragRegionOuterBoundary = 10;
+    const short dragRegionInnerBoundary = 9;
+
+    SetRectRgn(
+        dragRegion,
+        rec.where.h - dragRegionOuterBoundary,
+        rec.where.v  - dragRegionOuterBoundary,
+        rec.where.h + dragRegionOuterBoundary,
+        rec.where.v + dragRegionOuterBoundary );
+
+    SetRectRgn(
+        tempRgn,
+        rec.where.h - dragRegionInnerBoundary,
+        rec.where.v - dragRegionInnerBoundary,
+        rec.where.h + dragRegionInnerBoundary,
+        rec.where.v + dragRegionInnerBoundary );
+
+    DiffRgn( dragRegion, tempRgn, dragRegion );
+    DisposeRgn( tempRgn );
+
+    // TODO: work with promises in order to return data
+    // only when drag was successfully completed
+
+    gTrackingGlobals.m_currentSource = this;
+    gTrackingGlobals.m_result = wxDragNone;
+    gTrackingGlobals.m_flags = flags;
+
+    err = TrackDrag( theDrag, &rec, dragRegion );
+
+    DisposeRgn( dragRegion );
+    DisposeDrag( theDrag );
+    CFRelease( pasteboard );
+    gTrackingGlobals.m_currentSource = NULL;
+
+    return gTrackingGlobals.m_result;

 }
 
 bool wxDropSource::MacInstallDefaultCursor(wxDragResult effect)
 {
     const wxCursor& cursor = GetCursor(effect);
 }
 
 bool wxDropSource::MacInstallDefaultCursor(wxDragResult effect)
 {
     const wxCursor& cursor = GetCursor(effect);

-    if ( cursor.Ok() )
-    {
-        cursor.MacInstall() ;
+    bool result = cursor.Ok();

 
 

-        return TRUE;
-    }
-    else
-    {
-        return FALSE;
-    }
+    if ( result )
+        cursor.MacInstall();
+
+    return result;

 }
 
 }
 

-bool gTrackingGlobalsInstalled = false ;
+bool gTrackingGlobalsInstalled = false;

 
 // passing the globals via refcon is not needed by the CFM and later architectures anymore
 // but I'll leave it in there, just in case...
 
 
 // passing the globals via refcon is not needed by the CFM and later architectures anymore
 // but I'll leave it in there, just in case...
 

-pascal OSErr wxMacWindowDragTrackingHandler(DragTrackingMessage theMessage, WindowPtr theWindow,
-  void *handlerRefCon, DragReference theDrag) ;
-pascal OSErr wxMacWindowDragReceiveHandler(WindowPtr theWindow, void *handlerRefCon,
-DragReference theDrag) ;
+pascal OSErr wxMacWindowDragTrackingHandler(
+    DragTrackingMessage theMessage, WindowPtr theWindow,
+    void *handlerRefCon, DragReference theDrag );
+pascal OSErr wxMacWindowDragReceiveHandler(
+    WindowPtr theWindow, void *handlerRefCon,
+    DragReference theDrag );

 
 void wxMacEnsureTrackingHandlersInstalled()
 {
 
 void wxMacEnsureTrackingHandlersInstalled()
 {

-    if( !gTrackingGlobalsInstalled )
+    if ( !gTrackingGlobalsInstalled )

     {
     {

-        OSErr result;
+        OSStatus err;

 
 

-        result = InstallTrackingHandler(NewDragTrackingHandlerUPP(wxMacWindowDragTrackingHandler), 0L,&gTrackingGlobals);
-        wxASSERT( result == noErr ) ;
-        result = InstallReceiveHandler(NewDragReceiveHandlerUPP(wxMacWindowDragReceiveHandler), 0L, &gTrackingGlobals);
-        wxASSERT( result == noErr ) ;
+        err = InstallTrackingHandler( NewDragTrackingHandlerUPP(wxMacWindowDragTrackingHandler), 0L, &gTrackingGlobals );
+        verify_noerr( err );

 
 

-        gTrackingGlobalsInstalled = true ;
+        err = InstallReceiveHandler( NewDragReceiveHandlerUPP(wxMacWindowDragReceiveHandler), 0L, &gTrackingGlobals );
+        verify_noerr( err );
+
+        gTrackingGlobalsInstalled = true;

     }
 }
 
     }
 }
 

-pascal OSErr wxMacWindowDragTrackingHandler(DragTrackingMessage theMessage, WindowPtr theWindow,
-  void *handlerRefCon, DragReference theDrag)
-{ 
+pascal OSErr wxMacWindowDragTrackingHandler(
+    DragTrackingMessage theMessage, WindowPtr theWindow,
+    void *handlerRefCon, DragReference theDrag )
+{

     MacTrackingGlobals* trackingGlobals = (MacTrackingGlobals*) handlerRefCon;
     MacTrackingGlobals* trackingGlobals = (MacTrackingGlobals*) handlerRefCon;

+

     Point mouse, localMouse;
     DragAttributes attributes;
     Point mouse, localMouse;
     DragAttributes attributes;

-    GetDragAttributes(theDrag, &attributes);
-    wxTopLevelWindowMac* toplevel = wxFindWinFromMacWindow( theWindow ) ; 

 
 

-    KeyMap keymap;
-    GetKeys(keymap);
-    bool optionDown = keymap[1] & 4;
+    GetDragAttributes( theDrag, &attributes );
+
+    wxTopLevelWindowMac* toplevel = wxFindWinFromMacWindow( theWindow );
+
+    bool optionDown = GetCurrentKeyModifiers() & optionKey;

     wxDragResult result = optionDown ? wxDragCopy : wxDragMove;
 
     wxDragResult result = optionDown ? wxDragCopy : wxDragMove;
 

-    switch(theMessage) 
+    switch (theMessage)

     {
         case kDragTrackingEnterHandler:
     {
         case kDragTrackingEnterHandler:

-            break;

         case kDragTrackingLeaveHandler:
             break;
         case kDragTrackingLeaveHandler:
             break;

+

         case kDragTrackingEnterWindow:
         case kDragTrackingEnterWindow:

-            trackingGlobals->m_currentTargetWindow = NULL ;
-            trackingGlobals->m_currentTarget = NULL ;
+            if (trackingGlobals != NULL)
+            {
+                trackingGlobals->m_currentTargetWindow = NULL;
+                trackingGlobals->m_currentTarget = NULL;
+            }

             break;
             break;

+

         case kDragTrackingInWindow:
         case kDragTrackingInWindow:

+            if (trackingGlobals == NULL)
+                break;

             if (toplevel == NULL)
                 break;
 
             if (toplevel == NULL)
                 break;
 

-            GetDragMouse(theDrag, &mouse, 0L);
+            GetDragMouse( theDrag, &mouse, 0L );

             localMouse = mouse;
             localMouse = mouse;

-            GlobalToLocal(&localMouse);
-
+            wxMacGlobalToLocal( theWindow, &localMouse );

 
 

-            
-//            if (attributes & kDragHasLeftSenderWindow) 

             {
             {

-                wxPoint point(localMouse.h , localMouse.v) ;
-                wxWindow *win = NULL ;
-                toplevel->MacGetWindowFromPointSub( point , &win ) ;
-                int localx , localy ;
-                localx = localMouse.h ;
-                localy = localMouse.v ;
-                //TODO : should we use client coordinates
+                wxWindow *win = NULL;
+                ControlPartCode controlPart;
+                ControlRef control = FindControlUnderMouse( localMouse, theWindow, &controlPart );
+                if ( control )
+                    win = wxFindControlFromMacControl( control );
+                else
+                    win = toplevel;
+
+                int localx, localy;
+                localx = localMouse.h;
+                localy = localMouse.v;
+

                 if ( win )
                 if ( win )

-                    win->MacRootWindowToWindow( &localx , &localy ) ;
+                    win->MacRootWindowToWindow( &localx, &localy );

                 if ( win != trackingGlobals->m_currentTargetWindow )
                 {
                     if ( trackingGlobals->m_currentTargetWindow )
                 if ( win != trackingGlobals->m_currentTargetWindow )
                 {
                     if ( trackingGlobals->m_currentTargetWindow )


@@ -477,131 +410,153 @@ pascal OSErr wxMacWindowDragTrackingHandler(DragTrackingMessage theMessage, Wind
                         // this window is left
                         if ( trackingGlobals->m_currentTarget )
                         {
                         // this window is left
                         if ( trackingGlobals->m_currentTarget )
                         {

-                            HideDragHilite(theDrag);
-                            trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag ) ;
-                            trackingGlobals->m_currentTarget->OnLeave() ;
+#ifndef __LP64__
+                            HideDragHilite( theDrag );
+#endif
+                            trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag );
+                            trackingGlobals->m_currentTarget->OnLeave();

                             trackingGlobals->m_currentTarget = NULL;
                             trackingGlobals->m_currentTarget = NULL;

-                            trackingGlobals->m_currentTargetWindow = NULL ;
+                            trackingGlobals->m_currentTargetWindow = NULL;

                         }
                     }
                         }
                     }

+

                     if ( win )
                     {
                         // this window is entered
                     if ( win )
                     {
                         // this window is entered

-                        trackingGlobals->m_currentTargetWindow = win ;
-                        trackingGlobals->m_currentTarget = win->GetDropTarget() ;
+                        trackingGlobals->m_currentTargetWindow = win;
+                        trackingGlobals->m_currentTarget = win->GetDropTarget();

                         {
                         {

-
-                               if ( trackingGlobals->m_currentTarget )
-                               {
-                               trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag ) ;
-                               result = trackingGlobals->m_currentTarget->OnEnter(
-                                       localx , localy , result ) ;
+                            if ( trackingGlobals->m_currentTarget )
+                            {
+                                trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag );
+                                result = trackingGlobals->m_currentTarget->OnEnter( localx, localy, result );

                             }
                             }

-                                
-                           
+

                             if ( result != wxDragNone )
                             {
                             if ( result != wxDragNone )
                             {

-                                int x , y ;
-                                x = y = 0 ;
-                                win->MacWindowToRootWindow( &x , &y ) ;
-                                RgnHandle hiliteRgn = NewRgn() ;
-                                SetRectRgn( hiliteRgn , x , y , x+win->GetSize().x ,y+win->GetSize().y) ;
-                                ShowDragHilite(theDrag, hiliteRgn, true);
-                                DisposeRgn( hiliteRgn ) ;
+                                int x, y;
+
+                                x = y = 0;
+                                win->MacWindowToRootWindow( &x, &y );
+                                RgnHandle hiliteRgn = NewRgn();
+                                Rect r = { y, x, y + win->GetSize().y, x + win->GetSize().x };
+                                RectRgn( hiliteRgn, &r );
+#ifndef __LP64__
+                                ShowDragHilite( theDrag, hiliteRgn, true );
+#endif
+                                DisposeRgn( hiliteRgn );

                             }
                         }
                     }
                 }
                 else
                 {
                             }
                         }
                     }
                 }
                 else
                 {

-                    if( trackingGlobals->m_currentTarget )
+                    if ( trackingGlobals->m_currentTarget )

                     {
                     {

-                        trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag ) ;
-                        trackingGlobals->m_currentTarget->OnDragOver(
-                            localx , localy , result ) ;
+                        trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag );
+                        result = trackingGlobals->m_currentTarget->OnDragOver( localx, localy, result );

                     }
                 }
 
                 // set cursor for OnEnter and OnDragOver
                     }
                 }
 
                 // set cursor for OnEnter and OnDragOver

-                if ( trackingGlobals->m_currentSource && trackingGlobals->m_currentSource->GiveFeedback( result ) == FALSE )
+                if ( trackingGlobals->m_currentSource && !trackingGlobals->m_currentSource->GiveFeedback( result ) )

                 {
                 {

-                  if ( trackingGlobals->m_currentSource->MacInstallDefaultCursor( result ) == FALSE )
+                  if ( !trackingGlobals->m_currentSource->MacInstallDefaultCursor( result ) )

                   {
                   {

-                      switch( result )
+                      int cursorID = wxCURSOR_NONE;
+
+                      switch (result)

                       {
                       {

-                          case wxDragCopy :
-                              {
-                                  wxCursor cursor(wxCURSOR_COPY_ARROW) ;
-                                  cursor.MacInstall() ;
-                              }
-                              break ;
-                          case wxDragMove :
-                              {
-                                  wxCursor cursor(wxCURSOR_ARROW) ;
-                                  cursor.MacInstall() ;
-                              }
-                              break ;
-                          case wxDragNone :
-                              {
-                                  wxCursor cursor(wxCURSOR_NO_ENTRY) ;
-                                  cursor.MacInstall() ;
-                              }
-                              break ;
+                          case wxDragCopy:
+                              cursorID = wxCURSOR_COPY_ARROW;
+                              break;
+
+                          case wxDragMove:
+                              cursorID = wxCURSOR_ARROW;
+                              break;
+
+                          case wxDragNone:
+                              cursorID = wxCURSOR_NO_ENTRY;
+                              break;

 
                           case wxDragError:
                           case wxDragLink:
                           case wxDragCancel:
 
                           case wxDragError:
                           case wxDragLink:
                           case wxDragCancel:

+                          default:

                               // put these here to make gcc happy
                               ;
                       }
                               // put these here to make gcc happy
                               ;
                       }

-                  }
+
+                      if (cursorID != wxCURSOR_NONE)
+                      {
+                          wxCursor cursor( cursorID );
+                          cursor.MacInstall();
+                      }
+                   }

                 }
                 }

-                
-          }
-            // MyTrackItemUnderMouse(localMouse, theWindow);
+            }

             break;
             break;

+

         case kDragTrackingLeaveWindow:
         case kDragTrackingLeaveWindow:

-            if (trackingGlobals->m_currentTarget) 
+            if (trackingGlobals == NULL)
+                break;
+
+            if (trackingGlobals->m_currentTarget)

             {
             {

-                trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag ) ;
-                trackingGlobals->m_currentTarget->OnLeave() ;
-                HideDragHilite(theDrag);
-                trackingGlobals->m_currentTarget = NULL ;
+                trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag );
+                trackingGlobals->m_currentTarget->OnLeave();
+#ifndef __LP64__
+                HideDragHilite( theDrag );
+#endif
+                trackingGlobals->m_currentTarget = NULL;

             }
             }

-            trackingGlobals->m_currentTargetWindow = NULL ;
+            trackingGlobals->m_currentTargetWindow = NULL;
+            break;
+
+        default:

             break;
     }
             break;
     }

-    return(noErr);
+
+    return noErr;

 }
 
 }
 

-pascal OSErr wxMacWindowDragReceiveHandler(WindowPtr theWindow,
-                                           void *handlerRefCon,
-                                           DragReference theDrag)
-{ 
-    MacTrackingGlobals* trackingGlobals = (MacTrackingGlobals*) handlerRefCon;
+pascal OSErr wxMacWindowDragReceiveHandler(
+    WindowPtr theWindow,
+    void *handlerRefCon,
+    DragReference theDrag)
+{
+    MacTrackingGlobals* trackingGlobals = (MacTrackingGlobals*)handlerRefCon;

     if ( trackingGlobals->m_currentTarget )
     {
     if ( trackingGlobals->m_currentTarget )
     {

-        Point mouse,localMouse ;
-        int localx,localy ;
-        
-        trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag ) ;
-        GetDragMouse(theDrag, &mouse, 0L);
+        Point mouse, localMouse;
+        int localx, localy;
+
+        trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag );
+        GetDragMouse( theDrag, &mouse, 0L );

         localMouse = mouse;
         localMouse = mouse;

-        GlobalToLocal(&localMouse);
-        localx = localMouse.h ;
-        localy = localMouse.v ;
-        //TODO : should we use client coordinates
+        wxMacGlobalToLocal( theWindow, &localMouse );
+        localx = localMouse.h;
+        localy = localMouse.v;
+
+        // TODO : should we use client coordinates?

         if ( trackingGlobals->m_currentTargetWindow )
         if ( trackingGlobals->m_currentTargetWindow )

-            trackingGlobals->m_currentTargetWindow->MacRootWindowToWindow( &localx , &localy ) ;
-        if ( trackingGlobals->m_currentTarget->OnDrop( localx , localy ) )
+            trackingGlobals->m_currentTargetWindow->MacRootWindowToWindow( &localx, &localy );
+        if ( trackingGlobals->m_currentTarget->OnDrop( localx, localy ) )

         {
         {

-            KeyMap keymap;
-            GetKeys(keymap);
-            bool optionDown = keymap[1] & 4;
-            wxDragResult result = optionDown ? wxDragCopy : wxDragMove;
-            trackingGlobals->m_currentTarget->OnData( localx , localy , result ) ;
+            // the option key indicates copy in Mac UI, if it's not pressed do
+            // move by default if it's allowed at all
+            wxDragResult
+                result = !(trackingGlobals->m_flags & wxDrag_AllowMove) ||
+                            (GetCurrentKeyModifiers() & optionKey)
+                            ? wxDragCopy
+                            : wxDragMove;
+            trackingGlobals->m_result =
+                trackingGlobals->m_currentTarget->OnData( localx, localy, result );

         }
     }
         }
     }

-    return(noErr);
+
+    return noErr;

 }
 }

-#endif
+
+#endif // wxUSE_DRAG_AND_DROP
+