Fix [ 1574240 ] wx.RadioButton doesn't navigate correctly

[wxWidgets.git] / src / mgl / cursor.cpp

diff --git a/src/mgl/cursor.cpp b/src/mgl/cursor.cpp
index eb959e8ff8c3f6dfcf16a24867fea64e90e048df..709b59e34f341f94e25fcfd6366814c77656e59e 100644 (file)
--- a/src/mgl/cursor.cpp
+++ b/src/mgl/cursor.cpp
@@ -1,17 +1,12 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        cursor.cpp
+// Name:        src/mgl/cursor.cpp

 // Purpose:
 // Author:      Vaclav Slavik
 // Id:          $Id$
 // Purpose:
 // Author:      Vaclav Slavik
 // Id:          $Id$

-// Copyright:   (c) 2001 SciTech Software, Inc. (www.scitechsoft.com)
+// Copyright:   (c) 2001-2002 SciTech Software, Inc. (www.scitechsoft.com)

 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 

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

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


@@ -20,9 +15,14 @@
 #endif
 
 #include "wx/cursor.h"
 #endif
 
 #include "wx/cursor.h"

-#include "wx/utils.h"
-#include "wx/log.h"
-#include "wx/intl.h"
+
+#ifndef WX_PRECOMP
+    #include "wx/intl.h"
+    #include "wx/log.h"
+    #include "wx/utils.h"
+    #include "wx/hashmap.h"
+    #include "wx/module.h"
+#endif

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


@@ -36,7 +36,7 @@ class wxCursorRefData: public wxObjectRefData
   public:
 
     wxCursorRefData();
   public:
 
     wxCursorRefData();

-    ~wxCursorRefData();
+    virtual ~wxCursorRefData();

 
     MGLCursor *m_cursor;
 };
 
     MGLCursor *m_cursor;
 };


@@ -51,9 +51,13 @@ wxCursorRefData::~wxCursorRefData()
     delete m_cursor;
 }
 
     delete m_cursor;
 }
 

+#define M_CURSORDATA ((wxCursorRefData *)m_refData)
+

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

-#define M_CURSORDATA ((wxCursorRefData *)m_refData)
+WX_DECLARE_HASH_MAP(int, wxCursor, wxIntegerHash, wxIntegerEqual, wxCursorsHash);
+
+static wxCursorsHash *gs_cursorsHash = NULL;

 
 IMPLEMENT_DYNAMIC_CLASS(wxCursor,wxObject)
 
 
 IMPLEMENT_DYNAMIC_CLASS(wxCursor,wxObject)
 


@@ -63,15 +67,23 @@ wxCursor::wxCursor()
 
 wxCursor::wxCursor(int cursorId)
 {
 
 wxCursor::wxCursor(int cursorId)
 {

+    if ( !gs_cursorsHash )
+        gs_cursorsHash = new wxCursorsHash;
+
+    if ( gs_cursorsHash->find(cursorId) != gs_cursorsHash->end() )
+    {
+        wxLogTrace(_T("mglcursor"), _T("cursor id %i fetched from cache"), cursorId);
+        *this = (*gs_cursorsHash)[cursorId];
+        return;
+    }
+

     const char *cursorname = NULL;
     m_refData = new wxCursorRefData();
 
     switch (cursorId)
     {
     const char *cursorname = NULL;
     m_refData = new wxCursorRefData();
 
     switch (cursorId)
     {

-        // FIXME_MGL -- what about storing these default cursors in executable
-        //              as XPMs so that wxMGL binary wouldn't depend on 
-        //              tons of files in $MGL_ROOT/cursors? I don't know yet...

         case wxCURSOR_ARROW:           cursorname = "arrow.cur"; break;
         case wxCURSOR_ARROW:           cursorname = "arrow.cur"; break;

+        case wxCURSOR_RIGHT_ARROW:     cursorname = "rightarr.cur"; break;

         case wxCURSOR_BULLSEYE:        cursorname = "bullseye.cur"; break;
         case wxCURSOR_CHAR:            cursorname = "char.cur"; break;
         case wxCURSOR_CROSS:           cursorname = "cross.cur"; break;
         case wxCURSOR_BULLSEYE:        cursorname = "bullseye.cur"; break;
         case wxCURSOR_CHAR:            cursorname = "char.cur"; break;
         case wxCURSOR_CROSS:           cursorname = "cross.cur"; break;


@@ -100,13 +112,12 @@ wxCursor::wxCursor(int cursorId)
         case wxCURSOR_NONE:
             *this = wxNullCursor;
             return;
         case wxCURSOR_NONE:
             *this = wxNullCursor;
             return;

-            break;

 
         default:
             wxFAIL_MSG(wxT("unsupported cursor type"));
             break;
     }
 
         default:
             wxFAIL_MSG(wxT("unsupported cursor type"));
             break;
     }

-    
+

     M_CURSORDATA->m_cursor = new MGLCursor(cursorname);
 
     // if we cannot load arrow cursor, use MGL's default arrow cursor:
     M_CURSORDATA->m_cursor = new MGLCursor(cursorname);
 
     // if we cannot load arrow cursor, use MGL's default arrow cursor:


@@ -115,12 +126,18 @@ wxCursor::wxCursor(int cursorId)
         delete M_CURSORDATA->m_cursor;
         M_CURSORDATA->m_cursor = new MGLCursor(MGL_DEF_CURSOR);
     }
         delete M_CURSORDATA->m_cursor;
         M_CURSORDATA->m_cursor = new MGLCursor(MGL_DEF_CURSOR);
     }

-    
+

     if ( !M_CURSORDATA->m_cursor->valid() )
     {
         wxLogError(_("Couldn't create cursor."));
         UnRef();
     }
     if ( !M_CURSORDATA->m_cursor->valid() )
     {
         wxLogError(_("Couldn't create cursor."));
         UnRef();
     }

+    else
+    {
+        (*gs_cursorsHash)[cursorId] = *this;
+        wxLogTrace(_T("mglcursor"), _T("cursor id %i added to cache (%s)"),
+                   cursorId, cursorname);
+    }

 }
 
 wxCursor::wxCursor(const char WXUNUSED(bits)[],
 }
 
 wxCursor::wxCursor(const char WXUNUSED(bits)[],


@@ -135,7 +152,7 @@ wxCursor::wxCursor(const char WXUNUSED(bits)[],
 
 wxCursor::wxCursor(const wxString& cursor_file,
                    long flags,
 
 wxCursor::wxCursor(const wxString& cursor_file,
                    long flags,

-                   int hotSpotX, int hotSpotY)
+                   int WXUNUSED(hotSpotX), int WXUNUSED(hotSpotY))

 {
     if ( flags == wxBITMAP_TYPE_CUR || flags == wxBITMAP_TYPE_CUR_RESOURCE )
     {
 {
     if ( flags == wxBITMAP_TYPE_CUR || flags == wxBITMAP_TYPE_CUR_RESOURCE )
     {


@@ -153,24 +170,11 @@ wxCursor::wxCursor(const wxString& cursor_file,
     }
 }
 
     }
 }
 

-wxCursor::wxCursor(const wxCursor &cursor)
-{
-    Ref(cursor);
-}
-

 wxCursor::~wxCursor()
 {
     // wxObject unrefs data
 }
 
 wxCursor::~wxCursor()
 {
     // wxObject unrefs data
 }
 

-wxCursor& wxCursor::operator = (const wxCursor& cursor)
-{
-    if ( *this == cursor )
-        return (*this);
-    Ref(cursor);
-    return *this;
-}
-

 bool wxCursor::operator == (const wxCursor& cursor) const
 {
     return (m_refData == cursor.m_refData);
 bool wxCursor::operator == (const wxCursor& cursor) const
 {
     return (m_refData == cursor.m_refData);


@@ -181,7 +185,7 @@ bool wxCursor::operator != (const wxCursor& cursor) const
     return (m_refData != cursor.m_refData);
 }
 
     return (m_refData != cursor.m_refData);
 }
 

-bool wxCursor::Ok() const
+bool wxCursor::IsOk() const

 {
     return (m_refData != NULL);
 }
 {
     return (m_refData != NULL);
 }


@@ -197,14 +201,21 @@ MGLCursor *wxCursor::GetMGLCursor() const
 // Global cursor setting
 // ----------------------------------------------------------------------------
 
 // Global cursor setting
 // ----------------------------------------------------------------------------
 

-static wxCursor  g_globalCursor = wxNullCursor;
+static wxCursor  gs_globalCursor = wxNullCursor;

 
 void wxSetCursor(const wxCursor& cursor)
 {
     if ( cursor.Ok() )
     {
 
 void wxSetCursor(const wxCursor& cursor)
 {
     if ( cursor.Ok() )
     {

-        MGL_wmSetGlobalCursor(g_winMng, *cursor.GetMGLCursor());
-        g_globalCursor = cursor;
+        if ( g_winMng )
+            MGL_wmSetGlobalCursor(g_winMng, *cursor.GetMGLCursor());
+        gs_globalCursor = cursor;
+    }
+    else
+    {
+        if ( g_winMng )
+            MGL_wmSetGlobalCursor(g_winMng, NULL);
+        gs_globalCursor = wxNullCursor;

     }
 }
 
     }
 }
 


@@ -224,7 +235,7 @@ const wxCursor &wxBusyCursor::GetStoredCursor()
 
 const wxCursor wxBusyCursor::GetBusyCursor()
 {
 
 const wxCursor wxBusyCursor::GetBusyCursor()
 {

-    return wxCursor(wxCURSOR_WAIT);
+    return gs_globalCursor;

 }
 
 void wxEndBusyCursor()
 }
 
 void wxEndBusyCursor()


@@ -235,14 +246,14 @@ void wxEndBusyCursor()
     gs_savedCursor = wxNullCursor;
 }
 
     gs_savedCursor = wxNullCursor;
 }
 

-void wxBeginBusyCursor(wxCursor *cursor)
+void wxBeginBusyCursor(const wxCursor *cursor)

 {
     if ( gs_busyCount++ > 0 ) return;
 
     wxASSERT_MSG( !gs_savedCursor.Ok(),
                   wxT("forgot to call wxEndBusyCursor, will leak memory") );
 
 {
     if ( gs_busyCount++ > 0 ) return;
 
     wxASSERT_MSG( !gs_savedCursor.Ok(),
                   wxT("forgot to call wxEndBusyCursor, will leak memory") );
 

-    gs_savedCursor = g_globalCursor;
+    gs_savedCursor = gs_globalCursor;

     if ( cursor->Ok() )
         wxSetCursor(*cursor);
     else
     if ( cursor->Ok() )
         wxSetCursor(*cursor);
     else


@@ -254,3 +265,24 @@ bool wxIsBusy()
     return (gs_busyCount > 0);
 }
 
     return (gs_busyCount > 0);
 }
 

+
+
+//-----------------------------------------------------------------------------
+// module - clean up code
+//-----------------------------------------------------------------------------
+
+class wxCursorModule : public wxModule
+{
+public:
+    virtual bool OnInit() { return true; }
+
+    virtual void OnExit()
+    {
+        wxDELETE(gs_cursorsHash);
+    }
+
+private:
+    DECLARE_DYNAMIC_CLASS(wxCursorModule)
+};
+
+IMPLEMENT_DYNAMIC_CLASS(wxCursorModule, wxModule)