new.c: Order two return statements to match comment.

[nmh] / sbr / m_maildir.c

diff --git a/sbr/m_maildir.c b/sbr/m_maildir.c
index 6c8c5841736fe08f9fc2e9b45f2e318031ec0ce0..daf84a7328ca4dde4b4f7eb141c5a461850da4ff 100644 (file)
--- a/sbr/m_maildir.c
+++ b/sbr/m_maildir.c
@@ -1,6 +1,4 @@
-
-/*
- * m_maildir.c -- get the path for the mail directory
+/* m_maildir.c -- get the path for the mail directory

  *
  * This code is Copyright (c) 2002, by the authors of nmh.  See the
  * COPYRIGHT file in the root directory of the nmh distribution for
  *
  * This code is Copyright (c) 2002, by the authors of nmh.  See the
  * COPYRIGHT file in the root directory of the nmh distribution for


@@ -8,13 +6,13 @@
  */
 
 #include <h/mh.h>
  */
 
 #include <h/mh.h>

+#include <h/utils.h>
+#include "m_maildir.h"

 
 #define        CWD     "./"
 
 #define        CWD     "./"

-#define        NCWD    (sizeof(CWD) - 1)

 #define        DOT     "."
 #define        DOTDOT  ".."
 #define        PWD     "../"
 #define        DOT     "."
 #define        DOTDOT  ".."
 #define        PWD     "../"

-#define        NPWD    (sizeof(PWD) - 1)

 
 static char mailfold[BUFSIZ];
 
 
 static char mailfold[BUFSIZ];
 


@@ -24,10 +22,11 @@ static char mailfold[BUFSIZ];
 static char *exmaildir (char *);
 
 
 static char *exmaildir (char *);
 
 

+/* Returns static char[], never NULL. */

 char *
 m_maildir (char *folder)
 {
 char *
 m_maildir (char *folder)
 {

-    register char *cp, *ep;
+    char *cp, *ep;

 
     if ((cp = exmaildir (folder))
            && (ep = cp + strlen (cp) - 1) > cp
 
     if ((cp = exmaildir (folder))
            && (ep = cp + strlen (cp) - 1) > cp


@@ -38,17 +37,18 @@ m_maildir (char *folder)
 }
 
 
 }
 
 

+/* Return value must be free(3)'d. */

 char *
 m_mailpath (char *folder)
 {
 char *
 m_mailpath (char *folder)
 {

-    register char *cp;
+    char *cp;

     char maildir[BUFSIZ];
 
     if (*folder != '/'
     char maildir[BUFSIZ];
 
     if (*folder != '/'

-           && strncmp (folder, CWD, NCWD)
+           && !has_prefix(folder, CWD)

            && strcmp (folder, DOT)
            && strcmp (folder, DOTDOT)
            && strcmp (folder, DOT)
            && strcmp (folder, DOTDOT)

-           && strncmp (folder, PWD, NPWD)) {
+           && !has_prefix(folder, PWD)) {

        strncpy (maildir, mailfold, sizeof(maildir));   /* preserve... */
        cp = getcpy (m_maildir (folder));
        strncpy (mailfold, maildir, sizeof(mailfold));
        strncpy (maildir, mailfold, sizeof(maildir));   /* preserve... */
        cp = getcpy (m_maildir (folder));
        strncpy (mailfold, maildir, sizeof(mailfold));


@@ -60,20 +60,21 @@ m_mailpath (char *folder)
 }
 
 
 }
 
 

+/* Returns static char[], never NULL. */

 static char *
 exmaildir (char *folder)
 {
 static char *
 exmaildir (char *folder)
 {

-    register char *cp, *pp;
+    char *cp, *pp;

 
     /* use current folder if none is specified */
     if (folder == NULL)
        folder = getfolder(1);
 
     if (!(*folder != '/'
 
     /* use current folder if none is specified */
     if (folder == NULL)
        folder = getfolder(1);
 
     if (!(*folder != '/'

-           && strncmp (folder, CWD, NCWD)
+           && !has_prefix(folder, CWD)

            && strcmp (folder, DOT)
            && strcmp (folder, DOTDOT)
            && strcmp (folder, DOT)
            && strcmp (folder, DOTDOT)

-           && strncmp (folder, PWD, NPWD))) {
+           && !has_prefix(folder, PWD))) {

        strncpy (mailfold, folder, sizeof(mailfold));
        return mailfold;
     }
        strncpy (mailfold, folder, sizeof(mailfold));
        return mailfold;
     }


@@ -81,12 +82,14 @@ exmaildir (char *folder)
     cp = mailfold;
     if ((pp = context_find ("path")) && *pp) {
        if (*pp != '/') {
     cp = mailfold;
     if ((pp = context_find ("path")) && *pp) {
        if (*pp != '/') {

-           sprintf (cp, "%s/", mypath);
+           snprintf(cp, sizeof mailfold, "%s/", mypath);

            cp += strlen (cp);
        }
            cp += strlen (cp);
        }

-       cp = copy (pp, cp);
+       cp = stpcpy(cp, pp);

     } else {
     } else {

-       cp = copy (path ("./", TFOLDER), cp);
+        char *p = path("./", TFOLDER);
+       cp = stpcpy(cp, p);
+        free(p);

     }
     if (cp[-1] != '/')
        *cp++ = '/';
     }
     if (cp[-1] != '/')
        *cp++ = '/';