Noted by XXX comments that the XFlush() calls probably shouldn't be there.
[freeglut] / src / freeglut_cursor.c
index 4527fa5..9ba6f12 100644 (file)
 #include "freeglut_internal.h"
 
 #if TARGET_HOST_UNIX_X11
-    #include <X11/cursorfont.h>
+  #include <X11/cursorfont.h>
 #endif
 
 /*
  * TODO BEFORE THE STABLE RELEASE:
- *
- *  fgDisplayCursor()   -- this waits for better times
- *                         XXX Just delete fgDisplayCursor?
  *  glutSetCursor()     -- Win32 mappings are incomplete
  *                         X mappings are nearly right.
  *
 
 /* -- INTERNAL FUNCTIONS --------------------------------------------------- */
 
-/*
- * Display the mouse cursor using OpenGL calls
- */
-void fgDisplayCursor( void )
-{
-}
-
-
 /* -- INTERFACE FUNCTIONS -------------------------------------------------- */
 
 /*
@@ -68,28 +57,29 @@ void fgDisplayCursor( void )
  */
 void FGAPIENTRY glutSetCursor( int cursorID )
 {
-    freeglut_assert_ready;
-    freeglut_assert_window;
+  freeglut_assert_ready;
+  freeglut_assert_window;
 
 #if TARGET_HOST_UNIX_X11
-    /*
-     * Open issues:
-     * (a) GLUT_CURSOR_NONE doesn't do what it should.  We can probably
-     *     build an empty pixmap for it, though, quite painlessly.
-     * (b) Are we allocating resources, or causing X to do so?
-     *     If yes, we should arrange to deallocate!
-     * (c) No error checking.  Is that a problem?
-     * (d) FULL_CROSSHAIR demotes to plain CROSSHAIR.  Old GLUT allows
-     *     for this, but if there is a system that easily supports a full-
-     *     window (or full-screen) crosshair, we might consider it.
-     * (e) Out-of-range cursor-types are ignored.  Should we abort?
-     *     Print a warning message?
-     */
+  /*
+   * Open issues:
+   * (a) GLUT_CURSOR_NONE doesn't do what it should.  We can probably
+   *     build an empty pixmap for it, though, quite painlessly.
+   * (b) Are we allocating resources, or causing X to do so?
+   *     If yes, we should arrange to deallocate!
+   * (c) No error checking.  Is that a problem?
+   * (d) FULL_CROSSHAIR demotes to plain CROSSHAIR.  Old GLUT allows
+   *     for this, but if there is a system that easily supports a full-
+   *     window (or full-screen) crosshair, we might consider it.
+   * (e) Out-of-range cursor-types are ignored.  Should we abort?
+   *     Print a warning message?
+   */
     {
        Cursor cursor;
 #define MAP_CURSOR(a,b) case a: cursor = XCreateFontCursor( fgDisplay.Display, b ); break;
        if( GLUT_CURSOR_FULL_CROSSHAIR == cursorID )
            cursorID = GLUT_CURSOR_CROSSHAIR;
+       
        switch( cursorID )
         {
            MAP_CURSOR( GLUT_CURSOR_RIGHT_ARROW, XC_right_ptr);
@@ -118,7 +108,7 @@ void FGAPIENTRY glutSetCursor( int cursorID )
        default:
            return;
        }
-       
+
        if( GLUT_CURSOR_INHERIT == cursorID )
            XUndefineCursor( fgDisplay.Display, fgStructure.Window->Window.Handle );
        else
@@ -127,10 +117,10 @@ void FGAPIENTRY glutSetCursor( int cursorID )
 
 #elif TARGET_HOST_WIN32
 
-       /*
-        * This is a temporary solution only...
-        */
-       /* Set the cursor AND change it for this window class. */
+  /*
+   * This is a temporary solution only...
+   */
+  /* Set the cursor AND change it for this window class. */
 #      define MAP_CURSOR(a,b) case a: SetCursor( LoadCursor( NULL, b ) ); \
         SetClassLong(fgStructure.Window->Window.Handle,GCL_HCURSOR,(LONG)LoadCursor(NULL,b)); \
         break;
@@ -141,25 +131,25 @@ void FGAPIENTRY glutSetCursor( int cursorID )
 
        switch( cursorID )
        {
-               MAP_CURSOR( GLUT_CURSOR_RIGHT_ARROW, IDC_ARROW     );
-               MAP_CURSOR( GLUT_CURSOR_LEFT_ARROW,  IDC_ARROW     );
-               MAP_CURSOR( GLUT_CURSOR_INFO,        IDC_HELP      );
-               MAP_CURSOR( GLUT_CURSOR_DESTROY,     IDC_CROSS     );
-               MAP_CURSOR( GLUT_CURSOR_HELP,        IDC_HELP      );
-               MAP_CURSOR( GLUT_CURSOR_CYCLE,       IDC_SIZEALL   );
-               MAP_CURSOR( GLUT_CURSOR_SPRAY,       IDC_CROSS     );
-               MAP_CURSOR( GLUT_CURSOR_WAIT,            IDC_WAIT      );
-               MAP_CURSOR( GLUT_CURSOR_TEXT,        IDC_UPARROW   );
-               MAP_CURSOR( GLUT_CURSOR_CROSSHAIR,   IDC_CROSS     );
-               /* MAP_CURSOR( GLUT_CURSOR_NONE,        IDC_NO             ); */
-               ZAP_CURSOR( GLUT_CURSOR_NONE,        NULL          );
-
-               default:
-               MAP_CURSOR( GLUT_CURSOR_UP_DOWN,     IDC_ARROW     );
+           MAP_CURSOR( GLUT_CURSOR_RIGHT_ARROW, IDC_ARROW     );
+           MAP_CURSOR( GLUT_CURSOR_LEFT_ARROW,  IDC_ARROW     );
+           MAP_CURSOR( GLUT_CURSOR_INFO,        IDC_HELP      );
+           MAP_CURSOR( GLUT_CURSOR_DESTROY,     IDC_CROSS     );
+           MAP_CURSOR( GLUT_CURSOR_HELP,        IDC_HELP          );
+           MAP_CURSOR( GLUT_CURSOR_CYCLE,       IDC_SIZEALL   );
+           MAP_CURSOR( GLUT_CURSOR_SPRAY,       IDC_CROSS     );
+           MAP_CURSOR( GLUT_CURSOR_WAIT,                IDC_WAIT      );
+           MAP_CURSOR( GLUT_CURSOR_TEXT,        IDC_UPARROW   );
+           MAP_CURSOR( GLUT_CURSOR_CROSSHAIR,   IDC_CROSS     );
+           /* MAP_CURSOR( GLUT_CURSOR_NONE,        IDC_NO                 ); */
+           ZAP_CURSOR( GLUT_CURSOR_NONE,        NULL      );
+           
+       default:
+           MAP_CURSOR( GLUT_CURSOR_UP_DOWN,     IDC_ARROW     );
        }
 #endif
 
-    fgStructure.Window->State.Cursor = cursorID;
+  fgStructure.Window->State.Cursor = cursorID;
 }
 
 /*
@@ -167,33 +157,32 @@ void FGAPIENTRY glutSetCursor( int cursorID )
  */
 void FGAPIENTRY glutWarpPointer( int x, int y )
 {
-    freeglut_assert_ready;
-    freeglut_assert_window;
+  freeglut_assert_ready;
+  freeglut_assert_window;
 
 #if TARGET_HOST_UNIX_X11
 
-    XWarpPointer(
+  XWarpPointer(
         fgDisplay.Display,
         None,
         fgStructure.Window->Window.Handle,
         0, 0, 0, 0,
         x, y
-    );
-    XFlush( fgDisplay.Display );
+  );
+  XFlush( fgDisplay.Display );
 
 #elif TARGET_HOST_WIN32
 
-    {
-        POINT coords = { x, y };
-        /*
-         * ClientToScreen() translates {coords} for us.
-        */
-        ClientToScreen( fgStructure.Window->Window.Handle, &coords );
-        SetCursorPos( coords.x, coords.y );
-    }
+  {
+    POINT coords = { x, y };
+    /*
+     * ClientToScreen() translates {coords} for us.
+     */
+    ClientToScreen( fgStructure.Window->Window.Handle, &coords );
+    SetCursorPos( coords.x, coords.y );
+  }
 
 #endif
-
 }
 
 /*** END OF FILE ***/