]> git.saurik.com Git - wxWidgets.git/blobdiff - src/mac/carbon/dnd.cpp
projects / wxWidgets.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
fixing a scope error introduced in 1.41
[wxWidgets.git] / src / mac / carbon / dnd.cpp
diff --git a/src/mac/carbon/dnd.cpp b/src/mac/carbon/dnd.cpp
index 7cd8ecc62cbb46ecbd445ef77d9ec225b530c0fe..ba746ab3e628fc7ce8e4a876b19e58d13684c922 100644 (file)
--- a/src/mac/carbon/dnd.cpp
+++ b/src/mac/carbon/dnd.cpp
@@ -24,21 +24,22 @@
     #include <Scrap.h>
 #endif
 
     #include <Scrap.h>
 #endif
 
+
 // ----------------------------------------------------------------------------
 // globals
 // ----------------------------------------------------------------------------
 
 typedef struct
 {
 // ----------------------------------------------------------------------------
 // globals
 // ----------------------------------------------------------------------------
 
 typedef struct
 {
-    wxWindow* m_currentTargetWindow ;
-    wxDropTarget* m_currentTarget ;
-    wxDropSource* m_currentSource ;
+    wxWindow *m_currentTargetWindow;
+    wxDropTarget *m_currentTarget;
+    wxDropSource *m_currentSource;
 }
 }
-MacTrackingGlobals ;
+MacTrackingGlobals;
 
 
-MacTrackingGlobals gTrackingGlobals ;
+MacTrackingGlobals gTrackingGlobals;
 
 
-void wxMacEnsureTrackingHandlersInstalled() ;
+void wxMacEnsureTrackingHandlersInstalled();
 
 //----------------------------------------------------------------------------
 // wxDropTarget
 
 //----------------------------------------------------------------------------
 // wxDropTarget
@@ -47,12 +48,12 @@ void wxMacEnsureTrackingHandlersInstalled() ;
 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;
 }
 {
     return CurrentDragHasSupportedFormat() ? def : wxDragNone;
 }
@@ -62,11 +63,12 @@ bool wxDropTarget::OnDrop( wxCoord WXUNUSED(x), wxCoord WXUNUSED(y) )
     if (m_dataObject == NULL)
         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 == NULL)
         return wxDragNone;
 {
     if (m_dataObject == NULL)
         return wxDragNone;
@@ -79,206 +81,221 @@ wxDragResult wxDropTarget::OnData( wxCoord WXUNUSED(x), wxCoord WXUNUSED(y),
 
 bool wxDropTarget::CurrentDragHasSupportedFormat()
 {
 
 bool wxDropTarget::CurrentDragHasSupportedFormat()
 {
-    bool supported = false ;
+    bool supported = 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() ;
+            size_t formatcount = data->GetFormatCount();
             wxDataFormat *array = new wxDataFormat[formatcount];
             data->GetAllFormats( array );
             wxDataFormat *array = new wxDataFormat[formatcount];
             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] ;
+                wxDataFormat format = array[i];
                 if ( m_dataObject->IsSupported( format ) )
                 {
                 if ( m_dataObject->IsSupported( format ) )
                 {
-                    supported = true ;
-                    break ;
+                    supported = true;
+                    break;
                 }
             }
 
                 }
             }
 
-            delete [] array ;
+            delete [] array;
         }
     }
 
     if ( !supported )
     {
         }
     }
 
     if ( !supported )
     {
-        UInt16 items ;
+        UInt16 items;
         OSErr result;
         ItemReference theItem;
         OSErr result;
         ItemReference theItem;
-        FlavorType theType ;
-        UInt16 flavors = 0 ;
+        FlavorType theType;
+        UInt16 flavors = 0;
 
 
-        CountDragItems((DragReference)m_currentDrag, &items);
+        CountDragItems( (DragReference)m_currentDrag, &items );
         for (UInt16 index = 1; index <= items && !supported; ++index)
         {
         for (UInt16 index = 1; index <= items && !supported; ++index)
         {
-            flavors = 0 ;
+            flavors = 0;
             GetDragItemReferenceNumber( (DragReference)m_currentDrag, index, &theItem );
             CountDragItemFlavors( (DragReference)m_currentDrag, theItem, &flavors );
 
             GetDragItemReferenceNumber( (DragReference)m_currentDrag, index, &theItem );
             CountDragItemFlavors( (DragReference)m_currentDrag, theItem, &flavors );
 
-            for ( UInt16 flavor = 1 ; flavor <= flavors ; ++flavor )
+            for ( UInt16 flavor = 1; flavor <= flavors; ++flavor )
             {
                 result = GetFlavorType( (DragReference)m_currentDrag, theItem, flavor, &theType );
                 if ( m_dataObject->IsSupportedFormat( wxDataFormat( theType ) ) )
                 {
             {
                 result = GetFlavorType( (DragReference)m_currentDrag, theItem, flavor, &theType );
                 if ( m_dataObject->IsSupportedFormat( wxDataFormat( theType ) ) )
                 {
-                    supported = true ;
-                    break ;
+                    supported = true;
+                    break;
                 }
             }
         }
     }
 
                 }
             }
         }
     }
 
-    return supported ;
+    return supported;
 }
 
 bool wxDropTarget::GetData()
 {
 }
 
 bool wxDropTarget::GetData()
 {
-    if (!m_dataObject)
+    if (m_dataObject == NULL)
         return false;
 
     if ( !CurrentDragHasSupportedFormat() )
         return false;
 
     if ( !CurrentDragHasSupportedFormat() )
-        return false ;
+        return false;
 
 
-    bool transferred = false ;
+    bool transferred = 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 != NULL)
         {
         {
-            size_t formatcount = data->GetFormatCount() ;
+            size_t formatcount = data->GetFormatCount();
             wxDataFormat *array = new wxDataFormat[formatcount];
             data->GetAllFormats( array );
             wxDataFormat *array = new wxDataFormat[formatcount];
             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] ;
+                wxDataFormat format = array[i];
                 if ( m_dataObject->IsSupported( format ) )
                 {
                     int size = data->GetDataSize( format );
                 if ( m_dataObject->IsSupported( format ) )
                 {
                     int size = data->GetDataSize( format );
-                    transferred = true ;
+                    transferred = true;
 
                     if (size == 0)
                     {
 
                     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 ;
+        UInt16 items;
         OSErr result;
         ItemReference theItem;
         OSErr result;
         ItemReference theItem;
-        FlavorType theType ;
+        FlavorType theType;
         FlavorFlags theFlags;
         FlavorFlags theFlags;
-        UInt16 flavors ;
-        bool firstFileAdded = false ;
+        UInt16 flavors;
+        bool firstFileAdded = false;
 
         CountDragItems( (DragReference)m_currentDrag, &items );
         for (UInt16 index = 1; index <= items; ++index)
         {
 
         CountDragItems( (DragReference)m_currentDrag, &items );
         for (UInt16 index = 1; index <= items; ++index)
         {
-            flavors = 0 ;
+            flavors = 0;
             GetDragItemReferenceNumber( (DragReference)m_currentDrag, index, &theItem );
             GetDragItemReferenceNumber( (DragReference)m_currentDrag, index, &theItem );
-            CountDragItemFlavors( (DragReference)m_currentDrag, theItem , &flavors );
-            bool hasPreferredFormat = false ;
-            wxDataFormat preferredFormat = m_dataObject->GetPreferredFormat( wxDataObject::Set ) ;
+            CountDragItemFlavors( (DragReference)m_currentDrag, theItem, &flavors );
+            wxDataFormat preferredFormat = m_dataObject->GetPreferredFormat( wxDataObject::Set );
+            bool hasPreferredFormat = false;
 
 
-            for ( UInt16 flavor = 1 ; flavor <= flavors ; ++flavor )
+            for ( UInt16 flavor = 1; flavor <= flavors; ++flavor )
             {
                 result = GetFlavorType( (DragReference)m_currentDrag, theItem, flavor, &theType );
                 wxDataFormat format( theType );
                 if ( preferredFormat == format )
                 {
             {
                 result = GetFlavorType( (DragReference)m_currentDrag, theItem, flavor, &theType );
                 wxDataFormat format( theType );
                 if ( preferredFormat == format )
                 {
-                    hasPreferredFormat = true ;
-                    break ;
+                    hasPreferredFormat = true;
+                    break;
                 }
             }
 
                 }
             }
 
-            for ( UInt16 flavor = 1 ; flavor <= flavors ; ++flavor )
+            for ( UInt16 flavor = 1; flavor <= flavors; ++flavor )
             {
             {
-                result = GetFlavorType((DragReference)m_currentDrag, theItem, flavor , &theType);
-                wxDataFormat format(theType) ;
+                result = GetFlavorType( (DragReference)m_currentDrag, theItem, flavor, &theType );
+                wxDataFormat format( theType );
                 if ( (hasPreferredFormat && format == preferredFormat)
                     || (!hasPreferredFormat && m_dataObject->IsSupportedFormat( format )))
                 {
                     result = GetFlavorFlags( (DragReference)m_currentDrag, theItem, theType, &theFlags );
                     if (result == noErr)
                     {
                 if ( (hasPreferredFormat && format == preferredFormat)
                     || (!hasPreferredFormat && m_dataObject->IsSupportedFormat( format )))
                 {
                     result = GetFlavorFlags( (DragReference)m_currentDrag, theItem, theType, &theFlags );
                     if (result == noErr)
                     {
-                        Size dataSize ;
-                        Ptr theData ;
+                        Size dataSize;
+                        Ptr theData;
 
                         GetFlavorDataSize( (DragReference)m_currentDrag, theItem, theType, &dataSize );
                         if ( theType == kScrapFlavorTypeText )
                         {
 
                         GetFlavorDataSize( (DragReference)m_currentDrag, theItem, theType, &dataSize );
                         if ( theType == kScrapFlavorTypeText )
                         {
-                            // this increment is only valid for allocating, on the next GetFlavorData
-                            // call it is reset again to the original value
-                            dataSize++ ;
+                            // this increment is only valid for allocating:
+                            // on the next GetFlavorData call it is reset again to the original value
+                            dataSize++;
                         }
                         else if ( theType == kScrapFlavorTypeUnicode )
                         {
                         }
                         else if ( theType == kScrapFlavorTypeUnicode )
                         {
-                            // this increment is only valid for allocating, on the next GetFlavorData
-                            // call it is reset again to the original value
-                            dataSize++ ;
-                            dataSize++ ;
+                            // this increment is only valid for allocating:
+                            // on the next GetFlavorData call it is reset again to the original value
+                            dataSize++;
+                            dataSize++;
                         }
 
                         }
 
-                        theData = new char[dataSize];
+                        if (dataSize > 0)
+                            theData = new char[dataSize];
+                        else
+                            theData = NULL;
 
                         GetFlavorData( (DragReference)m_currentDrag, theItem, theType, (void*) theData, &dataSize, 0L );
 
                         GetFlavorData( (DragReference)m_currentDrag, theItem, theType, (void*) theData, &dataSize, 0L );
-                        if ( theType == kScrapFlavorTypeText )
-                        {
-                            theData[dataSize] = 0 ;
-                            m_dataObject->SetData( wxDataFormat(wxDF_TEXT), dataSize , theData );
-                        }
- #if wxUSE_UNICODE
-                        else if ( theType == kScrapFlavorTypeUnicode )
+                        switch (theType)
                         {
                         {
+                        case kScrapFlavorTypeText:
+                            theData[dataSize] = 0;
+                            m_dataObject->SetData( wxDataFormat(wxDF_TEXT), dataSize, theData );
+                            break;
+
+#if wxUSE_UNICODE
+                        case kScrapFlavorTypeUnicode:
                             theData[dataSize + 0] =
                             theData[dataSize + 0] =
-                            theData[dataSize + 1] = 0 ;
-                            m_dataObject->SetData( wxDataFormat(wxDF_UNICODETEXT), dataSize , theData );
-                        }
- #endif
-                        else if ( theType == kDragFlavorTypeHFS )
-                        {
-                            HFSFlavor* theFile = (HFSFlavor*) theData ;
-                            wxString name = wxMacFSSpec2MacFilename( &theFile->fileSpec ) ;
-                            if ( !firstFileAdded )
+                            theData[dataSize + 1] = 0;
+                            m_dataObject->SetData( wxDataFormat(wxDF_UNICODETEXT), dataSize, theData );
+                            break;
+#endif
+
+                        case kDragFlavorTypeHFS:
                             {
                             {
-                                // reset file list
-                                ((wxFileDataObject*)m_dataObject)->SetData( 0 , "" ) ;
-                                firstFileAdded = true ;
+                                wxFileDataObject *fdo = dynamic_cast<wxFileDataObject*>(m_dataObject);
+                                wxASSERT( fdo != NULL );
+
+                                if ((theData != NULL) && (fdo != NULL))
+                                {
+                                    HFSFlavor* theFile = (HFSFlavor*) theData;
+                                    wxString name = wxMacFSSpec2MacFilename( &theFile->fileSpec );
+
+                                    if ( !firstFileAdded )
+                                    {
+                                        // reset file list
+                                        fdo->SetData( 0, "" );
+                                        firstFileAdded = true;
+                                    }
+
+                                    if (!name.IsEmpty())
+                                        fdo->AddFile( name );
+                                }
                             }
                             }
-
-                            ((wxFileDataObject*)m_dataObject)->AddFile( name ) ;
-                        }
-                        else
-                        {
+                            break;
+ 
+                        default:
                             m_dataObject->SetData( format, dataSize, theData );
                             m_dataObject->SetData( format, dataSize, theData );
+                            break;
                         }
 
                         delete [] theData;
                     }
                         }
 
                         delete [] theData;
                     }
-                    break ;
+                    break;
                 }
             }
         }
     }
 
                 }
             }
         }
     }
 
-    return true ;
+    return true;
 }
 
 //-------------------------------------------------------------------------
 }
 
 //-------------------------------------------------------------------------
@@ -294,7 +311,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;
 }
 
@@ -305,7 +323,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;
 }
@@ -323,52 +342,52 @@ wxDragResult wxDropSource::DoDragDrop(int flags)
     if (m_data->GetFormatCount() == 0)
         return (wxDragResult) wxDragNone;
 
     if (m_data->GetFormatCount() == 0)
         return (wxDragResult) wxDragNone;
 
-    OSErr result;
+    OSStatus result;
     DragReference theDrag;
     RgnHandle dragRegion;
     if ((result = NewDrag(&theDrag)) != noErr)
     DragReference theDrag;
     RgnHandle dragRegion;
     if ((result = NewDrag(&theDrag)) != noErr)
-        return wxDragNone ;
+        return wxDragNone;
 
     // add data to drag
 
     // add data to drag
-    size_t formatCount = m_data->GetFormatCount() ;
-    wxDataFormat *formats = new wxDataFormat[formatCount] ;
-    m_data->GetAllFormats( formats ) ;
-    ItemReference theItem = 1 ;
+    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 )
+    for ( size_t i = 0; i < formatCount; ++i )
     {
     {
-        size_t dataSize = m_data->GetDataSize( formats[i] ) ;
-        Ptr dataPtr = new char[dataSize] ;
-        m_data->GetDataHere( formats[i] , dataPtr ) ;
-        OSType type = formats[i].GetFormatId() ;
+        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' || type == 'utxt' )
         {
             if ( dataSize > 0 )
         if ( type == 'TEXT' || type == 'utxt' )
         {
             if ( dataSize > 0 )
-                dataSize-- ;
-            dataPtr[ dataSize ] = 0 ;
+                dataSize--;
+            dataPtr[ dataSize ] = 0;
             if ( type == 'utxt' )
             {
                 if ( dataSize > 0 )
             if ( type == 'utxt' )
             {
                 if ( dataSize > 0 )
-                    dataSize-- ;
-                dataPtr[ dataSize ] = 0 ;
+                    dataSize--;
+                dataPtr[ dataSize ] = 0;
             }
 
             }
 
-            AddDragItemFlavor(theDrag, theItem, type , dataPtr, dataSize, 0);
+            AddDragItemFlavor( theDrag, theItem, type, dataPtr, dataSize, 0 );
         }
         else if (type == kDragFlavorTypeHFS )
         {
         }
         else if (type == kDragFlavorTypeHFS )
         {
-            HFSFlavor  theFlavor ;
+            HFSFlavor  theFlavor;
             OSErr err = noErr;
             CInfoPBRec cat;
 
             OSErr err = noErr;
             CInfoPBRec cat;
 
-            wxMacFilename2FSSpec( wxString( dataPtr , *wxConvCurrent ) , &theFlavor.fileSpec ) ;
+            wxMacFilename2FSSpec( wxString( dataPtr, *wxConvCurrent ), &theFlavor.fileSpec );
 
             memset( &cat, 0, sizeof(cat) );
             cat.hFileInfo.ioNamePtr = theFlavor.fileSpec.name;
             cat.hFileInfo.ioVRefNum = theFlavor.fileSpec.vRefNum;
             cat.hFileInfo.ioDirID = theFlavor.fileSpec.parID;
             cat.hFileInfo.ioFDirIndex = 0;
 
             memset( &cat, 0, sizeof(cat) );
             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);
+            err = PBGetCatInfoSync( &cat );
             if (err == noErr)
             {
                 theFlavor.fdFlags = cat.hFileInfo.ioFlFndrInfo.fdFlags;
             if (err == noErr)
             {
                 theFlavor.fdFlags = cat.hFileInfo.ioFlFndrInfo.fdFlags;
@@ -388,63 +407,63 @@ wxDragResult wxDropSource::DoDragDrop(int flags)
                     theFlavor.fileType = cat.hFileInfo.ioFlFndrInfo.fdType;
                 }
 
                     theFlavor.fileType = cat.hFileInfo.ioFlFndrInfo.fdType;
                 }
 
-                AddDragItemFlavor(theDrag, theItem, type , &theFlavor, sizeof(theFlavor), 0);
+                AddDragItemFlavor( theDrag, theItem, type, &theFlavor, sizeof(theFlavor), 0 );
             }
         }
         else
         {
             }
         }
         else
         {
-            AddDragItemFlavor(theDrag, theItem, type , dataPtr, dataSize, 0);
+            AddDragItemFlavor( theDrag, theItem, type, dataPtr, dataSize, 0 );
         }
 
         }
 
-        delete [] dataPtr ;
+        delete [] dataPtr;
     }
 
     }
 
-    delete [] formats ;
+    delete [] formats;
 
     dragRegion = NewRgn();
 
     dragRegion = NewRgn();
-    RgnHandle tempRgn = NewRgn() ;
+    RgnHandle tempRgn = NewRgn();
 
 
-    EventRecord* ev = NULL ;
+    EventRecord* ev = NULL;
 
 #if !TARGET_CARBON // TODO
 
 #if !TARGET_CARBON // TODO
-    ev = (EventRecord*) wxTheApp->MacGetCurrentEvent() ;
+    ev = (EventRecord*) wxTheApp->MacGetCurrentEvent();
 #else
 #else
-    {
-        EventRecord rec ;
-        ev = &rec ;
-        wxMacConvertEventToRecord( (EventRef) wxTheApp->MacGetCurrentEvent() , &rec ) ;
-    }
+    EventRecord rec;
+    ev = &rec;
+    wxMacConvertEventToRecord( (EventRef) wxTheApp->MacGetCurrentEvent(), &rec );
 #endif
 
 #endif
 
-    const short dragRegionOuterBoundary = 10 ;
-    const short dragRegionInnerBoundary = 9 ;
+    const short dragRegionOuterBoundary = 10;
+    const short dragRegionInnerBoundary = 9;
 
     SetRectRgn(
 
     SetRectRgn(
-        dragRegion , ev->where.h - dragRegionOuterBoundary ,
-        ev->where.v  - dragRegionOuterBoundary ,
-        ev->where.h + dragRegionOuterBoundary ,
-        ev->where.v + dragRegionOuterBoundary ) ;
+        dragRegion,
+        ev->where.h - dragRegionOuterBoundary,
+        ev->where.v  - dragRegionOuterBoundary,
+        ev->where.h + dragRegionOuterBoundary,
+        ev->where.v + dragRegionOuterBoundary );
 
     SetRectRgn(
 
     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 ;
-
-    bool optionDown = GetCurrentKeyModifiers() & optionKey ;
-    wxDragResult dndresult = wxDragCopy ;
+        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;
+
+    bool optionDown = GetCurrentKeyModifiers() & optionKey;
+    wxDragResult dndresult = wxDragCopy;
     if ( flags != wxDrag_CopyOnly )
         // on mac the option key is always the indication for copy
         dndresult = optionDown ? wxDragCopy : wxDragMove;
     if ( flags != wxDrag_CopyOnly )
         // on mac the option key is always the indication for copy
         dndresult = optionDown ? wxDragCopy : wxDragMove;
@@ -458,50 +477,53 @@ bool wxDropSource::MacInstallDefaultCursor(wxDragResult effect)
     bool result = cursor.Ok();
 
     if ( result )
     bool result = cursor.Ok();
 
     if ( result )
-        cursor.MacInstall() ;
+        cursor.MacInstall();
 
     return result;
 }
 
 
     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...
 
 pascal OSErr wxMacWindowDragTrackingHandler(
     DragTrackingMessage theMessage, WindowPtr theWindow,
 
 // 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) ;
+    void *handlerRefCon, DragReference theDrag );
 pascal OSErr wxMacWindowDragReceiveHandler(
     WindowPtr theWindow, void *handlerRefCon,
 pascal OSErr wxMacWindowDragReceiveHandler(
     WindowPtr theWindow, void *handlerRefCon,
-    DragReference theDrag) ;
+    DragReference theDrag );
 
 void wxMacEnsureTrackingHandlersInstalled()
 {
     if ( !gTrackingGlobalsInstalled )
     {
 
 void wxMacEnsureTrackingHandlersInstalled()
 {
     if ( !gTrackingGlobalsInstalled )
     {
-        OSErr result;
+        OSStatus err;
 
 
-        result = InstallTrackingHandler(NewDragTrackingHandlerUPP(wxMacWindowDragTrackingHandler), 0L,&gTrackingGlobals);
-        wxASSERT( result == noErr ) ;
+        err = InstallTrackingHandler( NewDragTrackingHandlerUPP(wxMacWindowDragTrackingHandler), 0L, &gTrackingGlobals );
+        verify_noerr( err );
 
 
-        result = InstallReceiveHandler(NewDragReceiveHandlerUPP(wxMacWindowDragReceiveHandler), 0L, &gTrackingGlobals);
-        wxASSERT( result == noErr ) ;
+        err = InstallReceiveHandler( NewDragReceiveHandlerUPP(wxMacWindowDragReceiveHandler), 0L, &gTrackingGlobals );
+        verify_noerr( err );
 
 
-        gTrackingGlobalsInstalled = true ;
+        gTrackingGlobalsInstalled = true;
     }
 }
 
 pascal OSErr wxMacWindowDragTrackingHandler(
     DragTrackingMessage theMessage, WindowPtr theWindow,
     }
 }
 
 pascal OSErr wxMacWindowDragTrackingHandler(
     DragTrackingMessage theMessage, WindowPtr theWindow,
-    void *handlerRefCon, DragReference theDrag)
+    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 ) ;
 
 
-    bool optionDown = GetCurrentKeyModifiers() & optionKey ;
+    GetDragAttributes( theDrag, &attributes );
+
+    wxTopLevelWindowMac* toplevel = wxFindWinFromMacWindow( theWindow );
+
+    bool optionDown = GetCurrentKeyModifiers() & optionKey;
     wxDragResult result = optionDown ? wxDragCopy : wxDragMove;
 
     switch (theMessage)
     wxDragResult result = optionDown ? wxDragCopy : wxDragMove;
 
     switch (theMessage)
@@ -511,35 +533,39 @@ pascal OSErr wxMacWindowDragTrackingHandler(
             break;
 
         case kDragTrackingEnterWindow:
             break;
 
         case kDragTrackingEnterWindow:
-            trackingGlobals->m_currentTargetWindow = NULL ;
-            trackingGlobals->m_currentTarget = NULL ;
+            if (trackingGlobals != NULL)
+            {
+                trackingGlobals->m_currentTargetWindow = NULL;
+                trackingGlobals->m_currentTarget = NULL;
+            }
             break;
 
         case kDragTrackingInWindow:
             break;
 
         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);
+            GlobalToLocal( &localMouse );
 
             {
 
             {
-                wxWindow *win = NULL ;
-                ControlPartCode controlPart ;
+                wxWindow *win = NULL;
+                ControlPartCode controlPart;
                 ControlRef control = wxMacFindControlUnderMouse(
                 ControlRef control = wxMacFindControlUnderMouse(
-                    toplevel , localMouse ,
-                    theWindow , &controlPart ) ;
+                    toplevel, localMouse, theWindow, &controlPart );
                 if ( control )
                 if ( control )
-                    win = wxFindControlFromMacControl( control ) ;
+                    win = wxFindControlFromMacControl( control );
                 else
                 else
-                    win = toplevel ;
+                    win = toplevel;
 
 
-                int localx , localy ;
-                localx = localMouse.h ;
-                localy = localMouse.v ;
+                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 )
@@ -558,26 +584,26 @@ pascal OSErr wxMacWindowDragTrackingHandler(
                     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 )
                             {
                         {
                             if ( trackingGlobals->m_currentTarget )
                             {
-                                trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag ) ;
-                                result = trackingGlobals->m_currentTarget->OnEnter( localx , localy , result ) ;
+                                trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag );
+                                result = trackingGlobals->m_currentTarget->OnEnter( localx, localy, result );
                             }
 
                             if ( result != wxDragNone )
                             {
                             }
 
                             if ( result != wxDragNone )
                             {
-                                int x , y ;
+                                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 ) ;
+                                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 );
                                 ShowDragHilite( theDrag, hiliteRgn, true );
                                 ShowDragHilite( theDrag, hiliteRgn, true );
-                                DisposeRgn( hiliteRgn ) ;
+                                DisposeRgn( hiliteRgn );
                             }
                         }
                     }
                             }
                         }
                     }
@@ -586,38 +612,38 @@ pascal OSErr wxMacWindowDragTrackingHandler(
                 {
                     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 );
+                        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 ) )
+                if ( trackingGlobals->m_currentSource && !trackingGlobals->m_currentSource->GiveFeedback( result ) )
                 {
                   if ( !trackingGlobals->m_currentSource->MacInstallDefaultCursor( result ) )
                   {
                       switch ( result )
                       {
                 {
                   if ( !trackingGlobals->m_currentSource->MacInstallDefaultCursor( result ) )
                   {
                       switch ( result )
                       {
-                          case wxDragCopy :
+                          case wxDragCopy:
                               {
                               {
-                                  wxCursor cursor(wxCURSOR_COPY_ARROW) ;
-                                  cursor.MacInstall() ;
+                                  wxCursor cursor(wxCURSOR_COPY_ARROW);
+                                  cursor.MacInstall();
                               }
                               }
-                              break ;
+                              break;
 
 
-                          case wxDragMove :
+                          case wxDragMove:
                               {
                               {
-                                  wxCursor cursor(wxCURSOR_ARROW) ;
-                                  cursor.MacInstall() ;
+                                  wxCursor cursor(wxCURSOR_ARROW);
+                                  cursor.MacInstall();
                               }
                               }
-                              break ;
+                              break;
 
 
-                          case wxDragNone :
+                          case wxDragNone:
                               {
                               {
-                                  wxCursor cursor(wxCURSOR_NO_ENTRY) ;
-                                  cursor.MacInstall() ;
+                                  wxCursor cursor(wxCURSOR_NO_ENTRY);
+                                  cursor.MacInstall();
                               }
                               }
-                              break ;
+                              break;
 
                           case wxDragError:
                           case wxDragLink:
 
                           case wxDragError:
                           case wxDragLink:
@@ -632,6 +658,9 @@ pascal OSErr wxMacWindowDragTrackingHandler(
             break;
 
         case kDragTrackingLeaveWindow:
             break;
 
         case kDragTrackingLeaveWindow:
+            if (trackingGlobals == NULL)
+                break;
+
             if (trackingGlobals->m_currentTarget)
             {
                 trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag );
             if (trackingGlobals->m_currentTarget)
             {
                 trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag );
@@ -657,24 +686,24 @@ pascal OSErr wxMacWindowDragReceiveHandler(
     MacTrackingGlobals* trackingGlobals = (MacTrackingGlobals*)handlerRefCon;
     if ( trackingGlobals->m_currentTarget )
     {
     MacTrackingGlobals* trackingGlobals = (MacTrackingGlobals*)handlerRefCon;
     if ( trackingGlobals->m_currentTarget )
     {
-        Point mouse, localMouse ;
-        int localx, localy ;
+        Point mouse, localMouse;
+        int localx, localy;
 
 
-        trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag ) ;
-        GetDragMouse(theDrag, &mouse, 0L);
+        trackingGlobals->m_currentTarget->SetCurrentDrag( theDrag );
+        GetDragMouse( theDrag, &mouse, 0L );
         localMouse = mouse;
         localMouse = mouse;
-        GlobalToLocal(&localMouse);
-        localx = localMouse.h ;
-        localy = localMouse.v ;
+        GlobalToLocal( &localMouse );
+        localx = localMouse.h;
+        localy = localMouse.v;
 
         // TODO : should we use client coordinates?
         if ( trackingGlobals->m_currentTargetWindow )
 
         // TODO : should we use client coordinates?
         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 ) )
         {
         {
-            bool optionDown = GetCurrentKeyModifiers() & optionKey ;
+            bool optionDown = GetCurrentKeyModifiers() & optionKey;
             wxDragResult result = optionDown ? wxDragCopy : wxDragMove;
             wxDragResult result = optionDown ? wxDragCopy : wxDragMove;
-            trackingGlobals->m_currentTarget->OnData( localx , localy , result ) ;
+            trackingGlobals->m_currentTarget->OnData( localx, localy, result );
         }
     }
 
         }
     }
 
wxWidgets (Impactor)