From 9fae34286c9101bc4004f4582d4296e414cbc423 Mon Sep 17 00:00:00 2001
From: Sam Lantinga <slouken@libsdl.org>
Date: Tue, 4 Sep 2001 22:48:09 +0000
Subject: [PATCH] Fixed Solaris nitpicks (thanks Mattias!)

---
 configure.in                   | 12 ++---------
 include/SDL_types.h            |  2 +-
 src/timer/linux/SDL_systimer.c |  4 ++--
 src/video/x11/SDL_x11image.c   | 39 ++++++++--------------------------
 4 files changed, 14 insertions(+), 43 deletions(-)

diff --git a/configure.in b/configure.in
index b892dce2c..e552a2175 100644
--- a/configure.in
+++ b/configure.in
@@ -1196,20 +1196,14 @@ CheckQUARTZ()
     # and what runs on older Macs - while in theory "Carbon" defns
     # are consistent between the two, in practice Carbon is still
     # changing. -sts Aug 2000
+    mac_autoconf_target_workaround="MAC"
     CFLAGS="$CFLAGS -I/System/Library/Frameworks/Carbon.framework/Headers \
       -I/System/Library/Frameworks/Cocoa.framework/Headers -fpascal-strings \
-      -DENABLE_QUARTZ -DMACOSX -DTARGET_API_MAC_CARBON=1 -I\$(top_srcdir)/src/video/quartz"
+      -DENABLE_QUARTZ -DMACOSX -DTARGET_API_${mac_autoconf_target_workaround}_CARBON=1 -I\$(top_srcdir)/src/video/quartz"
     VIDEO_SUBDIRS="$VIDEO_SUBDIRS quartz"
     VIDEO_DRIVERS="$VIDEO_DRIVERS quartz/libvideo_quartz.la"
 }
 
-dnl Set up the kernel statistics library for Solaris
-CheckKSTAT()
-{
-    CFLAGS="$CFLAGS -DHAVE_KSTAT"
-    SYSTEM_LIBS="$SYSTEM_LIBS -lkstat"
-}
-
 case "$target" in
     *-*-linux*)
         ARCH=linux
@@ -1516,7 +1510,6 @@ case "$target" in
         CheckAAlib
         CheckOpenGL
         CheckPTHREAD
-        CheckKSTAT
         # Set up files for the main() stub
         COPY_ARCH_SRC(src/main, linux, SDL_main.c)
         # Set up files for the audio library
@@ -1566,7 +1559,6 @@ case "$target" in
         CheckAAlib
         CheckOpenGL
         CheckPTHREAD
-        CheckKSTAT
         # Set up files for the main() stub
         COPY_ARCH_SRC(src/main, linux, SDL_main.c)
         # Set up files for the audio library
diff --git a/include/SDL_types.h b/include/SDL_types.h
index af0483a81..2e9c26ab1 100644
--- a/include/SDL_types.h
+++ b/include/SDL_types.h
@@ -47,7 +47,7 @@ typedef signed int	Sint32;
 
 /* Figure out how to support 64-bit datatypes */
 #if !defined(__STRICT_ANSI__)
-#if defined(__GNUC__) || defined(__MWERKS__) /* MJS */
+#if defined(__GNUC__) || defined(__MWERKS__) || defined(__SUNPRO_C)
 #define SDL_HAS_64BIT_TYPE	long long
 #elif defined(_MSC_VER) /* VC++ */
 #define SDL_HAS_64BIT_TYPE	__int64
diff --git a/src/timer/linux/SDL_systimer.c b/src/timer/linux/SDL_systimer.c
index a57cc1219..3646d3e8a 100644
--- a/src/timer/linux/SDL_systimer.c
+++ b/src/timer/linux/SDL_systimer.c
@@ -46,11 +46,11 @@ static char rcsid =
 
 /* The following defines should really be determined at configure time */
 
-#ifdef linux
+#if defined(linux)
 /* Linux select() changes its timeout parameter upon return to contain
    the remaining time. Most other unixen leave it unchanged or undefined. */
 #define SELECT_SETS_REMAINING
-#elif defined(__bsdi__) || defined(__FreeBSD__) || defined(__sun__)
+#elif defined(__bsdi__) || defined(__FreeBSD__) || defined(__sun)
 #define USE_NANOSLEEP
 #endif
 
diff --git a/src/video/x11/SDL_x11image.c b/src/video/x11/SDL_x11image.c
index 149563715..f7da206f3 100644
--- a/src/video/x11/SDL_x11image.c
+++ b/src/video/x11/SDL_x11image.c
@@ -26,23 +26,13 @@ static char rcsid =
 #endif
 
 #include <stdlib.h>
+#include <unistd.h>
 
 #include "SDL_error.h"
 #include "SDL_endian.h"
 #include "SDL_events_c.h"
 #include "SDL_x11image_c.h"
 
-#if defined(__USLC__)
-#ifdef HAVE_KSTAT
-#undef HAVE_KSTAT
-#endif
-#include <unistd.h>
-#endif
-
-#ifdef HAVE_KSTAT
-#include <kstat.h>
-#endif
-
 #ifndef NO_SHARED_MEMORY
 
 /* Shared memory information */
@@ -177,13 +167,13 @@ void X11_DestroyImage(_THIS, SDL_Surface *screen)
 	}
 }
 
-/* This is a hack to see whether this system has more than 1 CPU */
+/* Determine the number of CPUs in the system */
 static int num_CPU(void)
 {
        static int num_cpus = 0;
 
        if(!num_cpus) {
-#ifdef linux
+#if defined(__linux)
            char line[BUFSIZ];
            FILE *pstat = fopen("/proc/stat", "r");
            if ( pstat ) {
@@ -194,23 +184,12 @@ static int num_CPU(void)
                }
                fclose(pstat);
            }
-#elif defined(HAVE_KSTAT)
-           kstat_ctl_t *kc = kstat_open();
-           kstat_t *ks;
-           kstat_named_t *kn;
-           if(kc) {
-               if((ks = kstat_lookup(kc, "unix", -1, "system_misc"))
-                  && kstat_read(kc, ks, NULL) != -1
-                  && (kn = kstat_data_lookup(ks, "ncpus")))
-#ifdef KSTAT_DATA_UINT32
-                   num_cpus = kn->value.ui32;
-#else
-                   num_cpus = kn->value.ul; /* needed in solaris <2.6 */
-#endif
-               kstat_close(kc);
-           }
-#elif defined(__USLC__)
-           num_cpus = (int)sysconf(_SC_NPROCESSORS_CONF);
+#elif defined(_SC_NPROCESSORS_ONLN)
+	   /* number of processors online (SVR4.0MP compliant machines) */
+           num_cpus = sysconf(_SC_NPROCESSORS_ONLN);
+#elif defined(_SC_NPROCESSORS_CONF)
+	   /* number of processors configured (SVR4.0MP compliant machines) */
+           num_cpus = sysconf(_SC_NPROCESSORS_CONF);
 #endif
            if ( num_cpus <= 0 ) {
                num_cpus = 1;