Don't `else' after goto. Simplify control flow.

[nmh] / sbr / client.c

diff --git a/sbr/client.c b/sbr/client.c
index 6695e084e59e802a8e09f6808032d446ce526ea4..997889639e71847261674f91bdb0ce0bd1894a5b 100644 (file)
--- a/sbr/client.c
+++ b/sbr/client.c
@@ -37,11 +37,11 @@ client (char *args, char *service, char *response, int len_response, int debug)
 
     ap = arguments;
     if (args != NULL && *args != 0) {
-       ap = client_copyip (client_brkstring (getcpy(args), " ", "\n"),
+       ap = client_copyip (client_brkstring (mh_xstrdup(args), " ", "\n"),
                ap, MAXARGS);
     } else {
        if (servers != NULL && *servers != 0)
-           ap = client_copyip (client_brkstring (getcpy(servers), " ", "\n"),
+           ap = client_copyip (client_brkstring (mh_xstrdup(servers), " ", "\n"),
                ap, MAXARGS);
     }
     if (ap == arguments) {
@@ -136,8 +136,8 @@ static char *broken[MAXARGS + 1];
 static char **
 client_brkstring (char *strg, char *brksep, char *brkterm)
 {
-    register int bi;
-    register char c, *sp;
+    int bi;
+    char c, *sp;
 
     sp = strg;
 
@@ -166,7 +166,7 @@ client_brkstring (char *strg, char *brksep, char *brkterm)
 static int
 client_brkany (char chr, char *strg)
 {
-    register char *sp;
+    char *sp;
 
     if (strg)
        for (sp = strg; *sp; sp++)