changeset 4472:791b3256fb22

Mostly cleaned up warnings with -Wmissing-prototypes
author Sam Lantinga <slouken@libsdl.org>
date Sat, 26 Jun 2010 08:56:48 -0700
parents 11cedc036ca1
children e8045ba3f79f
files src/SDL.c src/SDL_assert.c src/SDL_assert_c.h src/SDL_compat.c src/SDL_error.c src/SDL_error_c.h src/audio/SDL_audio.c src/cpuinfo/SDL_cpuinfo.c src/file/cocoa/SDL_rwopsbundlesupport.m src/haptic/SDL_haptic.c src/haptic/SDL_haptic_c.h src/joystick/SDL_joystick_c.h src/main/dummy/SDL_dummy_main.c src/thread/SDL_thread.c src/video/SDL_blit_slow.c src/video/SDL_renderer_sw.c src/video/SDL_stretch.c src/video/SDL_sysvideo.h src/video/SDL_video.c src/video/x11/SDL_x11events.c src/video/x11/SDL_x11render.c src/video/x11/SDL_x11video.c src/video/x11/SDL_x11video.h
diffstat 23 files changed, 88 insertions(+), 39 deletions(-) [+]
line wrap: on
line diff
--- a/src/SDL.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/SDL.c	Sat Jun 26 08:56:48 2010 -0700
@@ -25,21 +25,15 @@
 
 #include "SDL.h"
 #include "SDL_fatal.h"
-#include "SDL_assert.h"
+#include "SDL_assert_c.h"
+#include "haptic/SDL_haptic_c.h"
+#include "joystick/SDL_joystick_c.h"
 
 #if !SDL_VIDEO_DISABLED
 #include "video/SDL_leaks.h"
 #endif
 
 /* Initialization/Cleanup routines */
-#if !SDL_JOYSTICK_DISABLED
-extern int SDL_JoystickInit(void);
-extern void SDL_JoystickQuit(void);
-#endif
-#if !SDL_HAPTIC_DISABLED
-extern int SDL_HapticInit(void);
-extern int SDL_HapticQuit(void);
-#endif
 #if !SDL_TIMERS_DISABLED
 extern void SDL_StartTicks(void);
 extern int SDL_TimerInit(void);
@@ -50,8 +44,6 @@
 extern int SDL_HelperWindowDestroy(void);
 #endif
 
-extern int SDL_AssertionsInit(void);
-extern void SDL_AssertionsQuit(void);
 
 /* The initialized subsystems */
 static Uint32 SDL_initialized = 0;
--- a/src/SDL_assert.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/SDL_assert.c	Sat Jun 26 08:56:48 2010 -0700
@@ -22,6 +22,7 @@
 
 #include "SDL.h"
 #include "SDL_assert.h"
+#include "SDL_assert_c.h"
 #include "video/SDL_sysvideo.h"
 
 #ifdef _WINDOWS
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/src/SDL_assert_c.h	Sat Jun 26 08:56:48 2010 -0700
@@ -0,0 +1,26 @@
+/*
+    SDL - Simple DirectMedia Layer
+    Copyright (C) 1997-2010 Sam Lantinga
+
+    This library is free software; you can redistribute it and/or
+    modify it under the terms of the GNU Lesser General Public
+    License as published by the Free Software Foundation; either
+    version 2.1 of the License, or (at your option) any later version.
+
+    This library is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+    Lesser General Public License for more details.
+
+    You should have received a copy of the GNU Lesser General Public
+    License along with this library; if not, write to the Free Software
+    Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+
+    Sam Lantinga
+    slouken@libsdl.org
+*/
+
+extern int SDL_AssertionsInit(void);
+extern void SDL_AssertionsQuit(void);
+
+/* vi: set ts=4 sw=4 expandtab: */
--- a/src/SDL_compat.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/SDL_compat.c	Sat Jun 26 08:56:48 2010 -0700
@@ -417,7 +417,7 @@
     }
 }
 
-int
+static int
 SDL_ResizeVideoMode(int width, int height, int bpp, Uint32 flags)
 {
     int w, h;
--- a/src/SDL_error.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/SDL_error.c	Sat Jun 26 08:56:48 2010 -0700
@@ -116,7 +116,7 @@
 /* This function has a bit more overhead than most error functions
    so that it supports internationalization and thread-safe errors.
 */
-char *
+static char *
 SDL_GetErrorMsg(char *errstr, unsigned int maxlen)
 {
     SDL_error *error;
--- a/src/SDL_error_c.h	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/SDL_error_c.h	Sat Jun 26 08:56:48 2010 -0700
@@ -58,4 +58,5 @@
 } SDL_error;
 
 #endif /* _SDL_error_c_h */
+
 /* vi: set ts=4 sw=4 expandtab: */
--- a/src/audio/SDL_audio.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/audio/SDL_audio.c	Sat Jun 26 08:56:48 2010 -0700
@@ -257,7 +257,7 @@
 
 /* Streaming functions (for when the input and output buffer sizes are different) */
 /* Write [length] bytes from buf into the streamer */
-void
+static void
 SDL_StreamWrite(SDL_AudioStreamer * stream, Uint8 * buf, int length)
 {
     int i;
@@ -269,7 +269,7 @@
 }
 
 /* Read [length] bytes out of the streamer into buf */
-void
+static void
 SDL_StreamRead(SDL_AudioStreamer * stream, Uint8 * buf, int length)
 {
     int i;
@@ -280,14 +280,14 @@
     }
 }
 
-int
+static int
 SDL_StreamLength(SDL_AudioStreamer * stream)
 {
     return (stream->write_pos - stream->read_pos) % stream->max_len;
 }
 
 /* Initialize the stream by allocating the buffer and setting the read/write heads to the beginning */
-int
+static int
 SDL_StreamInit(SDL_AudioStreamer * stream, int max_len, Uint8 silence)
 {
     /* First try to allocate the buffer */
@@ -307,7 +307,7 @@
 }
 
 /* Deinitialize the stream simply by freeing the buffer */
-void
+static void
 SDL_StreamDeinit(SDL_AudioStreamer * stream)
 {
     if (stream->buffer != NULL) {
--- a/src/cpuinfo/SDL_cpuinfo.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/cpuinfo/SDL_cpuinfo.c	Sat Jun 26 08:56:48 2010 -0700
@@ -335,7 +335,7 @@
 }
 
 /* Oh, such a sweet sweet trick, just not very useful. :) */
-const char *
+static const char *
 SDL_GetCPUType()
 {
     static char SDL_CPUType[48];
--- a/src/file/cocoa/SDL_rwopsbundlesupport.m	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/file/cocoa/SDL_rwopsbundlesupport.m	Sat Jun 26 08:56:48 2010 -0700
@@ -1,6 +1,8 @@
 #ifdef __APPLE__
 #import <Foundation/Foundation.h>
 
+#include "SDL_rwopsbundlesupport.h"
+
 /* For proper OS X applications, the resources are contained inside the application bundle.
  So the strategy is to first check the application bundle for the file, then fallback to the current working directory.
  Note: One additional corner-case is if the resource is in a framework's resource bundle instead of the app.
--- a/src/haptic/SDL_haptic.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/haptic/SDL_haptic.c	Sat Jun 26 08:56:48 2010 -0700
@@ -22,6 +22,7 @@
 #include "SDL_config.h"
 
 #include "SDL_syshaptic.h"
+#include "SDL_haptic_c.h"
 #include "../joystick/SDL_joystick_c.h" /* For SDL_PrivateJoystickValid */
 
 
--- /dev/null	Thu Jan 01 00:00:00 1970 +0000
+++ b/src/haptic/SDL_haptic_c.h	Sat Jun 26 08:56:48 2010 -0700
@@ -0,0 +1,26 @@
+/*
+    SDL - Simple DirectMedia Layer
+    Copyright (C) 1997-2010 Sam Lantinga
+
+    This library is free software; you can redistribute it and/or
+    modify it under the terms of the GNU Lesser General Public
+    License as published by the Free Software Foundation; either
+    version 2.1 of the License, or (at your option) any later version.
+
+    This library is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+    Lesser General Public License for more details.
+
+    You should have received a copy of the GNU Lesser General Public
+    License along with this library; if not, write to the Free Software
+    Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
+
+    Sam Lantinga
+    slouken@libsdl.org
+*/
+
+extern int SDL_HapticInit(void);
+extern void SDL_HapticQuit(void);
+
+/* vi: set ts=4 sw=4 expandtab: */
--- a/src/joystick/SDL_joystick_c.h	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/joystick/SDL_joystick_c.h	Sat Jun 26 08:56:48 2010 -0700
@@ -27,6 +27,10 @@
 /* The number of available joysticks on the system */
 extern Uint8 SDL_numjoysticks;
 
+/* Initialization and shutdown functions */
+extern int SDL_JoystickInit(void);
+extern void SDL_JoystickQuit(void);
+
 /* Internal event queueing functions */
 extern int SDL_PrivateJoystickAxis(SDL_Joystick * joystick,
                                    Uint8 axis, Sint16 value);
--- a/src/main/dummy/SDL_dummy_main.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/main/dummy/SDL_dummy_main.c	Sat Jun 26 08:56:48 2010 -0700
@@ -12,7 +12,10 @@
 #else
 /* Nothing to do on this platform */
 int
-SDL_main_stub_symbol()
+SDL_main_stub_symbol(void);
+
+int
+SDL_main_stub_symbol(void)
 {
     return 0;
 }
--- a/src/thread/SDL_thread.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/thread/SDL_thread.c	Sat Jun 26 08:56:48 2010 -0700
@@ -38,7 +38,7 @@
 static SDL_Thread **SDL_Threads = NULL;
 static SDL_mutex *thread_lock = NULL;
 
-int
+static int
 SDL_ThreadsInit(void)
 {
     int retval;
@@ -56,7 +56,7 @@
    clean up threads here.  If any threads are still running after this call,
    they will no longer have access to any per-thread data.
  */
-void
+static void
 SDL_ThreadsQuit(void)
 {
     SDL_mutex *mutex;
--- a/src/video/SDL_blit_slow.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/video/SDL_blit_slow.c	Sat Jun 26 08:56:48 2010 -0700
@@ -23,6 +23,7 @@
 
 #include "SDL_video.h"
 #include "SDL_blit.h"
+#include "SDL_blit_slow.h"
 
 /* The ONE TRUE BLITTER
  * This puppy has to handle all the unoptimized cases - yes, it's slow.
--- a/src/video/SDL_renderer_sw.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/video/SDL_renderer_sw.c	Sat Jun 26 08:56:48 2010 -0700
@@ -26,6 +26,7 @@
 #include "SDL_pixels_c.h"
 #include "SDL_rect_c.h"
 #include "SDL_yuv_sw_c.h"
+#include "SDL_renderer_sw.h"
 
 
 /* SDL surface based renderer implementation */
--- a/src/video/SDL_stretch.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/video/SDL_stretch.c	Sat Jun 26 08:56:48 2010 -0700
@@ -151,7 +151,7 @@
 #endif /* USE_ASM_STRETCH */
 
 #define DEFINE_COPY_ROW(name, type)			\
-void name(type *src, int src_w, type *dst, int dst_w)	\
+static void name(type *src, int src_w, type *dst, int dst_w)	\
 {							\
 	int i;						\
 	int pos, inc;					\
@@ -175,7 +175,7 @@
 /* *INDENT-ON* */
 
 /* The ASM code doesn't handle 24-bpp stretch blits */
-void
+static void
 copy_row3(Uint8 * src, int src_w, Uint8 * dst, int dst_w)
 {
     int i;
--- a/src/video/SDL_sysvideo.h	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/video/SDL_sysvideo.h	Sat Jun 26 08:56:48 2010 -0700
@@ -418,7 +418,7 @@
 #define SDL_CurrentDisplay	(&_this->displays[_this->current_display])
 #define SDL_CurrentRenderer	(SDL_CurrentDisplay->current_renderer)
 
-extern SDL_VideoDevice *SDL_GetVideoDevice();
+extern SDL_VideoDevice *SDL_GetVideoDevice(void);
 extern int SDL_AddBasicVideoDisplay(const SDL_DisplayMode * desktop_mode);
 extern int SDL_AddVideoDisplay(const SDL_VideoDisplay * display);
 extern SDL_bool SDL_AddDisplayMode(SDL_VideoDisplay *display, const SDL_DisplayMode * mode);
--- a/src/video/SDL_video.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/video/SDL_video.c	Sat Jun 26 08:56:48 2010 -0700
@@ -304,7 +304,7 @@
 }
 
 SDL_VideoDevice *
-SDL_GetVideoDevice()
+SDL_GetVideoDevice(void)
 {
     return _this;
 }
@@ -718,16 +718,6 @@
 }
 
 int
-SDL_SetDisplayMode(const SDL_DisplayMode * mode)
-{
-    if (!_this) {
-        SDL_UninitializedVideo();
-        return -1;
-    }
-    return SDL_SetDisplayModeForDisplay(SDL_CurrentDisplay, mode);
-}
-
-int
 SDL_SetWindowDisplayMode(SDL_Window * window, const SDL_DisplayMode * mode)
 {
     CHECK_WINDOW_MAGIC(window, -1);
--- a/src/video/x11/SDL_x11events.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/video/x11/SDL_x11events.c	Sat Jun 26 08:56:48 2010 -0700
@@ -296,7 +296,7 @@
 }
 
 /* Ack!  XPending() actually performs a blocking read if no events available */
-int
+static int
 X11_Pending(Display * display)
 {
     /* Flush the display connection and look to see if events are queued */
--- a/src/video/x11/SDL_x11render.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/video/x11/SDL_x11render.c	Sat Jun 26 08:56:48 2010 -0700
@@ -26,6 +26,7 @@
 #include <limits.h> /* For INT_MIN and INT_MAX */
 
 #include "SDL_x11video.h"
+#include "SDL_x11render.h"
 #include "../SDL_rect_c.h"
 #include "../SDL_pixels_c.h"
 #include "../SDL_yuv_sw_c.h"
--- a/src/video/x11/SDL_x11video.c	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/video/x11/SDL_x11video.c	Sat Jun 26 08:56:48 2010 -0700
@@ -292,7 +292,7 @@
 }
 
 SDL_bool
-X11_UseDirectColorVisuals()
+X11_UseDirectColorVisuals(void)
 {
     /* Once we implement DirectColor colormaps and gamma ramp support...
        return SDL_getenv("SDL_VIDEO_X11_NODIRECTCOLOR") ? SDL_FALSE : SDL_TRUE;
--- a/src/video/x11/SDL_x11video.h	Thu Jun 17 22:23:20 2010 -0700
+++ b/src/video/x11/SDL_x11video.h	Sat Jun 26 08:56:48 2010 -0700
@@ -71,7 +71,7 @@
     SDL_scancode key_layout[256];
 } SDL_VideoData;
 
-extern SDL_bool X11_UseDirectColorVisuals();
+extern SDL_bool X11_UseDirectColorVisuals(void);
 
 #endif /* _SDL_x11video_h */