Moving some Windows-specific "freeglut_init.c" helper functions into their own file
authorJohn F. Fay <johnffay@nettally.com>
Fri, 20 Jan 2012 04:10:00 +0000 (04:10 +0000)
committerJohn F. Fay <johnffay@nettally.com>
Fri, 20 Jan 2012 04:10:00 +0000 (04:10 +0000)
git-svn-id: svn+ssh://svn.code.sf.net/p/freeglut/code/trunk/freeglut/freeglut@987 7f0cb862-5218-0410-a997-914c9d46530a

src/Common/freeglut_init.c
src/mswin/freeglut_init_mswin.c

index e295a6a..1640482 100644 (file)
@@ -100,6 +100,10 @@ SFG_State fgState = { { -1, -1, GL_FALSE },  /* Position */
 \r
 /* -- PRIVATE FUNCTIONS ---------------------------------------------------- */\r
 \r
+extern void fghInitialize( const char* displayName );\r
+extern void fghDeinitialiseInputDevices ( void );\r
+extern void fghCloseDisplay ( void );\r
+\r
 #if TARGET_HOST_POSIX_X11\r
 \r
 /* Return the atom associated with "name". */\r
@@ -238,12 +242,12 @@ int fgHintPresent(Window window, Atom property, Atom hint)
 #endif /*  TARGET_HOST_POSIX_X11  */\r
 \r
 \r
+#if TARGET_HOST_POSIX_X11\r
 /*\r
  * A call to this function should initialize all the display stuff...\r
  */\r
 static void fghInitialize( const char* displayName )\r
 {\r
-#if TARGET_HOST_POSIX_X11\r
     fgDisplay.Display = XOpenDisplay( displayName );\r
 \r
     if( fgDisplay.Display == NULL )\r
@@ -307,102 +311,48 @@ static void fghInitialize( const char* displayName )
       }\r
     }\r
 \r
-#elif TARGET_HOST_MS_WINDOWS\r
-\r
-    WNDCLASS wc;\r
-    ATOM atom;\r
-\r
-    /* What we need to do is to initialize the fgDisplay global structure here. */\r
-    fgDisplay.Instance = GetModuleHandle( NULL );\r
-    fgDisplay.DisplayName= displayName ? strdup(displayName) : 0 ;\r
-    atom = GetClassInfo( fgDisplay.Instance, _T("FREEGLUT"), &wc );\r
 \r
-    if( atom == 0 )\r
-    {\r
-        ZeroMemory( &wc, sizeof(WNDCLASS) );\r
-\r
-        /*\r
-         * Each of the windows should have its own device context, and we\r
-         * want redraw events during Vertical and Horizontal Resizes by\r
-         * the user.\r
-         *\r
-         * XXX Old code had "| CS_DBCLCKS" commented out.  Plans for the\r
-         * XXX future?  Dead-end idea?\r
-         */\r
-        wc.lpfnWndProc    = fgWindowProc;\r
-        wc.cbClsExtra     = 0;\r
-        wc.cbWndExtra     = 0;\r
-        wc.hInstance      = fgDisplay.Instance;\r
-        wc.hIcon          = LoadIcon( fgDisplay.Instance, _T("GLUT_ICON") );\r
-\r
-#if defined(_WIN32_WCE)\r
-        wc.style          = CS_HREDRAW | CS_VREDRAW;\r
-#else\r
-        wc.style          = CS_OWNDC | CS_HREDRAW | CS_VREDRAW;\r
-        if (!wc.hIcon)\r
-          wc.hIcon        = LoadIcon( NULL, IDI_WINLOGO );\r
-#endif\r
+    fgState.Initialised = GL_TRUE;\r
 \r
-        wc.hCursor        = LoadCursor( NULL, IDC_ARROW );\r
-        wc.hbrBackground  = NULL;\r
-        wc.lpszMenuName   = NULL;\r
-        wc.lpszClassName  = _T("FREEGLUT");\r
+    atexit(fgDeinitialize);\r
 \r
-        /* Register the window class */\r
-        atom = RegisterClass( &wc );\r
-        FREEGLUT_INTERNAL_ERROR_EXIT ( atom, "Window Class Not Registered", "fghInitialize" );\r
-    }\r
+    /* InputDevice uses GlutTimerFunc(), so fgState.Initialised must be TRUE */\r
+    fgInitialiseInputDevices();\r
+}\r
 \r
-    /* The screen dimensions can be obtained via GetSystemMetrics() calls */\r
-    fgDisplay.ScreenWidth  = GetSystemMetrics( SM_CXSCREEN );\r
-    fgDisplay.ScreenHeight = GetSystemMetrics( SM_CYSCREEN );\r
+#endif\r
 \r
-    {\r
-        HWND desktop = GetDesktopWindow( );\r
-        HDC  context = GetDC( desktop );\r
+#if TARGET_HOST_POSIX_X11\r
+static void fghDeinitialiseInputDevices ( void )\r
+{\r
+    if ( fgState.JoysticksInitialised )\r
+        fgJoystickClose( );\r
 \r
-        fgDisplay.ScreenWidthMM  = GetDeviceCaps( context, HORZSIZE );\r
-        fgDisplay.ScreenHeightMM = GetDeviceCaps( context, VERTSIZE );\r
+    if ( fgState.InputDevsInitialised )\r
+        fgInputDeviceClose( );\r
 \r
-        ReleaseDC( desktop, context );\r
-    }\r
-    /* If we have a DisplayName try to use it for metrics */\r
-    if( fgDisplay.DisplayName )\r
-    {\r
-        HDC context = CreateDC(fgDisplay.DisplayName,0,0,0);\r
-        if( context )\r
-        {\r
-           fgDisplay.ScreenWidth  = GetDeviceCaps( context, HORZRES );\r
-           fgDisplay.ScreenHeight = GetDeviceCaps( context, VERTRES );\r
-           fgDisplay.ScreenWidthMM  = GetDeviceCaps( context, HORZSIZE );\r
-           fgDisplay.ScreenHeightMM = GetDeviceCaps( context, VERTSIZE );\r
-           DeleteDC(context);\r
-        }\r
-        else\r
-           fgWarning("fghInitialize: "\r
-                     "CreateDC failed, Screen size info may be incorrect\n"\r
-          "This is quite likely caused by a bad '-display' parameter");\r
-      \r
-    }\r
-    /* Set the timer granularity to 1 ms */\r
-    timeBeginPeriod ( 1 );\r
+    fgState.JoysticksInitialised = GL_FALSE;\r
+    fgState.InputDevsInitialised = GL_FALSE;\r
+}\r
 \r
-#endif\r
 \r
-    fgState.Initialised = GL_TRUE;\r
+static void fghCloseDisplay ( void )\r
+{\r
+    /*\r
+     * Make sure all X-client data we have created will be destroyed on\r
+     * display closing\r
+     */\r
+    XSetCloseDownMode( fgDisplay.Display, DestroyAll );\r
 \r
-    /* Avoid registering atexit callback on Win32 as it results in an access\r
-     * violation due to calling into a module which has been unloaded.\r
-     * Any cleanup isn't needed on Windows anyway, the OS takes care of it.c\r
-     * see: http://blogs.msdn.com/b/oldnewthing/archive/2012/01/05/10253268.aspx\r
+    /*\r
+     * Close the display connection, destroying all windows we have\r
+     * created so far\r
      */\r
-#if ( TARGET_HOST_MS_WINDOWS == 0 )\r
-    atexit(fgDeinitialize);\r
+    XCloseDisplay( fgDisplay.Display );\r
+}\r
+\r
 #endif\r
 \r
-    /* InputDevice uses GlutTimerFunc(), so fgState.Initialised must be TRUE */\r
-    fgInitialiseInputDevices();\r
-}\r
 \r
 /*\r
  * Perform the freeglut deinitialization...\r
@@ -446,15 +396,7 @@ void fgDeinitialize( void )
         free( timer );\r
     }\r
 \r
-#if !defined(_WIN32_WCE)\r
-    if ( fgState.JoysticksInitialised )\r
-        fgJoystickClose( );\r
-\r
-    if ( fgState.InputDevsInitialised )\r
-        fgInputDeviceClose( );\r
-#endif /* !defined(_WIN32_WCE) */\r
-    fgState.JoysticksInitialised = GL_FALSE;\r
-    fgState.InputDevsInitialised = GL_FALSE;\r
+       fghDeinitialiseInputDevices ();\r
 \r
        fgState.MouseWheelTicks = 0;\r
 \r
@@ -508,31 +450,7 @@ void fgDeinitialize( void )
         fgState.ProgramName = NULL;\r
     }\r
 \r
-#if TARGET_HOST_POSIX_X11\r
-\r
-    /*\r
-     * Make sure all X-client data we have created will be destroyed on\r
-     * display closing\r
-     */\r
-    XSetCloseDownMode( fgDisplay.Display, DestroyAll );\r
-\r
-    /*\r
-     * Close the display connection, destroying all windows we have\r
-     * created so far\r
-     */\r
-    XCloseDisplay( fgDisplay.Display );\r
-\r
-#elif TARGET_HOST_MS_WINDOWS\r
-    if( fgDisplay.DisplayName )\r
-    {\r
-        free( fgDisplay.DisplayName );\r
-        fgDisplay.DisplayName = NULL;\r
-    }\r
-\r
-    /* Reset the timer granularity */\r
-    timeEndPeriod ( 1 );\r
-\r
-#endif\r
+       fghCloseDisplay ();\r
 \r
     fgState.Initialised = GL_FALSE;\r
 }\r
index e69de29..c134a63 100644 (file)
@@ -0,0 +1,160 @@
+/*\r
+ * freeglut_cursor_mswin.c\r
+ *\r
+ * The Windows-specific mouse cursor related stuff.\r
+ *\r
+ * Copyright (c) 2012 Stephen J. Baker. All Rights Reserved.\r
+ * Written by John F. Fay, <fayjf@sourceforge.net>\r
+ * Creation date: Thu Jan 19, 2012\r
+ *\r
+ * Permission is hereby granted, free of charge, to any person obtaining a\r
+ * copy of this software and associated documentation files (the "Software"),\r
+ * to deal in the Software without restriction, including without limitation\r
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,\r
+ * and/or sell copies of the Software, and to permit persons to whom the\r
+ * Software is furnished to do so, subject to the following conditions:\r
+ *\r
+ * The above copyright notice and this permission notice shall be included\r
+ * in all copies or substantial portions of the Software.\r
+ *\r
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS\r
+ * OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,\r
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL\r
+ * PAWEL W. OLSZTA BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER\r
+ * IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN\r
+ * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.\r
+ */\r
+\r
+#include <GL/freeglut.h>\r
+#include "freeglut_internal_mswin.h"\r
+\r
+\r
+/*\r
+ * A call to this function should initialize all the display stuff...\r
+ */\r
+void fghInitialize( const char* displayName )\r
+{\r
+    WNDCLASS wc;\r
+    ATOM atom;\r
+\r
+    /* What we need to do is to initialize the fgDisplay global structure here. */\r
+    fgDisplay.Instance = GetModuleHandle( NULL );\r
+    fgDisplay.DisplayName= displayName ? strdup(displayName) : 0 ;\r
+    atom = GetClassInfo( fgDisplay.Instance, _T("FREEGLUT"), &wc );\r
+\r
+    if( atom == 0 )\r
+    {\r
+        ZeroMemory( &wc, sizeof(WNDCLASS) );\r
+\r
+        /*\r
+         * Each of the windows should have its own device context, and we\r
+         * want redraw events during Vertical and Horizontal Resizes by\r
+         * the user.\r
+         *\r
+         * XXX Old code had "| CS_DBCLCKS" commented out.  Plans for the\r
+         * XXX future?  Dead-end idea?\r
+         */\r
+        wc.lpfnWndProc    = fgWindowProc;\r
+        wc.cbClsExtra     = 0;\r
+        wc.cbWndExtra     = 0;\r
+        wc.hInstance      = fgDisplay.Instance;\r
+        wc.hIcon          = LoadIcon( fgDisplay.Instance, _T("GLUT_ICON") );\r
+\r
+#if defined(_WIN32_WCE)\r
+        wc.style          = CS_HREDRAW | CS_VREDRAW;\r
+#else\r
+        wc.style          = CS_OWNDC | CS_HREDRAW | CS_VREDRAW;\r
+        if (!wc.hIcon)\r
+          wc.hIcon        = LoadIcon( NULL, IDI_WINLOGO );\r
+#endif\r
+\r
+        wc.hCursor        = LoadCursor( NULL, IDC_ARROW );\r
+        wc.hbrBackground  = NULL;\r
+        wc.lpszMenuName   = NULL;\r
+        wc.lpszClassName  = _T("FREEGLUT");\r
+\r
+        /* Register the window class */\r
+        atom = RegisterClass( &wc );\r
+        FREEGLUT_INTERNAL_ERROR_EXIT ( atom, "Window Class Not Registered", "fghInitialize" );\r
+    }\r
+\r
+    /* The screen dimensions can be obtained via GetSystemMetrics() calls */\r
+    fgDisplay.ScreenWidth  = GetSystemMetrics( SM_CXSCREEN );\r
+    fgDisplay.ScreenHeight = GetSystemMetrics( SM_CYSCREEN );\r
+\r
+    {\r
+        HWND desktop = GetDesktopWindow( );\r
+        HDC  context = GetDC( desktop );\r
+\r
+        fgDisplay.ScreenWidthMM  = GetDeviceCaps( context, HORZSIZE );\r
+        fgDisplay.ScreenHeightMM = GetDeviceCaps( context, VERTSIZE );\r
+\r
+        ReleaseDC( desktop, context );\r
+    }\r
+    /* If we have a DisplayName try to use it for metrics */\r
+    if( fgDisplay.DisplayName )\r
+    {\r
+        HDC context = CreateDC(fgDisplay.DisplayName,0,0,0);\r
+        if( context )\r
+        {\r
+           fgDisplay.ScreenWidth  = GetDeviceCaps( context, HORZRES );\r
+           fgDisplay.ScreenHeight = GetDeviceCaps( context, VERTRES );\r
+           fgDisplay.ScreenWidthMM  = GetDeviceCaps( context, HORZSIZE );\r
+           fgDisplay.ScreenHeightMM = GetDeviceCaps( context, VERTSIZE );\r
+           DeleteDC(context);\r
+        }\r
+        else\r
+           fgWarning("fghInitialize: "\r
+                     "CreateDC failed, Screen size info may be incorrect\n"\r
+          "This is quite likely caused by a bad '-display' parameter");\r
+      \r
+    }\r
+    /* Set the timer granularity to 1 ms */\r
+    timeBeginPeriod ( 1 );\r
+\r
+\r
+    fgState.Initialised = GL_TRUE;\r
+\r
+    /* Avoid registering atexit callback on Win32 as it results in an access\r
+     * violation due to calling into a module which has been unloaded.\r
+     * Any cleanup isn't needed on Windows anyway, the OS takes care of it.c\r
+     * see: http://blogs.msdn.com/b/oldnewthing/archive/2012/01/05/10253268.aspx\r
+     */\r
+/*    atexit(fgDeinitialize); */\r
+\r
+    /* InputDevice uses GlutTimerFunc(), so fgState.Initialised must be TRUE */\r
+    fgInitialiseInputDevices();\r
+}\r
+\r
+\r
+\r
+/* Platform-Specific Deinitialization Functions: */\r
+void fghDeinitialiseInputDevices ( void )\r
+{\r
+#if !defined(_WIN32_WCE)\r
+    if ( fgState.JoysticksInitialised )\r
+        fgJoystickClose( );\r
+\r
+    if ( fgState.InputDevsInitialised )\r
+        fgInputDeviceClose( );\r
+#endif /* !defined(_WIN32_WCE) */\r
+    fgState.JoysticksInitialised = GL_FALSE;\r
+    fgState.InputDevsInitialised = GL_FALSE;\r
+}\r
+\r
+void fghCloseDisplay ( void )\r
+{\r
+    if( fgDisplay.DisplayName )\r
+    {\r
+        free( fgDisplay.DisplayName );\r
+        fgDisplay.DisplayName = NULL;\r
+    }\r
+\r
+    /* Reset the timer granularity */\r
+    timeEndPeriod ( 1 );\r
+}\r
+\r
+\r
+\r
+\r
+\r