More work for 1.3
authorPatrice Mandin <patmandin@gmail.com>
Wed, 20 Sep 2006 21:29:44 +0000
changeset 2033 b657f0bc72c2
parent 2032 ddf89133aebc
child 2034 59e1a50193aa
More work for 1.3
src/video/ataricommon/SDL_ataridevmouse_c.h
src/video/ataricommon/SDL_atarigl.c
src/video/ataricommon/SDL_atarigl_c.h
src/video/ataricommon/SDL_biosevents.c
src/video/ataricommon/SDL_gemdosevents.c
src/video/ataricommon/SDL_xbiosevents_c.h
--- a/src/video/ataricommon/SDL_ataridevmouse_c.h	Wed Sep 20 21:22:08 2006 +0000
+++ b/src/video/ataricommon/SDL_ataridevmouse_c.h	Wed Sep 20 21:29:44 2006 +0000
@@ -32,9 +32,6 @@
 
 #include "../SDL_sysvideo.h"
 
-/* Hidden "this" pointer for the video functions */
-#define _THIS	SDL_VideoDevice *this
-
 extern int SDL_AtariDevMouse_Open(void);
 extern void SDL_AtariDevMouse_Close(void);
 extern void SDL_AtariDevMouse_PostMouseEvents(_THIS, SDL_bool buttonEvents);
--- a/src/video/ataricommon/SDL_atarigl.c	Wed Sep 20 21:22:08 2006 +0000
+++ b/src/video/ataricommon/SDL_atarigl.c	Wed Sep 20 21:29:44 2006 +0000
@@ -80,9 +80,9 @@
 {
 #if SDL_VIDEO_OPENGL
     if (gl_oldmesa) {
-        gl_active = InitOld(this, current);
+        gl_active = InitOld(_this, current);
     } else {
-        gl_active = InitNew(this, current);
+        gl_active = InitNew(_this, current);
     }
 #endif
 
@@ -95,8 +95,8 @@
 #if SDL_VIDEO_OPENGL
     if (gl_oldmesa) {
         /* Old mesa implementations */
-        if (this->gl_data->OSMesaDestroyLDG) {
-            this->gl_data->OSMesaDestroyLDG();
+        if (_this->gl_data->OSMesaDestroyLDG) {
+            _this->gl_data->OSMesaDestroyLDG();
         }
         if (gl_shadow) {
             Mfree(gl_shadow);
@@ -105,15 +105,15 @@
     } else {
         /* New mesa implementation */
         if (gl_ctx) {
-            if (this->gl_data->OSMesaDestroyContext) {
-                this->gl_data->OSMesaDestroyContext(gl_ctx);
+            if (_this->gl_data->OSMesaDestroyContext) {
+                _this->gl_data->OSMesaDestroyContext(gl_ctx);
             }
             gl_ctx = NULL;
         }
     }
 
     if (unload) {
-        SDL_AtariGL_UnloadLibrary(this);
+        SDL_AtariGL_UnloadLibrary(_this);
     }
 #endif /* SDL_VIDEO_OPENGL */
     gl_active = 0;
@@ -134,7 +134,7 @@
     }
 
     /* Unload previous driver */
-    SDL_AtariGL_UnloadLibrary(this);
+    SDL_AtariGL_UnloadLibrary(_this);
 
     /* Load library given by path */
     handle = SDL_LoadObject(path);
@@ -167,17 +167,17 @@
         return -1;
     }
 
-    this->gl_data->glGetIntegerv = SDL_LoadFunction(handle, "glGetIntegerv");
-    this->gl_data->glFinish = SDL_LoadFunction(handle, "glFinish");
-    this->gl_data->glFlush = SDL_LoadFunction(handle, "glFlush");
+    _this->gl_data->glGetIntegerv = SDL_LoadFunction(handle, "glGetIntegerv");
+    _this->gl_data->glFinish = SDL_LoadFunction(handle, "glFinish");
+    _this->gl_data->glFlush = SDL_LoadFunction(handle, "glFlush");
 
     cancel_load = SDL_FALSE;
-    if (this->gl_data->glGetIntegerv == NULL) {
+    if (_this->gl_data->glGetIntegerv == NULL) {
         cancel_load = SDL_TRUE;
     } else {
         /* We need either glFinish (OSMesa) or glFlush (TinyGL) */
-        if ((this->gl_data->glFinish == NULL) &&
-            (this->gl_data->glFlush == NULL)) {
+        if ((_this->gl_data->glFinish == NULL) &&
+            (_this->gl_data->glFlush == NULL)) {
             cancel_load = SDL_TRUE;
         }
     }
@@ -190,53 +190,53 @@
     }
 
     /* Load functions pointers (osmesa.ldg) */
-    this->gl_data->OSMesaCreateContextExt =
+    _this->gl_data->OSMesaCreateContextExt =
         SDL_LoadFunction(handle, "OSMesaCreateContextExt");
-    this->gl_data->OSMesaDestroyContext =
+    _this->gl_data->OSMesaDestroyContext =
         SDL_LoadFunction(handle, "OSMesaDestroyContext");
-    this->gl_data->OSMesaMakeCurrent =
+    _this->gl_data->OSMesaMakeCurrent =
         SDL_LoadFunction(handle, "OSMesaMakeCurrent");
-    this->gl_data->OSMesaPixelStore =
+    _this->gl_data->OSMesaPixelStore =
         SDL_LoadFunction(handle, "OSMesaPixelStore");
-    this->gl_data->OSMesaGetProcAddress =
+    _this->gl_data->OSMesaGetProcAddress =
         SDL_LoadFunction(handle, "OSMesaGetProcAddress");
 
     /* Load old functions pointers (mesa_gl.ldg, tiny_gl.ldg) */
-    this->gl_data->OSMesaCreateLDG =
+    _this->gl_data->OSMesaCreateLDG =
         SDL_LoadFunction(handle, "OSMesaCreateLDG");
-    this->gl_data->OSMesaDestroyLDG =
+    _this->gl_data->OSMesaDestroyLDG =
         SDL_LoadFunction(handle, "OSMesaDestroyLDG");
 
     gl_oldmesa = 0;
 
-    if ((this->gl_data->OSMesaCreateContextExt == NULL) ||
-        (this->gl_data->OSMesaDestroyContext == NULL) ||
-        (this->gl_data->OSMesaMakeCurrent == NULL) ||
-        (this->gl_data->OSMesaPixelStore == NULL) ||
-        (this->gl_data->OSMesaGetProcAddress == NULL)) {
+    if ((_this->gl_data->OSMesaCreateContextExt == NULL) ||
+        (_this->gl_data->OSMesaDestroyContext == NULL) ||
+        (_this->gl_data->OSMesaMakeCurrent == NULL) ||
+        (_this->gl_data->OSMesaPixelStore == NULL) ||
+        (_this->gl_data->OSMesaGetProcAddress == NULL)) {
         /* Hum, maybe old library ? */
-        if ((this->gl_data->OSMesaCreateLDG == NULL) ||
-            (this->gl_data->OSMesaDestroyLDG == NULL)) {
+        if ((_this->gl_data->OSMesaCreateLDG == NULL) ||
+            (_this->gl_data->OSMesaDestroyLDG == NULL)) {
             SDL_SetError("Could not retrieve OSMesa functions");
             SDL_UnloadObject(handle);
             /* Restore pointers to static library */
-            SDL_AtariGL_InitPointers(this);
+            SDL_AtariGL_InitPointers(_this);
             return -1;
         } else {
             gl_oldmesa = 1;
         }
     }
 
-    this->gl_config.dll_handle = handle;
+    _this->gl_config.dll_handle = handle;
     if (path) {
-        SDL_strlcpy(this->gl_config.driver_path, path,
-                    SDL_arraysize(this->gl_config.driver_path));
+        SDL_strlcpy(_this->gl_config.driver_path, path,
+                    SDL_arraysize(_this->gl_config.driver_path));
     } else {
         *this->gl_config.driver_path = '\0';
     }
 
 #endif
-    this->gl_config.driver_loaded = 1;
+    _this->gl_config.driver_loaded = 1;
 
     return 0;
 #else
@@ -250,10 +250,10 @@
     void *func = NULL;
 #if SDL_VIDEO_OPENGL
 
-    if (this->gl_config.dll_handle) {
+    if (_this->gl_config.dll_handle) {
         func = SDL_LoadFunction(this->gl_config.dll_handle, (void *) proc);
-    } else if (this->gl_data->OSMesaGetProcAddress) {
-        func = this->gl_data->OSMesaGetProcAddress(proc);
+    } else if (_this->gl_data->OSMesaGetProcAddress) {
+        func = _this->gl_data->OSMesaGetProcAddress(proc);
     }
 #endif
     return func;
@@ -284,7 +284,7 @@
         mesa_attrib = GL_ALPHA_BITS;
         break;
     case SDL_GL_DOUBLEBUFFER:
-        surface = this->screen;
+        surface = _this->screen;
         *value = ((surface->flags & SDL_DOUBLEBUF) == SDL_DOUBLEBUF);
         return 0;
     case SDL_GL_DEPTH_SIZE:
@@ -309,7 +309,7 @@
         return -1;
     }
 
-    this->gl_data->glGetIntegerv(mesa_attrib, value);
+    _this->gl_data->glGetIntegerv(mesa_attrib, value);
     return 0;
 #else
     return -1;
@@ -327,9 +327,9 @@
         return 0;
     }
 
-    if (this->gl_config.dll_handle) {
-        if ((this->gl_data->OSMesaMakeCurrent == NULL) ||
-            (this->gl_data->OSMesaPixelStore == NULL)) {
+    if (_this->gl_config.dll_handle) {
+        if ((_this->gl_data->OSMesaMakeCurrent == NULL) ||
+            (_this->gl_data->OSMesaPixelStore == NULL)) {
             return -1;
         }
     }
@@ -339,7 +339,7 @@
         return -1;
     }
 
-    surface = this->screen;
+    surface = _this->screen;
 
     if ((surface->format->BitsPerPixel == 15)
         || (surface->format->BitsPerPixel == 16)) {
@@ -349,7 +349,7 @@
     }
 
     if (!
-        (this->gl_data->
+        (_this->gl_data->
          OSMesaMakeCurrent(gl_ctx, surface->pixels, type, surface->w,
                            surface->h))) {
         SDL_SetError("Can not make OpenGL context current");
@@ -357,7 +357,7 @@
     }
 
     /* OSMesa draws upside down */
-    this->gl_data->OSMesaPixelStore(OSMESA_Y_UP, 0);
+    _this->gl_data->OSMesaPixelStore(OSMESA_Y_UP, 0);
 
     return 0;
 #else
@@ -370,17 +370,17 @@
 {
 #if SDL_VIDEO_OPENGL
     if (gl_active) {
-        if (this->gl_config.dll_handle) {
-            if (this->gl_data->glFinish) {
-                this->gl_data->glFinish();
-            } else if (this->gl_data->glFlush) {
-                this->gl_data->glFlush();
+        if (_this->gl_config.dll_handle) {
+            if (_this->gl_data->glFinish) {
+                _this->gl_data->glFinish();
+            } else if (_this->gl_data->glFlush) {
+                _this->gl_data->glFlush();
             }
         } else {
-            this->gl_data->glFinish();
+            _this->gl_data->glFinish();
         }
-        gl_copyshadow(this, this->screen);
-        gl_convert(this, this->screen);
+        gl_copyshadow(_this, _this->screen);
+        gl_convert(_this, _this->screen);
     }
 #endif
 }
@@ -389,18 +389,18 @@
 SDL_AtariGL_InitPointers(_THIS)
 {
 #if SDL_VIDEO_OPENGL
-    this->gl_data->OSMesaCreateContextExt = OSMesaCreateContextExt;
-    this->gl_data->OSMesaDestroyContext = OSMesaDestroyContext;
-    this->gl_data->OSMesaMakeCurrent = OSMesaMakeCurrent;
-    this->gl_data->OSMesaPixelStore = OSMesaPixelStore;
-    this->gl_data->OSMesaGetProcAddress = OSMesaGetProcAddress;
+    _this->gl_data->OSMesaCreateContextExt = OSMesaCreateContextExt;
+    _this->gl_data->OSMesaDestroyContext = OSMesaDestroyContext;
+    _this->gl_data->OSMesaMakeCurrent = OSMesaMakeCurrent;
+    _this->gl_data->OSMesaPixelStore = OSMesaPixelStore;
+    _this->gl_data->OSMesaGetProcAddress = OSMesaGetProcAddress;
 
-    this->gl_data->glGetIntegerv = glGetIntegerv;
-    this->gl_data->glFinish = glFinish;
-    this->gl_data->glFlush = glFlush;
+    _this->gl_data->glGetIntegerv = glGetIntegerv;
+    _this->gl_data->glFinish = glFinish;
+    _this->gl_data->glFlush = glFlush;
 
-    this->gl_data->OSMesaCreateLDG = NULL;
-    this->gl_data->OSMesaDestroyLDG = NULL;
+    _this->gl_data->OSMesaCreateLDG = NULL;
+    _this->gl_data->OSMesaDestroyLDG = NULL;
 #endif
 }
 
@@ -410,12 +410,12 @@
 SDL_AtariGL_UnloadLibrary(_THIS)
 {
 #if SDL_VIDEO_OPENGL
-    if (this->gl_config.dll_handle) {
-        SDL_UnloadObject(this->gl_config.dll_handle);
-        this->gl_config.dll_handle = NULL;
+    if (_this->gl_config.dll_handle) {
+        SDL_UnloadObject(_this->gl_config.dll_handle);
+        _this->gl_config.dll_handle = NULL;
 
         /* Restore pointers to static library */
-        SDL_AtariGL_InitPointers(this);
+        SDL_AtariGL_InitPointers(_this);
     }
 #endif
 }
@@ -432,8 +432,8 @@
     int recreatecontext;
     GLint newaccumsize;
 
-    if (this->gl_config.dll_handle) {
-        if (this->gl_data->OSMesaCreateContextExt == NULL) {
+    if (_this->gl_config.dll_handle) {
+        if (_this->gl_data->OSMesaCreateContextExt == NULL) {
             return 0;
         }
     }
@@ -496,14 +496,14 @@
 
     /* Try to keep current context if possible */
     newaccumsize =
-        this->gl_config.accum_red_size +
-        this->gl_config.accum_green_size +
-        this->gl_config.accum_blue_size + this->gl_config.accum_alpha_size;
+        _this->gl_config.accum_red_size +
+        _this->gl_config.accum_green_size +
+        _this->gl_config.accum_blue_size + _this->gl_config.accum_alpha_size;
     recreatecontext = 1;
     if (gl_ctx &&
         (gl_curformat == osmesa_format) &&
-        (gl_curdepth == this->gl_config.depth_size) &&
-        (gl_curstencil == this->gl_config.stencil_size) &&
+        (gl_curdepth == _this->gl_config.depth_size) &&
+        (gl_curstencil == _this->gl_config.stencil_size) &&
         (gl_curaccum == newaccumsize)) {
         recreatecontext = 0;
     }
@@ -511,17 +511,17 @@
         SDL_AtariGL_Quit(this, SDL_FALSE);
 
         gl_ctx =
-            this->gl_data->OSMesaCreateContextExt(osmesa_format,
-                                                  this->gl_config.
+            _this->gl_data->OSMesaCreateContextExt(osmesa_format,
+                                                  _this->gl_config.
                                                   depth_size,
-                                                  this->gl_config.
+                                                  _this->gl_config.
                                                   stencil_size,
                                                   newaccumsize, NULL);
 
         if (gl_ctx) {
             gl_curformat = osmesa_format;
-            gl_curdepth = this->gl_config.depth_size;
-            gl_curstencil = this->gl_config.stencil_size;
+            gl_curdepth = _this->gl_config.depth_size;
+            gl_curstencil = _this->gl_config.stencil_size;
             gl_curaccum = newaccumsize;
         } else {
             gl_curformat = 0;
@@ -543,16 +543,16 @@
     Uint32 redmask;
     int recreatecontext, tinygl_present;
 
-    if (this->gl_config.dll_handle) {
-        if (this->gl_data->OSMesaCreateLDG == NULL) {
+    if (_this->gl_config.dll_handle) {
+        if (_this->gl_data->OSMesaCreateLDG == NULL) {
             return 0;
         }
     }
 
     /* TinyGL only supports VDI_RGB (OSMESA_RGB) */
     tinygl_present = 0;
-    if (this->gl_config.dll_handle) {
-        if (this->gl_data->glFinish == NULL) {
+    if (_this->gl_config.dll_handle) {
+        if (_this->gl_data->glFinish == NULL) {
             tinygl_present = 1;
         }
     }
@@ -658,7 +658,7 @@
         break;
     default:
         if (tinygl_present) {
-            SDL_AtariGL_Quit(this, SDL_FALSE);
+            SDL_AtariGL_Quit(_this, SDL_FALSE);
             return 0;
         }
         gl_pixelsize = 1;
@@ -675,10 +675,10 @@
         recreatecontext = 0;
     }
     if (recreatecontext) {
-        SDL_AtariGL_Quit(this, SDL_FALSE);
+        SDL_AtariGL_Quit(_this, SDL_FALSE);
 
         gl_shadow =
-            this->gl_data->OSMesaCreateLDG(osmesa_format, GL_UNSIGNED_BYTE,
+            _this->gl_data->OSMesaCreateLDG(osmesa_format, GL_UNSIGNED_BYTE,
                                            current->w, current->h);
 
         if (gl_shadow) {
--- a/src/video/ataricommon/SDL_atarigl_c.h	Wed Sep 20 21:22:08 2006 +0000
+++ b/src/video/ataricommon/SDL_atarigl_c.h	Wed Sep 20 21:29:44 2006 +0000
@@ -32,10 +32,7 @@
 
 #include "../SDL_sysvideo.h"
 
-/* Hidden "this" pointer for the video functions */
-#define _THIS   SDL_VideoDevice *this
-
-struct SDL_PrivateGLData
+struct SDL_GLDriverData
 {
 
     int gl_active;              /* to stop switching drivers while we have a valid context */
@@ -87,20 +84,20 @@
 };
 
 /* Variable names */
-#define gl_active		(this->gl_data->gl_active)
-#define gl_ctx			(this->gl_data->ctx)
-#define gl_oldmesa		(this->gl_data->gl_oldmesa)
-#define gl_pixelsize	(this->gl_data->gl_pixelsize)
-#define gl_upsidedown	(this->gl_data->gl_upsidedown)
-#define gl_shadow		(this->gl_data->gl_shadow)
-#define gl_convert		(this->gl_data->ConvertSurface)
-#define gl_copyshadow	(this->gl_data->CopyShadow)
-#define gl_curformat	(this->gl_data->format)
-#define gl_curdepth		(this->gl_data->depth)
-#define gl_curstencil	(this->gl_data->stencil)
-#define gl_curaccum		(this->gl_data->accum)
-#define gl_curwidth		(this->gl_data->width)
-#define gl_curheight	(this->gl_data->height)
+#define gl_active		(_this->gl_data->gl_active)
+#define gl_ctx			(_this->gl_data->ctx)
+#define gl_oldmesa		(_this->gl_data->gl_oldmesa)
+#define gl_pixelsize	(_this->gl_data->gl_pixelsize)
+#define gl_upsidedown	(_this->gl_data->gl_upsidedown)
+#define gl_shadow		(_this->gl_data->gl_shadow)
+#define gl_convert		(_this->gl_data->ConvertSurface)
+#define gl_copyshadow	(_this->gl_data->CopyShadow)
+#define gl_curformat	(_this->gl_data->format)
+#define gl_curdepth		(_this->gl_data->depth)
+#define gl_curstencil	(_this->gl_data->stencil)
+#define gl_curaccum		(_this->gl_data->accum)
+#define gl_curwidth		(_this->gl_data->width)
+#define gl_curheight	(_this->gl_data->height)
 
 /* OpenGL functions */
 extern int SDL_AtariGL_Init(_THIS, SDL_Surface * current);
--- a/src/video/ataricommon/SDL_biosevents.c	Wed Sep 20 21:22:08 2006 +0000
+++ b/src/video/ataricommon/SDL_biosevents.c	Wed Sep 20 21:29:44 2006 +0000
@@ -99,9 +99,9 @@
     }
 
     if (use_dev_mouse) {
-        SDL_AtariDevMouse_PostMouseEvents(this, SDL_TRUE);
+        SDL_AtariDevMouse_PostMouseEvents(_this, SDL_TRUE);
     } else {
-        SDL_AtariXbios_PostMouseEvents(this, SDL_TRUE);
+        SDL_AtariXbios_PostMouseEvents(_this, SDL_TRUE);
     }
 
     /* Will be previous table */
@@ -116,7 +116,6 @@
 	{	\
 		if (special_keys_state & (1<<(numbit))) { \
 			bios_currentkeyboard[scancode]=0xFF; \
-			bios_currentascii[scancode]=0; \
 		}	\
 	}
 
--- a/src/video/ataricommon/SDL_gemdosevents.c	Wed Sep 20 21:22:08 2006 +0000
+++ b/src/video/ataricommon/SDL_gemdosevents.c	Wed Sep 20 21:29:44 2006 +0000
@@ -106,9 +106,9 @@
     }
 
     if (use_dev_mouse) {
-        SDL_AtariDevMouse_PostMouseEvents(this, SDL_TRUE);
+        SDL_AtariDevMouse_PostMouseEvents(_this, SDL_TRUE);
     } else {
-        SDL_AtariXbios_PostMouseEvents(this, SDL_TRUE);
+        SDL_AtariXbios_PostMouseEvents(_this, SDL_TRUE);
     }
 
     /* Will be previous table */
@@ -123,7 +123,6 @@
 	{	\
 		if (special_keys_state & (1<<(numbit))) { \
 			gemdos_currentkeyboard[scancode]=0xFF; \
-			gemdos_currentascii[scancode]=0; \
 		}	\
 	}
 
--- a/src/video/ataricommon/SDL_xbiosevents_c.h	Wed Sep 20 21:22:08 2006 +0000
+++ b/src/video/ataricommon/SDL_xbiosevents_c.h	Wed Sep 20 21:29:44 2006 +0000
@@ -32,9 +32,6 @@
 
 #include "../SDL_sysvideo.h"
 
-/* Hidden "this" pointer for the video functions */
-#define _THIS	SDL_VideoDevice *this
-
 #define ATARI_XBIOS_MOUSEEVENTS (1<<0)
 #define ATARI_XBIOS_JOYSTICKEVENTS (1<<1)