From 516aaa5d6ceb3c93704ae02c0b3443e56713a439 Mon Sep 17 00:00:00 2001
From: "Ryan C. Gordon" <icculus@icculus.org>
Date: Wed, 6 Jan 2010 06:40:16 +0000
Subject: [PATCH] Merged r3787:3788 from branches/SDL-1.2: better failures for
 joystick opening.

--HG--
extra : convert_revision : svn%3Ac70aab31-4412-0410-b14c-859654838e24/trunk%404358
---
 src/joystick/SDL_joystick.c | 111 ++++++++++++++++++------------------
 1 file changed, 55 insertions(+), 56 deletions(-)

diff --git a/src/joystick/SDL_joystick.c b/src/joystick/SDL_joystick.c
index 4d2c17419..784a780e3 100644
--- a/src/joystick/SDL_joystick.c
+++ b/src/joystick/SDL_joystick.c
@@ -114,64 +114,63 @@ SDL_JoystickOpen(int device_index)
 
     /* Create and initialize the joystick */
     joystick = (SDL_Joystick *) SDL_malloc((sizeof *joystick));
-    if (joystick != NULL) {
-        SDL_memset(joystick, 0, (sizeof *joystick));
-        joystick->index = device_index;
-        if (SDL_SYS_JoystickOpen(joystick) < 0) {
-            SDL_free(joystick);
-            joystick = NULL;
-        } else {
-            if (joystick->naxes > 0) {
-                joystick->axes = (Sint16 *) SDL_malloc
-                    (joystick->naxes * sizeof(Sint16));
-            }
-            if (joystick->nhats > 0) {
-                joystick->hats = (Uint8 *) SDL_malloc
-                    (joystick->nhats * sizeof(Uint8));
-            }
-            if (joystick->nballs > 0) {
-                joystick->balls = (struct balldelta *) SDL_malloc
-                    (joystick->nballs * sizeof(*joystick->balls));
-            }
-            if (joystick->nbuttons > 0) {
-                joystick->buttons = (Uint8 *) SDL_malloc
-                    (joystick->nbuttons * sizeof(Uint8));
-            }
-            if (((joystick->naxes > 0) && !joystick->axes)
-                || ((joystick->nhats > 0) && !joystick->hats)
-                || ((joystick->nballs > 0) && !joystick->balls)
-                || ((joystick->nbuttons > 0) && !joystick->buttons)) {
-                SDL_OutOfMemory();
-                SDL_JoystickClose(joystick);
-                joystick = NULL;
-            }
-            if (joystick->axes) {
-                SDL_memset(joystick->axes, 0,
-                           joystick->naxes * sizeof(Sint16));
-            }
-            if (joystick->hats) {
-                SDL_memset(joystick->hats, 0,
-                           joystick->nhats * sizeof(Uint8));
-            }
-            if (joystick->balls) {
-                SDL_memset(joystick->balls, 0,
-                           joystick->nballs * sizeof(*joystick->balls));
-            }
-            if (joystick->buttons) {
-                SDL_memset(joystick->buttons, 0,
-                           joystick->nbuttons * sizeof(Uint8));
-            }
-        }
+    if (joystick == NULL) {
+        SDL_OutOfMemory();
+        return NULL;
     }
-    if (joystick) {
-        /* Add joystick to list */
-        ++joystick->ref_count;
-        SDL_Lock_EventThread();
-        for (i = 0; SDL_joysticks[i]; ++i)
-            /* Skip to next joystick */ ;
-        SDL_joysticks[i] = joystick;
-        SDL_Unlock_EventThread();
+
+    SDL_memset(joystick, 0, (sizeof *joystick));
+    joystick->index = device_index;
+    if (SDL_SYS_JoystickOpen(joystick) < 0) {
+        SDL_free(joystick);
+        return NULL;
     }
+    if (joystick->naxes > 0) {
+        joystick->axes = (Sint16 *) SDL_malloc
+            (joystick->naxes * sizeof(Sint16));
+    }
+    if (joystick->nhats > 0) {
+        joystick->hats = (Uint8 *) SDL_malloc
+            (joystick->nhats * sizeof(Uint8));
+    }
+    if (joystick->nballs > 0) {
+        joystick->balls = (struct balldelta *) SDL_malloc
+            (joystick->nballs * sizeof(*joystick->balls));
+    }
+    if (joystick->nbuttons > 0) {
+        joystick->buttons = (Uint8 *) SDL_malloc
+            (joystick->nbuttons * sizeof(Uint8));
+    }
+    if (((joystick->naxes > 0) && !joystick->axes)
+        || ((joystick->nhats > 0) && !joystick->hats)
+        || ((joystick->nballs > 0) && !joystick->balls)
+        || ((joystick->nbuttons > 0) && !joystick->buttons)) {
+        SDL_OutOfMemory();
+        SDL_JoystickClose(joystick);
+        return NULL;
+    }
+    if (joystick->axes) {
+        SDL_memset(joystick->axes, 0, joystick->naxes * sizeof(Sint16));
+    }
+    if (joystick->hats) {
+        SDL_memset(joystick->hats, 0, joystick->nhats * sizeof(Uint8));
+    }
+    if (joystick->balls) {
+        SDL_memset(joystick->balls, 0,
+            joystick->nballs * sizeof(*joystick->balls));
+    }
+    if (joystick->buttons) {
+        SDL_memset(joystick->buttons, 0, joystick->nbuttons * sizeof(Uint8));
+    }
+
+    /* Add joystick to list */
+    ++joystick->ref_count;
+    SDL_Lock_EventThread();
+    for (i = 0; SDL_joysticks[i]; ++i)
+        /* Skip to next joystick */ ;
+    SDL_joysticks[i] = joystick;
+    SDL_Unlock_EventThread();
+
     return (joystick);
 }