joystick updates from John Fay
[freeglut] / src / freeglut_menu.c
index 464081e..b22722d 100644 (file)
@@ -29,7 +29,7 @@
 #include "config.h"
 #endif
 
-#include "../include/GL/freeglut.h"
+#include <GL/freeglut.h>
 #include "freeglut_internal.h"
 
 /* -- DEFINITIONS ---------------------------------------------------------- */
@@ -56,7 +56,7 @@
  * that that wasn't the original intent...if not, perhaps we need another
  * symbolic constant, FREEGLUT_MENU_ITEM_BORDER, or such.)
  */
-#if TARGET_HOST_WIN32
+#if TARGET_HOST_WIN32 || TARGET_HOST_WINCE
 #define  FREEGLUT_MENU_FONT    GLUT_BITMAP_8_BY_13
 #else
 #define  FREEGLUT_MENU_FONT    GLUT_BITMAP_HELVETICA_18
@@ -75,7 +75,7 @@
  * too.  These variables should be stuffed into global state and initialized
  * via the glutInit*() system.
  */
-#if TARGET_HOST_WIN32
+#if TARGET_HOST_WIN32 || TARGET_HOST_WINCE
 static float menu_pen_fore  [4] = {0.0f,  0.0f,  0.0f,  1.0f};
 static float menu_pen_back  [4] = {0.85f, 0.85f, 0.85f, 1.0f};
 static float menu_pen_hfore [4] = {1.0f,  1.0f,  1.0f,  1.0f};
@@ -117,17 +117,15 @@ static GLboolean fghCheckMenuStatus( SFG_Window* window, SFG_Menu* menu )
     SFG_MenuEntry* menuEntry;
     int x, y;
 
-    /*
-     * First of all check any of the active sub menus...
-     */
+    /* First of all check any of the active sub menus... */
     for( menuEntry = (SFG_MenuEntry *)menu->Entries.First;
          menuEntry;
          menuEntry = (SFG_MenuEntry *)menuEntry->Node.Next )
     {
-        if( menuEntry->SubMenu && ( menuEntry->IsActive == TRUE ) )
+        if( menuEntry->SubMenu && menuEntry->IsActive )
         {
             /*
-             * OK, have the sub-menu checked, too. If it returns TRUE, it
+             * OK, have the sub-menu checked, too. If it returns GL_TRUE, it
              * will mean that it caught the mouse cursor and we do not need
              * to regenerate the activity list, and so our parents do...
              */
@@ -138,28 +136,24 @@ static GLboolean fghCheckMenuStatus( SFG_Window* window, SFG_Menu* menu )
              * Reactivate the submenu as the checkMenuStatus may have turned
              * it off if the mouse is in its parent menu entry.
              */
-            menuEntry->SubMenu->IsActive = TRUE;
-            if ( return_status == TRUE )
-                return TRUE;
+            menuEntry->SubMenu->IsActive = GL_TRUE;
+            if ( return_status )
+                return GL_TRUE;
         }
     }
 
-    /*
-     * That much about our sub menus, let's get to checking the current menu:
-     */
+    /* That much about our sub menus, let's get to checking the current menu: */
     x = window->State.MouseX;
     y = window->State.MouseY;
 
     for( menuEntry = (SFG_MenuEntry *)menu->Entries.First;
          menuEntry;
          menuEntry = (SFG_MenuEntry *)menuEntry->Node.Next )
-        menuEntry->IsActive = FALSE;
+        menuEntry->IsActive = GL_FALSE;
 
-    menu->IsActive = FALSE;
+    menu->IsActive = GL_FALSE;
 
-    /*
-     * Check if the mouse cursor is contained within the current menu box
-     */
+    /* Check if the mouse cursor is contained within the current menu box */
     if( ( x >= FREEGLUT_MENU_BORDER ) &&
         ( x < menu->Width  - FREEGLUT_MENU_BORDER ) &&
         ( y >= FREEGLUT_MENU_BORDER ) &&
@@ -168,13 +162,11 @@ static GLboolean fghCheckMenuStatus( SFG_Window* window, SFG_Menu* menu )
     {
         int menuID = ( y - FREEGLUT_MENU_BORDER ) / FREEGLUT_MENU_HEIGHT;
 
-        /*
-         * The mouse cursor is somewhere over our box, check it out.
-         */
+        /* The mouse cursor is somewhere over our box, check it out. */
         menuEntry = fghFindMenuEntry( menu, menuID + 1 );
         assert( menuEntry );
 
-        menuEntry->IsActive = TRUE;
+        menuEntry->IsActive = GL_TRUE;
         menuEntry->Ordinal = menuID;
 
         /*
@@ -183,13 +175,11 @@ static GLboolean fghCheckMenuStatus( SFG_Window* window, SFG_Menu* menu )
          * was a submenu then deactivate it.
          */
         if( menu->ActiveEntry && ( menuEntry != menu->ActiveEntry ) )
-        {
             if( menu->ActiveEntry->SubMenu )
                 fgDeactivateSubMenu( menu->ActiveEntry );
-        }
 
         menu->ActiveEntry = menuEntry;
-        menu->IsActive = TRUE;
+        menu->IsActive = GL_TRUE;
 
         /*
          * OKi, we have marked that entry as active, but it would be also
@@ -202,14 +192,10 @@ static GLboolean fghCheckMenuStatus( SFG_Window* window, SFG_Menu* menu )
             {
                 SFG_Window *current_window = fgStructure.Window;
 
-                /*
-                 * Set up the initial menu position now...
-                 */
-                menuEntry->SubMenu->IsActive = TRUE;
+                /* Set up the initial menu position now... */
+                menuEntry->SubMenu->IsActive = GL_TRUE;
 
-                /*
-                 * Set up the initial submenu position now:
-                 */
+                /* Set up the initial submenu position now: */
                 menuEntry->SubMenu->X = menu->X + menu->Width;
                 menuEntry->SubMenu->Y = menu->Y +
                     menuEntry->Ordinal * FREEGLUT_MENU_HEIGHT;
@@ -238,22 +224,16 @@ static GLboolean fghCheckMenuStatus( SFG_Window* window, SFG_Menu* menu )
 
             fghCheckMenuStatus( window, menuEntry->SubMenu );
 
-            /*
-             * Activate it because its parent entry is active
-             */
-            menuEntry->SubMenu->IsActive = TRUE;
+            /* Activate it because its parent entry is active */
+            menuEntry->SubMenu->IsActive = GL_TRUE;
         }
 
-        /*
-         * Report back that we have caught the menu cursor
-         */
-        return TRUE;
+        /* Report back that we have caught the menu cursor */
+        return GL_TRUE;
     }
 
-    /*
-     * Looks like the menu cursor is somewhere else...
-     */
-    return FALSE;
+    /* Looks like the menu cursor is somewhere else... */
+    return GL_FALSE;
 }
 
 /*
@@ -272,44 +252,40 @@ static void fghDisplayMenuBox( SFG_Menu* menu )
     /* a non-black dark version of the below. */
     glColor4f( 1.0f, 1.0f, 1.0f, 1.0f );
     glBegin( GL_QUAD_STRIP );
-        glVertex2i( menu->Width       , 0                  );
-        glVertex2i( menu->Width-border,              border);
-        glVertex2i( 0                 , 0                  );
-        glVertex2i(             border,              border);
-        glVertex2i( 0                 , menu->Height       );
-        glVertex2i(             border, menu->Height-border);
+        glVertex2i( menu->Width         , 0                    );
+        glVertex2i( menu->Width - border,                border);
+        glVertex2i( 0                   , 0                    );
+        glVertex2i(               border,                border);
+        glVertex2i( 0                   , menu->Height         );
+        glVertex2i(               border, menu->Height - border);
     glEnd( );
 
     /* a non-black dark version of the below. */
     glColor4f( 0.5f, 0.5f, 0.5f, 1.0f );
     glBegin( GL_QUAD_STRIP );
-        glVertex2i( 0                 , menu->Height       );
-        glVertex2i(             border, menu->Height-border);
-        glVertex2i( menu->Width       , menu->Height       );
-        glVertex2i( menu->Width-border, menu->Height-border);
-        glVertex2i( menu->Width       , 0                  );
-        glVertex2i( menu->Width-border,              border);
+        glVertex2i( 0                   , menu->Height         );
+        glVertex2i(               border, menu->Height - border);
+        glVertex2i( menu->Width         , menu->Height         );
+        glVertex2i( menu->Width - border, menu->Height - border);
+        glVertex2i( menu->Width         , 0                    );
+        glVertex2i( menu->Width - border,                border);
     glEnd( );
 
-    glColor4fv( menu_pen_back ) ;
+    glColor4fv( menu_pen_back );
     glBegin( GL_QUADS );
-        glVertex2i(             border,              border);
-        glVertex2i( menu->Width-border,              border);
-        glVertex2i( menu->Width-border, menu->Height-border);
-        glVertex2i(             border, menu->Height-border);
+        glVertex2i(               border,                border);
+        glVertex2i( menu->Width - border,                border);
+        glVertex2i( menu->Width - border, menu->Height - border);
+        glVertex2i(               border, menu->Height - border);
     glEnd( );
 
-    /*
-     * Check if any of the submenus is currently active...
-     */
+    /* Check if any of the submenus is currently active... */
     for( menuEntry = (SFG_MenuEntry *)menu->Entries.First;
          menuEntry;
          menuEntry = (SFG_MenuEntry *)menuEntry->Node.Next )
     {
-        /*
-         * Has the menu been marked as active, maybe?
-         */
-        if( menuEntry->IsActive == TRUE )
+        /* Has the menu been marked as active, maybe? */
+        if( menuEntry->IsActive )
         {
             /*
              * That's truly right, and we need to have it highlighted.
@@ -318,98 +294,73 @@ static void fghDisplayMenuBox( SFG_Menu* menu )
              */
             int menuID = menuEntry->Ordinal;
 
-            /*
-             * So have the highlight drawn...
-             */
+            /* So have the highlight drawn... */
             glColor4fv( menu_pen_hback );
             glBegin( GL_QUADS );
-                glVertex2i( FREEGLUT_MENU_BORDER,
-                            (menuID + 0)*FREEGLUT_MENU_HEIGHT +
-                            FREEGLUT_MENU_BORDER );
-                glVertex2i( menu->Width-FREEGLUT_MENU_BORDER,
-                            (menuID + 0)*FREEGLUT_MENU_HEIGHT +
-                            FREEGLUT_MENU_BORDER );
-                glVertex2i( menu->Width-FREEGLUT_MENU_BORDER,
-                            (menuID + 1)*FREEGLUT_MENU_HEIGHT +
-                            FREEGLUT_MENU_BORDER );
-                glVertex2i( FREEGLUT_MENU_BORDER,
-                            (menuID + 1)*FREEGLUT_MENU_HEIGHT +
-                            FREEGLUT_MENU_BORDER );
+                glVertex2i( border,
+                            (menuID + 0)*FREEGLUT_MENU_HEIGHT + border );
+                glVertex2i( menu->Width - border,
+                            (menuID + 0)*FREEGLUT_MENU_HEIGHT + border );
+                glVertex2i( menu->Width - border,
+                            (menuID + 1)*FREEGLUT_MENU_HEIGHT + border );
+                glVertex2i( border,
+                            (menuID + 1)*FREEGLUT_MENU_HEIGHT + border );
             glEnd( );
         }
     }
 
-    /*
-     * Print the menu entries now...
-     */
+    /* Print the menu entries now... */
 
     glColor4fv( menu_pen_fore );
 
-    for( menuEntry = (SFG_MenuEntry *)menu->Entries.First, i=0;
+    for( menuEntry = (SFG_MenuEntry *)menu->Entries.First, i = 0;
          menuEntry;
          menuEntry = (SFG_MenuEntry *)menuEntry->Node.Next, ++i )
     {
-        /*
-         * If the menu entry is active, set the color to white
-         */
+        /* If the menu entry is active, set the color to white */
         if( menuEntry->IsActive )
             glColor4fv( menu_pen_hfore );
 
-        /*
-         * Move the raster into position...
-         */
+        /* Move the raster into position... */
         /* Try to center the text - JCJ 31 July 2003*/
         glRasterPos2i(
-            2 * FREEGLUT_MENU_BORDER,
-            (i + 1)*FREEGLUT_MENU_HEIGHT-(int)( FREEGLUT_MENU_HEIGHT*0.3 -
-                                                FREEGLUT_MENU_BORDER )
+            2 * border,
+            ( i + 1 )*FREEGLUT_MENU_HEIGHT -
+            ( int )( FREEGLUT_MENU_HEIGHT*0.3 - border )
         );
 
-        /*
-         * Have the label drawn, character after character:
-         */
-        glutBitmapString( FREEGLUT_MENU_FONT, menuEntry->Text);
+        /* Have the label drawn, character after character: */
+        glutBitmapString( FREEGLUT_MENU_FONT,
+                          (unsigned char *)menuEntry->Text);
 
-        /*
-         * If it's a submenu, draw a right arrow
-         */
+        /* If it's a submenu, draw a right arrow */
         if( menuEntry->SubMenu )
         {
             int width = glutBitmapWidth( FREEGLUT_MENU_FONT, '_' );
             int x_base = menu->Width - 2 - width;
-            int y_base = i*FREEGLUT_MENU_HEIGHT + FREEGLUT_MENU_BORDER;
+            int y_base = i*FREEGLUT_MENU_HEIGHT + border;
             glBegin( GL_TRIANGLES );
-                glVertex2i( x_base, y_base + 2*FREEGLUT_MENU_BORDER);
+                glVertex2i( x_base, y_base + 2*border);
                 glVertex2i( menu->Width - 2, y_base +
-                            ( FREEGLUT_MENU_HEIGHT +
-                              FREEGLUT_MENU_BORDER) / 2 );
-                glVertex2i( x_base, y_base + FREEGLUT_MENU_HEIGHT -
-                            FREEGLUT_MENU_BORDER);
+                            ( FREEGLUT_MENU_HEIGHT + border) / 2 );
+                glVertex2i( x_base, y_base + FREEGLUT_MENU_HEIGHT - border );
             glEnd( );
         }
 
-        /*
-         * If the menu entry is active, reset the color
-         */
+        /* If the menu entry is active, reset the color */
         if( menuEntry->IsActive )
             glColor4fv( menu_pen_fore );
     }
 
-    /*
-     * Now we are ready to check if any of our children needs to be redrawn:
-     */
-    for( menuEntry = (SFG_MenuEntry *)menu->Entries.First;
+    /* Now we are ready to check if any of our children needs to be redrawn: */
+    for( menuEntry = ( SFG_MenuEntry * )menu->Entries.First;
          menuEntry;
-         menuEntry = (SFG_MenuEntry *)menuEntry->Node.Next )
+         menuEntry = ( SFG_MenuEntry * )menuEntry->Node.Next )
     {
-        /*
-         * Is that an active sub menu by any case?
-         */
-        if( menuEntry->SubMenu && ( menuEntry->IsActive == TRUE ) )
+        /* Is that an active sub menu by any case? */
+        if( menuEntry->SubMenu && menuEntry->IsActive )
         {
-            /*
-             * Yeah, indeed. Have it redrawn now:
-             */
+            /* Yeah, indeed. Have it redrawn now: */
             fgSetWindow( menuEntry->SubMenu->Window );
             fghDisplayMenuBox( menuEntry->SubMenu );
             fgSetWindow( menu->Window );
@@ -421,17 +372,17 @@ static void fghDisplayMenuBox( SFG_Menu* menu )
  * Private static function to set the parent window of a submenu and all
  * of its submenus
  */
-static void fghSetSubmenuParentWindow ( SFG_Window *window, SFG_Menu *menu )
+static void fghSetSubmenuParentWindow( SFG_Window *window, SFG_Menu *menu )
 {
     SFG_MenuEntry *menuEntry;
 
     menu->ParentWindow = window;
 
-    for( menuEntry = (SFG_MenuEntry *)menu->Entries.First;
+    for( menuEntry = ( SFG_MenuEntry * )menu->Entries.First;
          menuEntry;
-         menuEntry = (SFG_MenuEntry *)menuEntry->Node.Next )
-        if ( menuEntry->SubMenu )
-            fghSetSubmenuParentWindow ( window, menuEntry->SubMenu ) ;
+         menuEntry = ( SFG_MenuEntry * )menuEntry->Node.Next )
+        if( menuEntry->SubMenu )
+            fghSetSubmenuParentWindow( window, menuEntry->SubMenu );
 }
 
 
@@ -443,26 +394,14 @@ void fgDisplayMenu( void )
     SFG_Window* window = fgStructure.Window;
     SFG_Menu* menu = NULL;
 
-    /*
-     * Make sure there is a current window available
-     */
     freeglut_assert_window;
 
-    /*
-     * Check if there is an active menu attached to this window...
-     */
+    /* Check if there is an active menu attached to this window... */
     menu = window->ActiveMenu;
-
-    /*
-     * Did we find an active menu?
-     */
     freeglut_return_if_fail( menu );
 
-    fgSetWindow ( menu->Window );
+    fgSetWindow( menu->Window );
 
-    /*
-     * Prepare the OpenGL state to do the rendering first:
-     */
     glPushAttrib( GL_DEPTH_BUFFER_BIT | GL_TEXTURE_BIT | GL_LIGHTING_BIT |
                   GL_POLYGON_BIT );
 
@@ -471,38 +410,22 @@ void fgDisplayMenu( void )
     glDisable( GL_LIGHTING   );
     glDisable( GL_CULL_FACE  );
 
-    /*
-     * We'll use an orthogonal projection matrix to draw the menu:
-     */
     glMatrixMode( GL_PROJECTION );
-    glPushMatrix();
-    glLoadIdentity();
+    glPushMatrix( );
+    glLoadIdentity( );
     glOrtho(
          0, glutGet( GLUT_WINDOW_WIDTH  ),
          glutGet( GLUT_WINDOW_HEIGHT ), 0,
         -1, 1
     );
 
-    /*
-     * Model-view matix gets reset to identity:
-     */
     glMatrixMode( GL_MODELVIEW );
-    glPushMatrix();
-    glLoadIdentity();
+    glPushMatrix( );
+    glLoadIdentity( );
 
-    /*
-     * First of all, have the exact menu status check:
-     */
     fghCheckMenuStatus( window, menu );
-
-    /*
-     * The status has been updated and we're ready to have the menu drawn now:
-     */
     fghDisplayMenuBox( menu );
 
-    /*
-     * Restore the old OpenGL settings now
-     */
     glPopAttrib( );
 
     glMatrixMode( GL_PROJECTION );
@@ -512,9 +435,6 @@ void fgDisplayMenu( void )
 
     glutSwapBuffers( );
 
-    /*
-     * Restore the current window
-     */
     fgSetWindow ( window );
 }
 
@@ -523,22 +443,15 @@ void fgDisplayMenu( void )
  */
 void fgActivateMenu( SFG_Window* window, int button )
 {
-    /*
-     * We'll be referencing this menu a lot, so remember its address:
-     */
+    /* We'll be referencing this menu a lot, so remember its address: */
     SFG_Menu* menu = window->Menu[ button ];
 
-    /*
-     * Mark the menu as active, so that it gets displayed:
-     */
+    /* Mark the menu as active, so that it gets displayed: */
     window->ActiveMenu = menu;
-    menu->IsActive = TRUE;
+    menu->IsActive = GL_TRUE;
     fgState.ActiveMenus++;
 
-    /*
-     * Set up the initial menu position now:
-     */
-
+    /* Set up the initial menu position now: */
     menu->X = window->State.MouseX + glutGet( GLUT_WINDOW_X );
     menu->Y = window->State.MouseY + glutGet( GLUT_WINDOW_Y );
 
@@ -583,41 +496,18 @@ void fgExecuteMenuCallback( SFG_Menu* menu )
 {
     SFG_MenuEntry *menuEntry;
 
-    /*
-     * First of all check any of the active sub menus...
-     */
+    /* First of all check any of the active sub menus... */
     for( menuEntry = (SFG_MenuEntry *)menu->Entries.First;
          menuEntry;
          menuEntry = (SFG_MenuEntry *)menuEntry->Node.Next)
     {
-        /*
-         * Is this menu entry active?
-         */
-        if( menuEntry->IsActive == TRUE )
+        if( menuEntry->IsActive )
         {
-            /*
-             * If there is not a sub menu, execute the menu callback and
-             * return...
-             */
-            if( !( menuEntry->SubMenu ) )
-            {
-                /*
-                 * ...certainly given that there is one...
-                 */
+            if( menuEntry->SubMenu )
+                fgExecuteMenuCallback( menuEntry->SubMenu );
+            else
                 if( menu->Callback )
                     menu->Callback( menuEntry->ID );
-
-                return;
-            }
-
-            /*
-             * Otherwise recurse into the submenu.
-             */
-            fgExecuteMenuCallback( menuEntry->SubMenu );
-
-            /*
-             * There is little sense in dwelling the search on
-             */
             return;
         }
     }
@@ -630,47 +520,35 @@ void fgDeactivateMenu( SFG_Window *window )
 {
     SFG_Window *current_window = fgStructure.Window;
 
-    /*
-     * Check if there is an active menu attached to this window...
-     */
+    /* Check if there is an active menu attached to this window... */
     SFG_Menu* menu = window->ActiveMenu;
     SFG_MenuEntry *menuEntry;
 
-    /*
-     * Did we find an active window?
-     */
+    /* Did we find an active window? */
     freeglut_return_if_fail( menu );
 
-    /*
-     * Hide the present menu's window
-     */
+    /* Hide the present menu's window */
     fgSetWindow( menu->Window );
     glutHideWindow( );
 
-    /*
-     * Forget about having that menu active anymore, now:
-     */
+    /* Forget about having that menu active anymore, now: */
     menu->Window->ActiveMenu = NULL;
     menu->ParentWindow->ActiveMenu = NULL;
-    menu->IsActive = FALSE;
+    menu->IsActive = GL_FALSE;
 
     fgState.ActiveMenus--;
 
-    /*
-     * Hide all submenu windows, and the root menu's window.
-     */
-    for ( menuEntry = (SFG_MenuEntry *)menu->Entries.First;
+    /* Hide all submenu windows, and the root menu's window. */
+    for ( menuEntry = ( SFG_MenuEntry * )menu->Entries.First;
           menuEntry;
-          menuEntry = (SFG_MenuEntry *)menuEntry->Node.Next )
+          menuEntry = ( SFG_MenuEntry * )menuEntry->Node.Next )
     {
-        /*
-         * Is that an active submenu by any case?
-         */
+        /* Is that an active submenu by any case? */
         if( menuEntry->SubMenu )
             fgDeactivateSubMenu( menuEntry );
     }
 
-    fgSetWindow ( current_window ) ;
+    fgSetWindow( current_window );
 }
 
 /*
@@ -680,28 +558,20 @@ void fgDeactivateSubMenu( SFG_MenuEntry *menuEntry )
 {
     SFG_Window *current_window = fgStructure.Window;
     SFG_MenuEntry *subMenuIter;
-    /*
-     * Hide the present menu's window
-     */
+    /* Hide the present menu's window */
     fgSetWindow( menuEntry->SubMenu->Window );
     glutHideWindow( );
 
-    /*
-     * Forget about having that menu active anymore, now:
-     */
+    /* Forget about having that menu active anymore, now: */
     menuEntry->SubMenu->Window->ActiveMenu = NULL;
-    menuEntry->SubMenu->IsActive = FALSE;
+    menuEntry->SubMenu->IsActive = GL_FALSE;
 
-    /*
-     * Hide all submenu windows, and the root menu's window.
-     */
+    /* Hide all submenu windows, and the root menu's window. */
     for ( subMenuIter = (SFG_MenuEntry *)menuEntry->SubMenu->Entries.First;
           subMenuIter;
           subMenuIter = (SFG_MenuEntry *)subMenuIter->Node.Next )
     {
-        /*
-         * Is that an active submenu by any case?
-         */
+        /* Is that an active submenu by any case? */
         if( subMenuIter->SubMenu )
             fgDeactivateSubMenu( subMenuIter );
     }
@@ -717,44 +587,39 @@ void fghCalculateMenuBoxSize( void )
     SFG_MenuEntry* menuEntry;
     int width = 0, height = 0;
 
-    /*
-     * Make sure there is a current menu set
-     */
+    /* Make sure there is a current menu set */
     freeglut_assert_ready;
     freeglut_return_if_fail( fgStructure.Menu );
 
-    /*
-     * The menu's box size depends on the menu entries:
-     */
-    for( menuEntry = (SFG_MenuEntry *)fgStructure.Menu->Entries.First;
+    /* The menu's box size depends on the menu entries: */
+    for( menuEntry = ( SFG_MenuEntry * )fgStructure.Menu->Entries.First;
          menuEntry;
-         menuEntry = (SFG_MenuEntry *)menuEntry->Node.Next)
+         menuEntry = ( SFG_MenuEntry * )menuEntry->Node.Next )
     {
-        /*
-         * Update the menu entry's width value
-         */
-        menuEntry->Width = glutBitmapLength( FREEGLUT_MENU_FONT,
-                                             menuEntry->Text );
+        /* Update the menu entry's width value */
+        menuEntry->Width = glutBitmapLength(
+            FREEGLUT_MENU_FONT,
+            (unsigned char *)menuEntry->Text
+        );
 
         /*
          * If the entry is a submenu, then it needs to be wider to
          * accomodate the arrow. JCJ 31 July 2003
          */
         if (menuEntry->SubMenu )
-            menuEntry->Width += glutBitmapLength( FREEGLUT_MENU_FONT, "_" );
+            menuEntry->Width += glutBitmapLength(
+                FREEGLUT_MENU_FONT,
+                (unsigned char *)"_"
+            );
 
-        /*
-         * Check if it's the biggest we've found
-         */
+        /* Check if it's the biggest we've found */
         if( menuEntry->Width > width )
             width = menuEntry->Width;
 
         height += FREEGLUT_MENU_HEIGHT;
     }
 
-    /*
-     * Store the menu's box size now:
-     */
+    /* Store the menu's box size now: */
     fgStructure.Menu->Height = height + 2 * FREEGLUT_MENU_BORDER;
     fgStructure.Menu->Width  = width  + 4 * FREEGLUT_MENU_BORDER;
 }
@@ -765,11 +630,9 @@ void fghCalculateMenuBoxSize( void )
 /*
  * Creates a new menu object, adding it to the freeglut structure
  */
-int FGAPIENTRY glutCreateMenu( void (* callback)( int ) )
+int FGAPIENTRY glutCreateMenu( void(* callback)( int ) )
 {
-    /*
-     * The menu object creation code resides in freeglut_structure.c
-     */
+    /* The menu object creation code resides in freeglut_structure.c */
     return fgCreateMenu( callback )->ID;
 }
 
@@ -783,9 +646,7 @@ void FGAPIENTRY glutDestroyMenu( int menuID )
     freeglut_assert_ready;
     freeglut_return_if_fail( menu );
 
-    /*
-     * The menu object destruction code resides in freeglut_structure.c
-     */
+    /* The menu object destruction code resides in freeglut_structure.c */
     fgDestroyMenu( menu );
 }
 
@@ -829,9 +690,7 @@ void FGAPIENTRY glutAddMenuEntry( const char* label, int value )
     menuEntry->Text = strdup( label );
     menuEntry->ID   = value;
 
-    /*
-     * Have the new menu entry attached to the current menu
-     */
+    /* Have the new menu entry attached to the current menu */
     fgListAppend( &fgStructure.Menu->Entries, &menuEntry->Node );
 
     fghCalculateMenuBoxSize( );
@@ -840,23 +699,21 @@ void FGAPIENTRY glutAddMenuEntry( const char* label, int value )
 /*
  * Add a sub menu to the bottom of the current menu
  */
-void FGAPIENTRY glutAddSubMenu( const char* label, int subMenuID )
+void FGAPIENTRY glutAddSubMenu( const char *label, int subMenuID )
 {
-    SFG_MenuEntry* menuEntry =
-        (SFG_MenuEntry *)calloc( sizeof(SFG_MenuEntry), 1 );
-    SFG_Menu*      subMenu = fgMenuByID( subMenuID );
+    SFG_MenuEntry *menuEntry =
+        ( SFG_MenuEntry * )calloc( sizeof( SFG_MenuEntry ), 1 );
+    SFG_Menu *subMenu = fgMenuByID( subMenuID );
 
     freeglut_assert_ready;
     freeglut_return_if_fail( fgStructure.Menu );
     freeglut_return_if_fail( subMenu );
 
-    menuEntry->Text = strdup( label );
+    menuEntry->Text    = strdup( label );
     menuEntry->SubMenu = subMenu;
     menuEntry->ID      = -1;
 
-    /*
-     * Make the submenu's parent window be the menu's parent window
-     */
+    /* Make the submenu's parent window be the menu's parent window */
     fghSetSubmenuParentWindow( fgStructure.Menu->ParentWindow, subMenu );
 
     fgListAppend( &fgStructure.Menu->Entries, &menuEntry->Node );
@@ -873,20 +730,16 @@ void FGAPIENTRY glutChangeToMenuEntry( int item, const char* label, int value )
     freeglut_assert_ready;
     freeglut_return_if_fail( fgStructure.Menu );
 
-    /*
-     * Get n-th menu entry in the current menu, starting from one:
-     */
+    /* Get n-th menu entry in the current menu, starting from one: */
     menuEntry = fghFindMenuEntry( fgStructure.Menu, item );
 
     freeglut_return_if_fail( menuEntry );
 
-    /*
-     * We want it to become a normal menu entry, so:
-     */
+    /* We want it to become a normal menu entry, so: */
     if( menuEntry->Text )
         free( menuEntry->Text );
 
-    menuEntry->Text = strdup( label );
+    menuEntry->Text    = strdup( label );
     menuEntry->ID      = value;
     menuEntry->SubMenu = NULL;
     fghCalculateMenuBoxSize( );
@@ -895,7 +748,8 @@ void FGAPIENTRY glutChangeToMenuEntry( int item, const char* label, int value )
 /*
  * Changes the specified menu item in the current menu into a sub-menu trigger.
  */
-void FGAPIENTRY glutChangeToSubMenu( int item, const char* label, int subMenuID )
+void FGAPIENTRY glutChangeToSubMenu( int item, const char* label,
+                                     int subMenuID )
 {
     SFG_Menu*      subMenu = fgMenuByID( subMenuID );
     SFG_MenuEntry* menuEntry = NULL;
@@ -904,20 +758,16 @@ void FGAPIENTRY glutChangeToSubMenu( int item, const char* label, int subMenuID
     freeglut_return_if_fail( fgStructure.Menu );
     freeglut_return_if_fail( subMenu );
 
-    /*
-     * Get n-th menu entry in the current menu, starting from one:
-     */
+    /* Get n-th menu entry in the current menu, starting from one: */
     menuEntry = fghFindMenuEntry( fgStructure.Menu, item );
 
     freeglut_return_if_fail( menuEntry );
 
-    /*
-     * We want it to become a sub menu entry, so:
-     */
+    /* We want it to become a sub menu entry, so: */
     if( menuEntry->Text )
         free( menuEntry->Text );
 
-    menuEntry->Text = strdup( label );
+    menuEntry->Text    = strdup( label );
     menuEntry->SubMenu = subMenu;
     menuEntry->ID      = -1;
     fghCalculateMenuBoxSize( );
@@ -933,9 +783,7 @@ void FGAPIENTRY glutRemoveMenuItem( int item )
     freeglut_assert_ready;
     freeglut_return_if_fail( fgStructure.Menu );
 
-    /*
-     * Get n-th menu entry in the current menu, starting from one:
-     */
+    /* Get n-th menu entry in the current menu, starting from one: */
     menuEntry = fghFindMenuEntry( fgStructure.Menu, item );
 
     freeglut_return_if_fail( menuEntry );
@@ -963,10 +811,8 @@ void FGAPIENTRY glutAttachMenu( int button )
 
     fgStructure.Window->Menu[ button ] = fgStructure.Menu;
 
-    /*
-     * Make the parent window of the menu (and all submenus) the current window
-     */
-    fghSetSubmenuParentWindow ( fgStructure.Window, fgStructure.Menu ) ;
+    /* Make the parent window of the menu (and all submenus) the current window */
+    fghSetSubmenuParentWindow( fgStructure.Window, fgStructure.Menu );
 }
 
 /*