Minor refactor

- Check all pointers
  - Declaratons always at top of func/scope
  - Use established variable nomenclature
  - Skip useless if() stmt

Signed-off-by: Joachim Wiberg <troglobit@gmail.com>

src/api.c

@@ -116,17 +116,13 @@ static int do_reload (char *buf, size_t len) { return call(reload,  buf, len); }
 
 static int do_signal_svc(svc_t *svc, void *user_data)
 {
-	if (!svc)
+	int signo;
+
+	if (!svc || !user_data || !svc_is_running(svc))
 		return 1;
 
-	if (!svc_is_running(svc))
+	signo = *(int *)user_data;
-		return 1;
+	return !!kill(svc->pid, signo);
-
-	int sig = *(int *)user_data;
-	if (kill(svc->pid, sig))
-		return 1;
-
-	return 0;
 }
 
 static int do_signal(char *buf, size_t len, int sig)