curlsshsink: remove redundant else statement
authorLuis de Bethencourt <luis@debethencourt.com>
Mon, 6 Jul 2015 14:24:06 +0000 (15:24 +0100)
committerLuis de Bethencourt <luis@debethencourt.com>
Mon, 6 Jul 2015 14:25:16 +0000 (15:25 +0100)
Else statement is redundant when the if block finishes with an return.

ext/curl/gstcurlsshsink.c

index d2dac8d8913f1666ba26168004c9a4b385cf4b2d..b06cbcd5bf5e80a5ee3fa7ee7304bdbf53e182de 100644 (file)
@@ -416,16 +416,15 @@ gst_curl_ssh_sink_set_options_unlocked (GstCurlBaseSink * bcsink)
     bcsink->error = g_strdup_printf ("failed to set SSH_KEYFUNCTION callback: "
         "%s", curl_easy_strerror (curl_err));
     return FALSE;
-  } else {
-    /* SSH_KEYFUNCTION callback successfully installed so go on and
-     * set the '*clientp' parameter as well */
-    if ((curl_err =
-            curl_easy_setopt (bcsink->curl, CURLOPT_SSH_KEYDATA,
-                sink)) != CURLE_OK) {
-      bcsink->error = g_strdup_printf ("failed to set CURLOPT_SSH_KEYDATA: %s",
-          curl_easy_strerror (curl_err));
-      return FALSE;
-    }
+  }
+  /* SSH_KEYFUNCTION callback successfully installed so go on and
+   * set the '*clientp' parameter as well */
+  if ((curl_err =
+          curl_easy_setopt (bcsink->curl, CURLOPT_SSH_KEYDATA,
+              sink)) != CURLE_OK) {
+    bcsink->error = g_strdup_printf ("failed to set CURLOPT_SSH_KEYDATA: %s",
+        curl_easy_strerror (curl_err));
+    return FALSE;
   }
 
   return TRUE;