[Scummvm-cvs-logs] SF.net SVN: scummvm:[54575] scummvm/trunk/backends

fingolfin at users.sourceforge.net fingolfin at users.sourceforge.net
Mon Nov 29 17:35:30 CET 2010


Revision: 54575
          http://scummvm.svn.sourceforge.net/scummvm/?rev=54575&view=rev
Author:   fingolfin
Date:     2010-11-29 16:35:29 +0000 (Mon, 29 Nov 2010)

Log Message:
-----------
SDL: Trying to cleanup the FORBIDDEN_SYMBOL situation

Modified Paths:
--------------
    scummvm/trunk/backends/audiocd/sdl/sdl-audiocd.cpp
    scummvm/trunk/backends/events/dinguxsdl/dinguxsdl-events.cpp
    scummvm/trunk/backends/events/gp2xsdl/gp2xsdl-events.cpp
    scummvm/trunk/backends/events/linuxmotosdl/linuxmotosdl-events.cpp
    scummvm/trunk/backends/events/sdl/sdl-events.cpp
    scummvm/trunk/backends/fs/windows/windows-fs-factory.cpp
    scummvm/trunk/backends/graphics/dinguxsdl/dinguxsdl-graphics.cpp
    scummvm/trunk/backends/graphics/gp2xsdl/gp2xsdl-graphics.cpp
    scummvm/trunk/backends/graphics/linuxmotosdl/linuxmotosdl-graphics.h
    scummvm/trunk/backends/midi/windows.cpp
    scummvm/trunk/backends/platform/ps2/systemps2.cpp
    scummvm/trunk/backends/platform/sdl/sdl-sys.h
    scummvm/trunk/backends/platform/sdl/sdl.cpp
    scummvm/trunk/backends/platform/sdl/win32/win32.cpp
    scummvm/trunk/backends/timer/sdl/sdl-timer.cpp

Modified: scummvm/trunk/backends/audiocd/sdl/sdl-audiocd.cpp
===================================================================
--- scummvm/trunk/backends/audiocd/sdl/sdl-audiocd.cpp	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/audiocd/sdl/sdl-audiocd.cpp	2010-11-29 16:35:29 UTC (rev 54575)
@@ -27,9 +27,6 @@
 
 #if defined(SDL_BACKEND)
 
-// Disable symbol overrides so that we can use system headers.
-#define FORBIDDEN_SYMBOL_ALLOW_ALL
-
 #include "backends/audiocd/sdl/sdl-audiocd.h"
 
 SdlAudioCDManager::SdlAudioCDManager()

Modified: scummvm/trunk/backends/events/dinguxsdl/dinguxsdl-events.cpp
===================================================================
--- scummvm/trunk/backends/events/dinguxsdl/dinguxsdl-events.cpp	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/events/dinguxsdl/dinguxsdl-events.cpp	2010-11-29 16:35:29 UTC (rev 54575)
@@ -27,9 +27,6 @@
 
 #if defined(DINGUX)
 
-// Disable symbol overrides so that we can use system headers.
-#define FORBIDDEN_SYMBOL_EXCEPTION_FILE
-
 #include "backends/events/dinguxsdl/dinguxsdl-events.h"
 
 #define PAD_UP    SDLK_UP

Modified: scummvm/trunk/backends/events/gp2xsdl/gp2xsdl-events.cpp
===================================================================
--- scummvm/trunk/backends/events/gp2xsdl/gp2xsdl-events.cpp	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/events/gp2xsdl/gp2xsdl-events.cpp	2010-11-29 16:35:29 UTC (rev 54575)
@@ -27,9 +27,6 @@
 
 #if defined(GP2X) || defined(GP2XWIZ)
 
-// Disable symbol overrides so that we can use system headers.
-#define FORBIDDEN_SYMBOL_EXCEPTION_FILE
-
 #include "backends/events/gp2xsdl/gp2xsdl-events.h"
 #if defined(GP2X)
 #include "backends/platform/gp2x/gp2x-hw.h"

Modified: scummvm/trunk/backends/events/linuxmotosdl/linuxmotosdl-events.cpp
===================================================================
--- scummvm/trunk/backends/events/linuxmotosdl/linuxmotosdl-events.cpp	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/events/linuxmotosdl/linuxmotosdl-events.cpp	2010-11-29 16:35:29 UTC (rev 54575)
@@ -27,9 +27,6 @@
 
 #if defined(LINUXMOTO)
 
-// Disable symbol overrides so that we can use system headers.
-#define FORBIDDEN_SYMBOL_EXCEPTION_FILE
-
 #include "backends/events/linuxmotosdl/linuxmotosdl-events.h"
 #include "backends/platform/linuxmoto/linuxmoto-sdl.h"
 

Modified: scummvm/trunk/backends/events/sdl/sdl-events.cpp
===================================================================
--- scummvm/trunk/backends/events/sdl/sdl-events.cpp	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/events/sdl/sdl-events.cpp	2010-11-29 16:35:29 UTC (rev 54575)
@@ -27,9 +27,6 @@
 
 #if defined(SDL_BACKEND)
 
-// Disable symbol overrides so that we can use system headers.
-#define FORBIDDEN_SYMBOL_EXCEPTION_FILE
-
 #include "backends/events/sdl/sdl-events.h"
 #include "backends/platform/sdl/sdl.h"
 #include "common/config-manager.h"

Modified: scummvm/trunk/backends/fs/windows/windows-fs-factory.cpp
===================================================================
--- scummvm/trunk/backends/fs/windows/windows-fs-factory.cpp	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/fs/windows/windows-fs-factory.cpp	2010-11-29 16:35:29 UTC (rev 54575)
@@ -22,11 +22,11 @@
  * $Id$
  */
 
-#if defined(WIN32)
-
 // Disable symbol overrides so that we can use system headers.
 #define FORBIDDEN_SYMBOL_ALLOW_ALL
 
+#if defined(WIN32)
+
 #include "backends/fs/windows/windows-fs-factory.h"
 #include "backends/fs/windows/windows-fs.cpp"
 

Modified: scummvm/trunk/backends/graphics/dinguxsdl/dinguxsdl-graphics.cpp
===================================================================
--- scummvm/trunk/backends/graphics/dinguxsdl/dinguxsdl-graphics.cpp	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/graphics/dinguxsdl/dinguxsdl-graphics.cpp	2010-11-29 16:35:29 UTC (rev 54575)
@@ -27,9 +27,6 @@
 
 #if defined(DINGUX)
 
-// Disable symbol overrides so that we can use system headers.
-#define FORBIDDEN_SYMBOL_EXCEPTION_FILE
-
 #include "backends/graphics/dinguxsdl/dinguxsdl-graphics.h"
 #include "backends/events/dinguxsdl/dinguxsdl-events.h"
 #include "graphics/scaler/aspect.h"

Modified: scummvm/trunk/backends/graphics/gp2xsdl/gp2xsdl-graphics.cpp
===================================================================
--- scummvm/trunk/backends/graphics/gp2xsdl/gp2xsdl-graphics.cpp	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/graphics/gp2xsdl/gp2xsdl-graphics.cpp	2010-11-29 16:35:29 UTC (rev 54575)
@@ -27,9 +27,6 @@
 
 #ifdef GP2X
 
-// Disable symbol overrides so that we can use system headers.
-#define FORBIDDEN_SYMBOL_EXCEPTION_FILE
-
 #include "backends/graphics/gp2xsdl/gp2xsdl-graphics.h"
 #include "graphics/scaler/aspect.h"
 #include <SDL_gp2x.h>

Modified: scummvm/trunk/backends/graphics/linuxmotosdl/linuxmotosdl-graphics.h
===================================================================
--- scummvm/trunk/backends/graphics/linuxmotosdl/linuxmotosdl-graphics.h	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/graphics/linuxmotosdl/linuxmotosdl-graphics.h	2010-11-29 16:35:29 UTC (rev 54575)
@@ -26,9 +26,6 @@
 #ifndef BACKENDS_GRAPHICS_SDL_LINUXMOTO_H
 #define BACKENDS_GRAPHICS_SDL_LINUXMOTO_H
 
-// Disable symbol overrides so that we can use system headers.
-#define FORBIDDEN_SYMBOL_EXCEPTION_FILE
-
 #include "backends/graphics/sdl/sdl-graphics.h"
 
 class LinuxmotoSdlGraphicsManager : public SdlGraphicsManager {

Modified: scummvm/trunk/backends/midi/windows.cpp
===================================================================
--- scummvm/trunk/backends/midi/windows.cpp	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/midi/windows.cpp	2010-11-29 16:35:29 UTC (rev 54575)
@@ -22,13 +22,13 @@
  * $Id$
  */
 
+// Disable symbol overrides so that we can use system headers.
+#define FORBIDDEN_SYMBOL_ALLOW_ALL
+
 #include "common/scummsys.h"
 
 #if defined(WIN32) && !defined(_WIN32_WCE)
 
-// Disable symbol overrides so that we can use system headers.
-#define FORBIDDEN_SYMBOL_ALLOW_ALL
-
 #define WIN32_LEAN_AND_MEAN
 #include <windows.h>
 // winnt.h defines ARRAYSIZE, but we want our own one...

Modified: scummvm/trunk/backends/platform/ps2/systemps2.cpp
===================================================================
--- scummvm/trunk/backends/platform/ps2/systemps2.cpp	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/platform/ps2/systemps2.cpp	2010-11-29 16:35:29 UTC (rev 54575)
@@ -26,9 +26,6 @@
 // Disable symbol overrides so that we can use system headers.
 #define FORBIDDEN_SYMBOL_ALLOW_ALL
 
-// Disable symbol overrides so that we can use system headers.
-#define FORBIDDEN_SYMBOL_ALLOW_ALL
-
 #include <kernel.h>
 #include <stdio.h>
 #include <stdlib.h>

Modified: scummvm/trunk/backends/platform/sdl/sdl-sys.h
===================================================================
--- scummvm/trunk/backends/platform/sdl/sdl-sys.h	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/platform/sdl/sdl-sys.h	2010-11-29 16:35:29 UTC (rev 54575)
@@ -26,16 +26,21 @@
 #ifndef BACKEND_SDL_SYS_H
 #define BACKEND_SDL_SYS_H
 
-// Include the SDL headers, working around the fact that SDL_rwops.h
-// uses a FILE pointer in one place, which conflicts with common/forbidden.h
+// The purpose of this header is to include the SDL headers in a uniform
+// fashion, even on the Symbian port.
+// Moreover, it contains a workaround for the fact that SDL_rwops.h uses
+// a FILE pointer in one place, which conflicts with common/forbidden.h.
 
+
 #include "common/scummsys.h"
 
 // Remove FILE override from common/forbidden.h, and replace
 // it with an alternate slightly less unfriendly override.
+#if !defined(FORBIDDEN_SYMBOL_ALLOW_ALL) && !defined(FORBIDDEN_SYMBOL_EXCEPTION_FILE)
 #undef FILE
 typedef struct { int FAKE; } FAKE_FILE;
 #define FILE FAKE_FILE
+#endif
 
 #if defined(__SYMBIAN32__)
 #include <esdl\SDL.h>
@@ -43,8 +48,10 @@
 #include <SDL.h>
 #endif
 
-// Finally forbid FILE again
+// Finally forbid FILE again (if it was forbidden to start with)
+#if !defined(FORBIDDEN_SYMBOL_ALLOW_ALL) && !defined(FORBIDDEN_SYMBOL_EXCEPTION_FILE)
 #undef FILE 
 #define FILE	FORBIDDEN_SYMBOL_REPLACEMENT
+#endif
 
 #endif

Modified: scummvm/trunk/backends/platform/sdl/sdl.cpp
===================================================================
--- scummvm/trunk/backends/platform/sdl/sdl.cpp	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/platform/sdl/sdl.cpp	2010-11-29 16:35:29 UTC (rev 54575)
@@ -23,9 +23,6 @@
  *
  */
 
-// Disable symbol overrides so that we can use system headers.
-#define FORBIDDEN_SYMBOL_EXCEPTION_FILE
-
 #ifdef WIN32
 #define WIN32_LEAN_AND_MEAN
 #include <windows.h>

Modified: scummvm/trunk/backends/platform/sdl/win32/win32.cpp
===================================================================
--- scummvm/trunk/backends/platform/sdl/win32/win32.cpp	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/platform/sdl/win32/win32.cpp	2010-11-29 16:35:29 UTC (rev 54575)
@@ -23,6 +23,9 @@
  *
  */
 
+// Disable symbol overrides so that we can use system headers.
+#define FORBIDDEN_SYMBOL_ALLOW_ALL
+
 #include "common/scummsys.h"
 
 #ifdef WIN32
@@ -31,9 +34,6 @@
 #include <windows.h>
 #undef ARRAYSIZE // winnt.h defines ARRAYSIZE, but we want our own one...
 
-// Disable symbol overrides so that we can use system headers.
-#define FORBIDDEN_SYMBOL_ALLOW_ALL
-
 #include "backends/platform/sdl/win32/win32.h"
 #include "backends/fs/windows/windows-fs-factory.h"
 

Modified: scummvm/trunk/backends/timer/sdl/sdl-timer.cpp
===================================================================
--- scummvm/trunk/backends/timer/sdl/sdl-timer.cpp	2010-11-29 16:22:04 UTC (rev 54574)
+++ scummvm/trunk/backends/timer/sdl/sdl-timer.cpp	2010-11-29 16:35:29 UTC (rev 54575)
@@ -28,9 +28,6 @@
 
 #if defined(SDL_BACKEND)
 
-// Disable symbol overrides so that we can use system headers.
-#define FORBIDDEN_SYMBOL_EXCEPTION_FILE
-
 #include "backends/timer/sdl/sdl-timer.h"
 
 static Uint32 timer_handler(Uint32 interval, void *param) {


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.




More information about the Scummvm-git-logs mailing list