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

[nmh] / sbr / client.c

diff --git a/sbr/client.c b/sbr/client.c
index 7ab49a98f05e9c274e3791f2ed9a4381a718fe1b..997889639e71847261674f91bdb0ce0bd1894a5b 100644 (file)
--- a/sbr/client.c
+++ b/sbr/client.c
@@ -10,7 +10,6 @@
 #include <h/mh.h>
 #include <h/mts.h>
 #include <h/utils.h>
 #include <h/mh.h>
 #include <h/mts.h>
 #include <h/utils.h>

-#include <errno.h>

 #include <sys/socket.h>
 #include <netinet/in.h>
 #include <netdb.h>
 #include <sys/socket.h>
 #include <netinet/in.h>
 #include <netdb.h>


@@ -26,7 +25,6 @@
 static char **client_brkstring (char *, char *, char *);
 static int client_brkany (char, char *);
 static char **client_copyip (char **, char **, int);
 static char **client_brkstring (char *, char *, char *);
 static int client_brkany (char, char *);
 static char **client_copyip (char **, char **, int);

-static char *client_getcpy (char *);

 static void client_freelist(char **);
 
 
 static void client_freelist(char **);
 
 


@@ -39,15 +37,15 @@ client (char *args, char *service, char *response, int len_response, int debug)
 
     ap = arguments;
     if (args != NULL && *args != 0) {
 
     ap = arguments;
     if (args != NULL && *args != 0) {

-       ap = client_copyip (client_brkstring (client_getcpy (args), " ", "\n"),
+       ap = client_copyip (client_brkstring (mh_xstrdup(args), " ", "\n"),

                ap, MAXARGS);
     } else {
        if (servers != NULL && *servers != 0)
                ap, MAXARGS);
     } else {
        if (servers != NULL && *servers != 0)

-           ap = client_copyip (client_brkstring (client_getcpy (servers), " ", "\n"),
+           ap = client_copyip (client_brkstring (mh_xstrdup(servers), " ", "\n"),

                ap, MAXARGS);
     }
     if (ap == arguments) {
                ap, MAXARGS);
     }
     if (ap == arguments) {

-       *ap++ = client_getcpy ("localhost");
+       *ap++ = mh_xstrdup("localhost");

        *ap = NULL;
     }
 
        *ap = NULL;
     }
 


@@ -77,12 +75,15 @@ client (char *args, char *service, char *response, int len_response, int debug)
        for (ai = res; ai != NULL; ai = ai->ai_next) {
            if (debug) {
                char address[NI_MAXHOST];
        for (ai = res; ai != NULL; ai = ai->ai_next) {
            if (debug) {
                char address[NI_MAXHOST];

+               char port[NI_MAXSERV];

 
                rc = getnameinfo(ai->ai_addr, ai->ai_addrlen, address,
 
                rc = getnameinfo(ai->ai_addr, ai->ai_addrlen, address,

-                                sizeof(address), NULL, 0, NI_NUMERICHOST);
+                                sizeof(address), port, sizeof port,
+                                NI_NUMERICHOST | NI_NUMERICSERV);

 
 

-               fprintf(stderr, "Connecting to %s...\n",
-                       rc ? "unknown" : address);
+               fprintf(stderr, "Connecting to %s:%s...\n",
+                       rc ? "unknown" : address,
+                       rc ? "--" : port);

            }
 
            sd = socket(ai->ai_family, ai->ai_socktype, ai->ai_protocol);
            }
 
            sd = socket(ai->ai_family, ai->ai_socktype, ai->ai_protocol);


@@ -135,8 +136,8 @@ static char *broken[MAXARGS + 1];
 static char **
 client_brkstring (char *strg, char *brksep, char *brkterm)
 {
 static char **
 client_brkstring (char *strg, char *brksep, char *brkterm)
 {

-    register int bi;
-    register char c, *sp;
+    int bi;
+    char c, *sp;

 
     sp = strg;
 
 
     sp = strg;
 


@@ -165,8 +166,8 @@ client_brkstring (char *strg, char *brksep, char *brkterm)
 static int
 client_brkany (char chr, char *strg)
 {
 static int
 client_brkany (char chr, char *strg)
 {

-    register char *sp;
- 
+    char *sp;
+

     if (strg)
        for (sp = strg; *sp; sp++)
            if (chr == *sp)
     if (strg)
        for (sp = strg; *sp; sp++)
            if (chr == *sp)


@@ -188,18 +189,3 @@ client_copyip (char **p, char **q, int len_q)
 
     return q;
 }
 
     return q;
 }

-
-
-static char *
-client_getcpy (char *str)
-{
-    char *cp;
-    size_t len;
-
-    len = strlen(str) + 1;
-    cp = mh_xmalloc(len);
-
-    memcpy (cp, str, len);
-    return cp;
-}
-