From f536a7f1374ef4a8d787c76c790792db529e0751 Mon Sep 17 00:00:00 2001 From: Fabian Keil Date: Fri, 22 Aug 2008 15:34:45 +0000 Subject: [PATCH] - Silence LLVM/Clang complaint. - Make received_hup_signal static. - Hide definitions for basedir, pidfile and received_hup_signal from __EMX__ as they only seem to be used in case of #ifdef unix. --- jcc.c | 19 ++++++++++++++++--- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/jcc.c b/jcc.c index f637d4a8..f150b580 100644 --- a/jcc.c +++ b/jcc.c @@ -1,4 +1,4 @@ -const char jcc_rcs[] = "$Id: jcc.c,v 1.182 2008/06/27 11:13:56 fabiankeil Exp $"; +const char jcc_rcs[] = "$Id: jcc.c,v 1.183 2008/08/21 07:09:35 fabiankeil Exp $"; /********************************************************************* * * File : $Source: /cvsroot/ijbswa/current/jcc.c,v $ @@ -33,6 +33,10 @@ const char jcc_rcs[] = "$Id: jcc.c,v 1.182 2008/06/27 11:13:56 fabiankeil Exp $" * * Revisions : * $Log: jcc.c,v $ + * Revision 1.183 2008/08/21 07:09:35 fabiankeil + * Accept Shoutcast responses again. Problem reported + * and fix suggested by Stefan in #2062860. + * * Revision 1.182 2008/06/27 11:13:56 fabiankeil * Fix possible NULL-pointer dereference reported * by din_a4 in #2003937. Pointy hat to me. @@ -1179,10 +1183,10 @@ pthread_mutex_t rand_mutex; #endif /* FEATURE_PTHREAD */ -#if defined(unix) || defined(__EMX__) +#if defined(unix) const char *basedir = NULL; const char *pidfile = NULL; -int received_hup_signal = 0; +static int received_hup_signal = 0; #endif /* defined unix */ /* HTTP snipplets. */ @@ -1291,6 +1295,13 @@ static const struct cruncher crunchers_light[] = { }; +/* + * XXX: Don't we really mean + * + * #if defined(unix) + * + * here? + */ #if !defined(_WIN32) && !defined(__OS2__) && !defined(AMIGA) /********************************************************************* * @@ -1324,7 +1335,9 @@ static void sig_handler(int the_signal) break; case SIGHUP: +#if defined(unix) received_hup_signal = 1; +#endif break; default: -- 2.39.2