X-Git-Url: http://git.marmaro.de/?p=mmh;a=blobdiff_plain;f=sbr%2Fcontext_foil.c;h=93b198da2f2cddfc1d65be75da95e88aabd6d125;hp=d6c90c48b3ba89814ef22f217f2d9d74997b288e;hb=ced6090a330d3d83d0bce709f756aa3d7d65fea4;hpb=0e7106da702d97e10f3bd24d8284a2ab86044ebd

diff --git a/sbr/context_foil.c b/sbr/context_foil.c
index d6c90c4..93b198d 100644
--- a/sbr/context_foil.c
+++ b/sbr/context_foil.c
@@ -1,74 +1,68 @@
-
 /*
- * context_foil.c -- foil search of profile and context
- *
- * $Id$
- *
- * This code is Copyright (c) 2002, by the authors of nmh.  See the
- * COPYRIGHT file in the root directory of the nmh distribution for
- * complete copyright information.
- */
+** context_foil.c -- foil search of profile and context
+**
+** This code is Copyright (c) 2002, by the authors of nmh.  See the
+** COPYRIGHT file in the root directory of the nmh distribution for
+** complete copyright information.
+*/
 
 #include <h/mh.h>
+#include <h/utils.h>
 
 /*
- * Foil search of users .mh_profile
- * If error, return -1, else return 0
- */
+** Foil search of users .mh_profile
+** If error, return -1, else return 0
+*/
 
 int
 context_foil (char *path)
 {
-    register struct node *np;
+	register struct node *np;
 
-    /* In fact, nobody examines defpath in code paths where
-     * it's been set by us -- the uses in the source tree are:
-     *  1 sbr/context_read.c uses it only after setting it itself
-     *  2 uip/install_mh.c uses it only after setting it itself
-     *  3 uip/mshcmds.c and uip/mark.c print it if given the -debug switch
-     * A worthwhile piece of code cleanup would be to make 1 and
-     * 2 use a local variable and just delete 3.
-     *
-     * Similarly, context and ctxpath are not really used
-     * outside the context_* routines. It might be worth combining
-     * them into one file so the variables can be made static.
-     */
+	/* In fact, nobody examines defpath in code paths where
+	** it's been set by us -- the uses in the source tree are:
+	**  1 sbr/context_read.c uses it only after setting it itself
+	**  2 uip/install_mh.c uses it only after setting it itself
+	**  3 uip/mark.c print it if given the -debug switch
+	** A worthwhile piece of code cleanup would be to make 1 and
+	** 2 use a local variable and just delete 3.
+	**
+	** Similarly, context and ctxpath are not really used
+	** outside the context_* routines. It might be worth combining
+	** them into one file so the variables can be made static.
+	*/
 
-    /* We set context to NULL to indicate that no context file
-     * is to be read. (Using /dev/null doesn't work because we
-     * would try to lock it, which causes timeouts with some
-     * locking methods.)
-     */
-    defpath = context = NULL;
+	/* We set context to NULL to indicate that no context file
+	** is to be read. (Using /dev/null doesn't work because we
+	** would try to lock it, which causes timeouts with some
+	** locking methods.)
+	*/
+	defpath = context = NULL;
 
-    /*
-     * If path is given, create a minimal profile/context list
-     */
-    if (path) {
-	if (!(m_defs = (struct node *) malloc (sizeof(*np)))) {
-	    advise (NULL, "unable to allocate profile storage");
-	    return -1;
-	}
+	/*
+	** If path is given, create a minimal profile/context list
+	*/
+	if (path) {
+		m_defs = (struct node *) mh_xmalloc (sizeof(*np));
 
-	np = m_defs;
-	if (!(np->n_name = strdup ("Path"))) {
-	    advise (NULL, "strdup failed");
-	    return -1;
-	}
-	if (!(np->n_field = strdup (path))) {
-	    advise (NULL, "strdup failed");
-	    return -1;
-	}
-	np->n_context = 0;
-	np->n_next = NULL;
+		np = m_defs;
+		if (!(np->n_name = strdup ("Path"))) {
+			advise (NULL, "strdup failed");
+			return -1;
+		}
+		if (!(np->n_field = strdup (path))) {
+			advise (NULL, "strdup failed");
+			return -1;
+		}
+		np->n_context = 0;
+		np->n_next = NULL;
 
-	if (mypath == NULL && (mypath = getenv ("HOME")) != NULL)
-	    if (!(mypath = strdup (mypath))) {
-		advise (NULL, "strdup failed");
-		return -1;
-	    }
-    }
+		if (mypath == NULL && (mypath = getenv ("HOME")) != NULL)
+			if (!(mypath = strdup (mypath))) {
+				advise (NULL, "strdup failed");
+				return -1;
+			}
+	}
 
-    return 0;
+	return 0;
 }
-