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

[nmh] / uip / show.c

diff --git a/uip/show.c b/uip/show.c
index 5b815f4e3c67824ab41f6cec007a43eb66bc6edd..b7f85fe13d1eae72b0451e5e113e0ae2be151dc9 100644 (file)
--- a/uip/show.c
+++ b/uip/show.c
@@ -32,8 +32,8 @@
     /*                         \
      * switches for mhlproc    \
      */                                \
-    X("concat", -6, CONCATSW) \
-    X("noconcat", -8, NCONCATSW) \
+    X("concat", 0, CONCATSW) \
+    X("noconcat", 0, NCONCATSW) \
     /*                         \
      * switches for mhshow     \
      */                                \
@@ -259,7 +259,7 @@ usage:
 
     for (msgnum = mp->lowsel; msgnum <= mp->hghsel; msgnum++)
        if (is_selected(mp, msgnum))
-           app_msgarg(&vec, getcpy (m_name (msgnum)));
+           app_msgarg(&vec, mh_xstrdup(m_name (msgnum)));
 
     seq_setcur (mp, mp->hghsel);       /* update current message  */
     seq_save (mp);                     /* synchronize sequences   */
@@ -447,7 +447,7 @@ invalid:
                    if (*dp) {
                        if ((result = !uprf (dp, "charset")))
                            goto out;
-                       dp += sizeof("charset") - 1;
+                       dp += LEN("charset");
                        while (isspace ((unsigned char) *dp))
                            dp++;
                        if (*dp++ != '=')