Fixed bug 1021 (misplaced HAVE_SIGNAL_H condition in playmus.c and pl…

…aywave.c) Ozkan Sezer 2010-10-08 01:45:34 PDT The HAVE_SIGNAL_H dependent signal.h includes in playmus.c and playwave.c are misplaced, because SDL_config.h is not included at that point and this causes signal.h to be never included. Moving it after the SDL.h and SDL_mixer.h includes fixes the issue, like in the following.
libsdl-org · Feb 17, 2011 · 22d3147 · 22d3147
1 parent 9cb3e16
commit 22d3147
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/playmus.c b/playmus.c
@@ -26,16 +26,17 @@
 #include <stdio.h>
 #include <string.h>
 
-#ifdef HAVE_SIGNAL_H
-#include <signal.h>
-#endif
 #ifdef unix
 #include <unistd.h>
 #endif
 
 #include "SDL.h"
 #include "SDL_mixer.h"
 
+#ifdef HAVE_SIGNAL_H
+#include <signal.h>
+#endif
+
 
 static int audio_open = 0;
 static Mix_Music *music = NULL;

diff --git a/playwave.c b/playwave.c
@@ -26,16 +26,17 @@
 #include <stdio.h>
 #include <string.h>
 
-#ifdef HAVE_SIGNAL_H
-#include <signal.h>
-#endif
 #ifdef unix
 #include <unistd.h>
 #endif
 
 #include "SDL.h"
 #include "SDL_mixer.h"
 
+#ifdef HAVE_SIGNAL_H
+#include <signal.h>
+#endif
+
 
 /*
  * rcg06132001 various mixer tests. Define the ones you want.