Clean-up in libman: make all calls to man_*msg not check return value.
authorKristaps Dzonsons <kristaps@bsd.lv>
Thu, 17 Mar 2011 11:56:17 +0000 (11:56 +0000)
committerKristaps Dzonsons <kristaps@bsd.lv>
Thu, 17 Mar 2011 11:56:17 +0000 (11:56 +0000)
Also convert man_vmsg to return void.

libman.h
man.c
man_macro.c
man_validate.c

index e085693a742f8955426078f63e30cab4db8fa486..b264a3ac260881462899c5a560e8199a72674160 100644 (file)
--- a/libman.h
+++ b/libman.h
@@ -1,4 +1,4 @@
-/*     $Id: libman.h,v 1.45 2011/01/12 10:43:22 kristaps Exp $ */
+/*     $Id: libman.h,v 1.46 2011/03/17 11:56:17 kristaps Exp $ */
 /*
  * Copyright (c) 2009, 2010 Kristaps Dzonsons <kristaps@bsd.lv>
  *
@@ -81,7 +81,7 @@ int             man_args(struct man *, int, int *, char *, char **);
 #define        ARGS_EOLN       (0)
 #define        ARGS_WORD       (1)
 #define        ARGS_QWORD      (1)
-int              man_vmsg(struct man *, enum mandocerr,
+void             man_vmsg(struct man *, enum mandocerr,
                        int, int, const char *, ...);
 int              man_valid_post(struct man *);
 int              man_valid_pre(struct man *, struct man_node *);
diff --git a/man.c b/man.c
index 8bcb7070109aad33463a6623d8af82d3c9cf05a7..51ed9dc4aa5148285d44e8bee29d3fd3fcf90216 100644 (file)
--- a/man.c
+++ b/man.c
@@ -1,4 +1,4 @@
-/*     $Id: man.c,v 1.102 2011/03/07 01:35:51 schwarze Exp $ */
+/*     $Id: man.c,v 1.103 2011/03/17 11:56:17 kristaps Exp $ */
 /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  *
@@ -622,7 +622,7 @@ err:        /* Error out. */
 }
 
 
-int
+void
 man_vmsg(struct man *man, enum mandocerr t, 
                int ln, int pos, const char *fmt, ...)
 {
@@ -632,7 +632,7 @@ man_vmsg(struct man *man, enum mandocerr t,
        va_start(ap, fmt);
        vsnprintf(buf, sizeof(buf) - 1, fmt, ap);
        va_end(ap);
-       return((*man->msg)(t, man->data, ln, pos, buf));
+       (*man->msg)(t, man->data, ln, pos, buf);
 }
 
 
index dbc4b4b032b6e15e4d255009b44b93a4bb32090f..979232c9e7ce32616b7fd09c03c253a58eb2ffd8 100644 (file)
@@ -1,4 +1,4 @@
-/*     $Id: man_macro.c,v 1.55 2011/01/12 16:55:22 kristaps Exp $ */
+/*     $Id: man_macro.c,v 1.56 2011/03/17 11:56:17 kristaps Exp $ */
 /*
  * Copyright (c) 2008, 2009, 2010 Kristaps Dzonsons <kristaps@bsd.lv>
  *
@@ -43,7 +43,7 @@ static        enum rew         rew_dohalt(enum mant, enum man_type,
                                const struct man_node *);
 static enum rew         rew_block(enum mant, enum man_type, 
                                const struct man_node *);
-static int              rew_warn(struct man *, 
+static void             rew_warn(struct man *, 
                                struct man_node *, enum mandocerr);
 
 const  struct man_macro __man_macros[MAN_MAX] = {
@@ -88,17 +88,19 @@ const       struct man_macro * const man_macros = __man_macros;
 /*
  * Warn when "n" is an explicit non-roff macro.
  */
-static int
+static void
 rew_warn(struct man *m, struct man_node *n, enum mandocerr er)
 {
 
        if (er == MANDOCERR_MAX || MAN_BLOCK != n->type)
-               return(1);
+               return;
        if (MAN_VALID & n->flags)
-               return(1);
+               return;
        if ( ! (MAN_EXPLICIT & man_macros[n->tok].flags))
-               return(1);
-       return(man_nmsg(m, n, er));
+               return;
+
+       assert(er < MANDOCERR_FATAL);
+       man_nmsg(m, n, er);
 }
 
 
@@ -123,16 +125,14 @@ man_unscope(struct man *m, const struct man_node *to,
                 * out to be lost.
                 */
                n = m->last->parent;
-               if ( ! rew_warn(m, m->last, er))
-                       return(0);
+               rew_warn(m, m->last, er);
                if ( ! man_valid_post(m))
                        return(0);
                m->last = n;
                assert(m->last);
        }
 
-       if ( ! rew_warn(m, m->last, er))
-               return(0);
+       rew_warn(m, m->last, er);
        if ( ! man_valid_post(m))
                return(0);
 
@@ -279,8 +279,7 @@ blk_close(MACRO_PROT_ARGS)
                        break;
 
        if (NULL == nn)
-               if ( ! man_pmsg(m, line, ppos, MANDOCERR_NOSCOPE))
-                       return(0);
+               man_pmsg(m, line, ppos, MANDOCERR_NOSCOPE);
 
        if ( ! rew_scope(MAN_BODY, m, ntok))
                return(0);
index c09ab36c5f9173488721a8476cc982a11a373b4a..e0e8e1cf0d8c771dd22210cc8481cb31d1eb160e 100644 (file)
@@ -1,4 +1,4 @@
-/*     $Id: man_validate.c,v 1.63 2011/03/07 01:35:51 schwarze Exp $ */
+/*     $Id: man_validate.c,v 1.64 2011/03/17 11:56:17 kristaps Exp $ */
 /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010 Ingo Schwarze <schwarze@openbsd.org>
@@ -223,9 +223,8 @@ check_text(CHKARGS)
                if ('\t' == *p) {
                        if (MAN_LITERAL & m->flags)
                                continue;
-                       if (man_pmsg(m, n->line, pos, MANDOCERR_BADTAB))
-                               continue;
-                       return(0);
+                       man_pmsg(m, n->line, pos, MANDOCERR_BADTAB);
+                       continue;
                }
 
                /* Check the special character. */