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

fingolfin at users.sourceforge.net fingolfin at users.sourceforge.net
Thu Nov 13 12:26:47 CET 2008


Revision: 35038
          http://scummvm.svn.sourceforge.net/scummvm/?rev=35038&view=rev
Author:   fingolfin
Date:     2008-11-13 11:26:47 +0000 (Thu, 13 Nov 2008)

Log Message:
-----------
* got rid of CDECL
* got rid of scumm_strrev
* added DISABLE_TEXT_CONSOLE flag which disables printf, warning, debug (but not error)

Modified Paths:
--------------
    scummvm/trunk/common/scummsys.h
    scummvm/trunk/common/util.cpp
    scummvm/trunk/common/util.h
    scummvm/trunk/engines/scumm/debugger.cpp
    scummvm/trunk/engines/scumm/scumm.h

Modified: scummvm/trunk/common/scummsys.h
===================================================================
--- scummvm/trunk/common/scummsys.h	2008-11-13 11:21:16 UTC (rev 35037)
+++ scummvm/trunk/common/scummsys.h	2008-11-13 11:26:47 UTC (rev 35038)
@@ -208,7 +208,6 @@
 	#define PLUGIN_EXPORT __declspec(dllexport)
 
 	#if _WIN32_WCE < 300
-	#define CDECL __cdecl
 	#define SMALL_SCREEN_DEVICE
 	#endif
 
@@ -404,10 +403,6 @@
 #define FORCEINLINE inline
 #endif
 
-#ifndef CDECL
-#define	CDECL
-#endif
-
 #ifndef PLUGIN_EXPORT
 #define PLUGIN_EXPORT
 #endif

Modified: scummvm/trunk/common/util.cpp
===================================================================
--- scummvm/trunk/common/util.cpp	2008-11-13 11:21:16 UTC (rev 35037)
+++ scummvm/trunk/common/util.cpp	2008-11-13 11:26:47 UTC (rev 35038)
@@ -458,8 +458,8 @@
  */
 int gDebugLevel = -1;
 
+#ifndef DISABLE_TEXT_CONSOLE
 
-
 static void debugHelper(const char *in_buf, bool caret = true) {
 	char buf[STRINGBUFLEN];
 
@@ -490,7 +490,7 @@
 	fflush(stdout);
 }
 
-void CDECL debug(int level, const char *s, ...) {
+void debug(int level, const char *s, ...) {
 	char buf[STRINGBUFLEN];
 	va_list va;
 
@@ -504,7 +504,7 @@
 	debugHelper(buf);
 }
 
-void CDECL debugN(int level, const char *s, ...) {
+void debugN(int level, const char *s, ...) {
 	char buf[STRINGBUFLEN];
 	va_list va;
 
@@ -518,7 +518,7 @@
 	debugHelper(buf, false);
 }
 
-void CDECL debug(const char *s, ...) {
+void debug(const char *s, ...) {
 	char buf[STRINGBUFLEN];
 	va_list va;
 
@@ -529,7 +529,7 @@
 	debugHelper(buf);
 }
 
-void CDECL debugC(int level, uint32 engine_level, const char *s, ...) {
+void debugC(int level, uint32 engine_level, const char *s, ...) {
 	char buf[STRINGBUFLEN];
 	va_list va;
 
@@ -544,7 +544,33 @@
 	debugHelper(buf);
 }
 
-void NORETURN CDECL error(const char *s, ...) {
+void warning(const char *s, ...) {
+	char buf[STRINGBUFLEN];
+	va_list va;
+
+	va_start(va, s);
+	vsnprintf(buf, STRINGBUFLEN, s, va);
+	va_end(va);
+
+#if !defined (__SYMBIAN32__)
+	fprintf(stderr, "WARNING: %s!\n", buf);
+#endif
+
+#if defined( USE_WINDBG )
+	strcat(buf, "\n");
+#if defined( _WIN32_WCE )
+	TCHAR buf_unicode[1024];
+	MultiByteToWideChar(CP_ACP, 0, buf, strlen(buf) + 1, buf_unicode, sizeof(buf_unicode));
+	OutputDebugString(buf_unicode);
+#else
+	OutputDebugString(buf);
+#endif
+#endif
+}
+
+#endif
+
+void NORETURN error(const char *s, ...) {
 	char buf_input[STRINGBUFLEN];
 	char buf_output[STRINGBUFLEN];
 	va_list va;
@@ -612,43 +638,6 @@
 	exit(1);
 }
 
-void CDECL warning(const char *s, ...) {
-	char buf[STRINGBUFLEN];
-	va_list va;
-
-	va_start(va, s);
-	vsnprintf(buf, STRINGBUFLEN, s, va);
-	va_end(va);
-
-#if !defined (__SYMBIAN32__)
-	fprintf(stderr, "WARNING: %s!\n", buf);
-#endif
-
-#if defined( USE_WINDBG )
-	strcat(buf, "\n");
-#if defined( _WIN32_WCE )
-	TCHAR buf_unicode[1024];
-	MultiByteToWideChar(CP_ACP, 0, buf, strlen(buf) + 1, buf_unicode, sizeof(buf_unicode));
-	OutputDebugString(buf_unicode);
-#else
-	OutputDebugString(buf);
-#endif
-#endif
-}
-
-char *scumm_strrev(char *str) {
-	if (!str)
-		return str;
-	int len = strlen(str);
-	if (len < 2)
-		return str;
-	char *p1, *p2;
-	for (p1 = str, p2 = str + len - 1; p1 < p2; p1++, p2--) {
-		SWAP(*p1, *p2);
-	}
-	return str;
-}
-
 Common::String tag2string(uint32 tag) {
 	char str[5];
 	str[0] = (char)(tag >> 24);

Modified: scummvm/trunk/common/util.h
===================================================================
--- scummvm/trunk/common/util.h	2008-11-13 11:21:16 UTC (rev 35037)
+++ scummvm/trunk/common/util.h	2008-11-13 11:26:47 UTC (rev 35038)
@@ -306,22 +306,35 @@
 
 
 #if defined(__GNUC__)
-void CDECL error(const char *s, ...) GCC_PRINTF(1, 2) NORETURN;
+void error(const char *s, ...) GCC_PRINTF(1, 2) NORETURN;
 #else
-void CDECL NORETURN error(const char *s, ...);
+void NORETURN error(const char *s, ...);
 #endif
 
-void CDECL warning(const char *s, ...) GCC_PRINTF(1, 2);
+#ifdef DISABLE_TEXT_CONSOLE
 
-void CDECL debug(int level, const char *s, ...) GCC_PRINTF(2, 3);
-void CDECL debug(const char *s, ...) GCC_PRINTF(1, 2);
-void CDECL debugN(int level, const char *s, ...) GCC_PRINTF(2, 3);
-void CDECL debugC(int level, uint32 engine_level, const char *s, ...) GCC_PRINTF(3, 4);
+inline void printf(const char *s, ...) {}
 
+inline void warning(const char *s, ...) {}
+
+inline void debug(int level, const char *s, ...) {}
+inline void debug(const char *s, ...) {}
+inline void debugN(int level, const char *s, ...) {}
+inline void debugC(int level, uint32 engine_level, const char *s, ...) {}
+
+#else
+
+void warning(const char *s, ...) GCC_PRINTF(1, 2);
+
+void debug(int level, const char *s, ...) GCC_PRINTF(2, 3);
+void debug(const char *s, ...) GCC_PRINTF(1, 2);
+void debugN(int level, const char *s, ...) GCC_PRINTF(2, 3);
+void debugC(int level, uint32 engine_level, const char *s, ...) GCC_PRINTF(3, 4);
+
+#endif
+
 extern int gDebugLevel;
 
-char *scumm_strrev(char *str);
-
 Common::String tag2string(uint32 tag);
 #define tag2str(x)	tag2string(x).c_str()
 

Modified: scummvm/trunk/engines/scumm/debugger.cpp
===================================================================
--- scummvm/trunk/engines/scumm/debugger.cpp	2008-11-13 11:21:16 UTC (rev 35037)
+++ scummvm/trunk/engines/scumm/debugger.cpp	2008-11-13 11:26:47 UTC (rev 35038)
@@ -41,7 +41,7 @@
 
 namespace Scumm {
 
-void CDECL debugC(int channel, const char *s, ...) {
+void debugC(int channel, const char *s, ...) {
 	char buf[STRINGBUFLEN];
 	va_list va;
 

Modified: scummvm/trunk/engines/scumm/scumm.h
===================================================================
--- scummvm/trunk/engines/scumm/scumm.h	2008-11-13 11:21:16 UTC (rev 35037)
+++ scummvm/trunk/engines/scumm/scumm.h	2008-11-13 11:26:47 UTC (rev 35038)
@@ -130,7 +130,7 @@
 };
 
 /* SCUMM Debug Channels */
-void CDECL debugC(int level, const char *s, ...);
+void debugC(int level, const char *s, ...);
 
 enum {
 	DEBUG_GENERAL	=	1 << 0,		// General debug


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