some really minor changes (the most important one: small memory hole in

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

diff --git a/src/gtk1/cursor.cpp b/src/gtk1/cursor.cpp
index 1f84b80f69cb51d89913d26d9618b1cb9b339d39..eac07cc9047a3bdf414e60082227418f2f2d4b2c 100644 (file)
--- a/src/gtk1/cursor.cpp
+++ b/src/gtk1/cursor.cpp
@@ -2,10 +2,9 @@
 // Name:        cursor.cpp
 // Purpose:
 // Author:      Robert Roebling
-// Created:     01/02/97
-// Id:
-// Copyright:   (c) 1998 Robert Roebling, Julian Smart and Markus Holzem
-// Licence:    wxWindows licence
+// Id:          $Id$
+// Copyright:   (c) 1998 Robert Roebling
+// Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 
@@ -15,6 +14,8 @@
 
 #include "wx/cursor.h"
 
+#include "gdk/gdk.h"
+
 //-----------------------------------------------------------------------------
 // wxCursor
 //-----------------------------------------------------------------------------
@@ -22,22 +23,22 @@
 class wxCursorRefData: public wxObjectRefData
 {
   public:
-  
-    wxCursorRefData(void);
-    ~wxCursorRefData(void);
-  
+
+    wxCursorRefData();
+    ~wxCursorRefData();
+
     GdkCursor *m_cursor;
 };
 
-wxCursorRefData::wxCursorRefData(void)
+wxCursorRefData::wxCursorRefData()
 {
-  m_cursor = NULL;
-};
+    m_cursor = (GdkCursor *) NULL;
+}
 
-wxCursorRefData::~wxCursorRefData(void)
+wxCursorRefData::~wxCursorRefData()
 {
-  if (m_cursor) gdk_cursor_destroy( m_cursor );
-};
+    if (m_cursor) gdk_cursor_destroy( m_cursor );
+}
 
 //-----------------------------------------------------------------------------
 
@@ -45,101 +46,84 @@ wxCursorRefData::~wxCursorRefData(void)
 
 IMPLEMENT_DYNAMIC_CLASS(wxCursor,wxObject)
 
-wxCursor::wxCursor(void)
+wxCursor::wxCursor()
 {
-};
+}
 
 wxCursor::wxCursor( int cursorId )
 {
-  m_refData = new wxCursorRefData();
-  
-  GdkCursorType gdk_cur = GDK_LEFT_PTR;
-  switch (cursorId)
-  {
-    case wxCURSOR_HAND:       gdk_cur = GDK_HAND1; break;
-    case wxCURSOR_CROSS:      gdk_cur = GDK_CROSSHAIR; break;
-    case wxCURSOR_SIZEWE:     gdk_cur = GDK_SB_H_DOUBLE_ARROW; break;
-    case wxCURSOR_SIZENS:     gdk_cur = GDK_SB_V_DOUBLE_ARROW; break;
-    case wxCURSOR_WAIT:       gdk_cur = GDK_WATCH; break;
-    case wxCURSOR_WATCH:      gdk_cur = GDK_WATCH; break;
-    case wxCURSOR_SIZING:     gdk_cur = GDK_SIZING; break;
-    case wxCURSOR_SPRAYCAN:   gdk_cur = GDK_SPRAYCAN; break;
-    case wxCURSOR_IBEAM:      gdk_cur = GDK_XTERM; break;
-    case wxCURSOR_PENCIL:     gdk_cur = GDK_PENCIL; break;
-    case wxCURSOR_NO_ENTRY:   gdk_cur = GDK_PIRATE; break;
-  };
-  
-  M_CURSORDATA->m_cursor = gdk_cursor_new( gdk_cur );
-  
+    m_refData = new wxCursorRefData();
+
+    GdkCursorType gdk_cur = GDK_LEFT_PTR;
+    switch (cursorId)
+    {
+        case wxCURSOR_HAND:             gdk_cur = GDK_HAND1; break;
+        case wxCURSOR_CROSS:            gdk_cur = GDK_CROSSHAIR; break;
+        case wxCURSOR_SIZEWE:           gdk_cur = GDK_SB_H_DOUBLE_ARROW; break;
+        case wxCURSOR_SIZENS:           gdk_cur = GDK_SB_V_DOUBLE_ARROW; break;
+        case wxCURSOR_WAIT:                                        
+        case wxCURSOR_WATCH:            gdk_cur = GDK_WATCH; break;
+        case wxCURSOR_SIZING:           gdk_cur = GDK_SIZING; break;
+        case wxCURSOR_SPRAYCAN:         gdk_cur = GDK_SPRAYCAN; break;
+        case wxCURSOR_IBEAM:            gdk_cur = GDK_XTERM; break;
+        case wxCURSOR_PENCIL:           gdk_cur = GDK_PENCIL; break;
+        case wxCURSOR_NO_ENTRY:         gdk_cur = GDK_PIRATE; break;
+        case wxCURSOR_SIZENWSE:                                     
+        case wxCURSOR_SIZENESW:         gdk_cur = GDK_FLEUR; break;
+        case wxCURSOR_QUESTION_ARROW:   gdk_cur = GDK_QUESTION_ARROW; break;
+       case wxCURSOR_PAINT_BRUSH:      gdk_cur = GDK_SPRAYCAN; break;
+       case wxCURSOR_MAGNIFIER:        gdk_cur = GDK_PLUS; break;
+       case wxCURSOR_CHAR:             gdk_cur = GDK_XTERM; break;
+        case wxCURSOR_LEFT_BUTTON:      gdk_cur = GDK_LEFTBUTTON; break;
+        case wxCURSOR_MIDDLE_BUTTON:    gdk_cur = GDK_MIDDLEBUTTON; break;
+        case wxCURSOR_RIGHT_BUTTON:     gdk_cur = GDK_RIGHTBUTTON; break;
 /*
-  do that yourself
-   
-  wxCURSOR_BULLSEYE,
-  wxCURSOR_CHAR,
-  wxCURSOR_LEFT_BUTTON,
-  wxCURSOR_MAGNIFIER,
-  wxCURSOR_MIDDLE_BUTTON,
-  wxCURSOR_NO_ENTRY,
-  wxCURSOR_PAINT_BRUSH,
-  wxCURSOR_POINT_LEFT,
-  wxCURSOR_POINT_RIGHT,
-  wxCURSOR_QUESTION_ARROW,
-  wxCURSOR_RIGHT_BUTTON,
-  wxCURSOR_SIZENESW,
-  wxCURSOR_SIZENS,
-  wxCURSOR_SIZENWSE,
-  wxCURSOR_SIZEWE,
-  wxCURSOR_BLANK
-,
-  wxCURSOR_CROSS_REVERSE,
-  wxCURSOR_DOUBLE_ARROW,
-  wxCURSOR_BASED_ARROW_UP,
-  wxCURSOR_BASED_ARROW_DOWN
+       case wxCURSOR_DOUBLE_ARROW:     gdk_cur = GDK_DOUBLE_ARROW; break;
+       case wxCURSOR_CROSS_REVERSE:    gdk_cur = GDK_CROSS_REVERSE; break;
+        case wxCURSOR_BASED_ARROW_UP:   gdk_cur = GDK_BASED_ARROW_UP; break;
+        case wxCURSOR_BASED_ARROW_DOWN: gdk_cur = GDK_BASED_ARROW_DOWN; break;
 */
-   
-};
+    }
 
-wxCursor::wxCursor( const wxCursor &cursor )
-{
-  Ref( cursor );
-};
+    M_CURSORDATA->m_cursor = gdk_cursor_new( gdk_cur );
+}
 
-wxCursor::wxCursor( const wxCursor *cursor )
+wxCursor::wxCursor( const wxCursor &cursor )
 {
-  UnRef();
-  if (cursor) Ref( *cursor );
-};
+    Ref( cursor );
+}
 
-wxCursor::~wxCursor(void)
+wxCursor::~wxCursor()
 {
-};
+}
 
 wxCursor& wxCursor::operator = ( const wxCursor& cursor )
 {
-  if (*this == cursor) return (*this); 
-  Ref( cursor ); 
-  return *this; 
-};
+    if (*this == cursor) return (*this);
+    Ref( cursor );
+    
+    return *this;
+}
 
-bool wxCursor::operator == ( const wxCursor& cursor )
+bool wxCursor::operator == ( const wxCursor& cursor ) const
 {
-  return m_refData == cursor.m_refData; 
-};
+    return m_refData == cursor.m_refData;
+}
 
-bool wxCursor::operator != ( const wxCursor& cursor )
+bool wxCursor::operator != ( const wxCursor& cursor ) const
 {
-  return m_refData != cursor.m_refData; 
-};
+    return m_refData != cursor.m_refData;
+}
 
-bool wxCursor::Ok(void) const
+bool wxCursor::Ok() const
 {
-  return TRUE;
-};
+    return (m_refData != NULL);
+}
 
-GdkCursor *wxCursor::GetCursor(void) const
+GdkCursor *wxCursor::GetCursor() const
 {
-  return M_CURSORDATA->m_cursor;
-};
+    return M_CURSORDATA->m_cursor;
+}
 
 //-----------------------------------------------------------------------------
 // busy cursor routines
@@ -147,27 +131,27 @@ GdkCursor *wxCursor::GetCursor(void) const
 
 bool g_isBusy = FALSE;
 
-void wxEndBusyCursor(void)
+void wxEndBusyCursor()
 {
-  g_isBusy = FALSE;
-};
+    g_isBusy = FALSE;
+}
 
 void wxBeginBusyCursor( wxCursor *WXUNUSED(cursor) )
 {
-  g_isBusy = TRUE;
-};
+    g_isBusy = TRUE;
+}
 
-bool wxIsBusy(void)
+bool wxIsBusy()
 {
-  return g_isBusy;
-};
+    return g_isBusy;
+}
 
 void wxSetCursor( const wxCursor& cursor )
 {
-  extern wxCursor *g_globalCursor;
-  if (g_globalCursor) (*g_globalCursor) = cursor;
+    extern wxCursor *g_globalCursor;
+    if (g_globalCursor) (*g_globalCursor) = cursor;
 
-  if (cursor.Ok()) {};
-};
+    if (cursor.Ok()) {}
+}