path.c: Re-format path() to make it more clear what's happening.

[nmh] / sbr / read_switch_multiword_via_readline.c

diff --git a/sbr/read_switch_multiword_via_readline.c b/sbr/read_switch_multiword_via_readline.c
index 570469bdc13e0b460cbdecbee6eca2e2c4c49d19..3d6e88184368d2ae79d9b0244122d870c3e6a197 100644 (file)
--- a/sbr/read_switch_multiword_via_readline.c
+++ b/sbr/read_switch_multiword_via_readline.c
@@ -1,6 +1,4 @@
-
-/*
- * read_switch_multiword_via_readline.c -- get an answer from the user, with readline
+/* read_switch_multiword_via_readline.c -- get an answer from the user, with readline
  *
  * This code is Copyright (c) 2012, by the authors of nmh.  See the
  * COPYRIGHT file in the root directory of the nmh distribution for
@@ -43,7 +41,7 @@ read_switch_multiword_via_readline(char *prompt, struct swit *ansp)
            return NULL;
 
        if (ans[0] == '?' || ans[0] == '\0') {
-           printf("Options are:\n");
+           puts("Options are:");
            print_sw(ALL, ansp, "", stdout);
            free(ans);
            continue;
@@ -51,6 +49,8 @@ read_switch_multiword_via_readline(char *prompt, struct swit *ansp)
        add_history(ans);
        strncpy(ansbuf, ans, sizeof(ansbuf));
        ansbuf[sizeof(ansbuf) - 1] = '\0';
+        free(ans);
+
        cpp = brkstring(ansbuf, " ", NULL);
        switch (smatch(*cpp, ansp)) {
            case AMBIGSW:
@@ -60,10 +60,8 @@ read_switch_multiword_via_readline(char *prompt, struct swit *ansp)
                printf(" -%s unknown. Hit <CR> for help.\n", *cpp);
                continue;
            default:
-               free(ans);
                return cpp;
        }
-       free(ans);
     }
 }
 
@@ -107,10 +105,9 @@ nmh_command_generator(const char *text, int state)
        buf[sizeof(buf) - 1] = '\0';
        p = *brkstring(buf, " ", NULL);
        if (strncmp(p, text, len) == 0)
-       return strdup(p);
+            return strdup(p);
     }
 
     return NULL;
 }
 #endif /* READLINE_SUPPORT */
-