]> git.cameronkatri.com Git - mandoc.git/blobdiff - roff.c
Bugfix in roff_setstrn():
[mandoc.git] / roff.c
diff --git a/roff.c b/roff.c
index f99ff2dcd387fdcf57891ef28ca21c45d2fe5e6a..30e923098edf90b9cf067b56a2690f2d8a8554d0 100644 (file)
--- a/roff.c
+++ b/roff.c
@@ -1,7 +1,7 @@
-/*     $Id: roff.c,v 1.175 2012/11/19 17:57:23 schwarze Exp $ */
+/*     $Id: roff.c,v 1.211 2014/06/29 21:20:31 schwarze Exp $ */
 /*
  * Copyright (c) 2010, 2011, 2012 Kristaps Dzonsons <kristaps@bsd.lv>
 /*
  * Copyright (c) 2010, 2011, 2012 Kristaps Dzonsons <kristaps@bsd.lv>
- * Copyright (c) 2010, 2011, 2012 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2010-2014 Ingo Schwarze <schwarze@openbsd.org>
  *
  * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
  *
  * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
 
 #include <assert.h>
 #include <ctype.h>
 
 #include <assert.h>
 #include <ctype.h>
+#include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 
 #include "mandoc.h"
 #include <stdlib.h>
 #include <string.h>
 
 #include "mandoc.h"
+#include "mandoc_aux.h"
 #include "libroff.h"
 #include "libmandoc.h"
 
 #include "libroff.h"
 #include "libmandoc.h"
 
@@ -39,12 +41,16 @@ enum        rofft {
        ROFF_am,
        ROFF_ami,
        ROFF_am1,
        ROFF_am,
        ROFF_ami,
        ROFF_am1,
+       ROFF_as,
        ROFF_cc,
        ROFF_cc,
+       ROFF_ce,
        ROFF_de,
        ROFF_dei,
        ROFF_de1,
        ROFF_ds,
        ROFF_el,
        ROFF_de,
        ROFF_dei,
        ROFF_de1,
        ROFF_ds,
        ROFF_el,
+       ROFF_fam,
+       ROFF_hw,
        ROFF_hy,
        ROFF_ie,
        ROFF_if,
        ROFF_hy,
        ROFF_ie,
        ROFF_if,
@@ -56,6 +62,7 @@ enum  rofft {
        ROFF_ns,
        ROFF_ps,
        ROFF_rm,
        ROFF_ns,
        ROFF_ps,
        ROFF_rm,
+       ROFF_rr,
        ROFF_so,
        ROFF_ta,
        ROFF_tr,
        ROFF_so,
        ROFF_ta,
        ROFF_tr,
@@ -67,26 +74,10 @@ enum        rofft {
        ROFF_EQ,
        ROFF_EN,
        ROFF_cblock,
        ROFF_EQ,
        ROFF_EN,
        ROFF_cblock,
-       ROFF_ccond,
        ROFF_USERDEF,
        ROFF_MAX
 };
 
        ROFF_USERDEF,
        ROFF_MAX
 };
 
-enum   roffrule {
-       ROFFRULE_ALLOW,
-       ROFFRULE_DENY
-};
-
-/*
- * A single register entity.  If "set" is zero, the value of the
- * register should be the default one, which is per-register.
- * Registers are assumed to be unsigned ints for now.
- */
-struct reg {
-       int              set; /* whether set or not */
-       unsigned int     u; /* unsigned integer */
-};
-
 /*
  * An incredibly-simple string buffer.
  */
 /*
  * An incredibly-simple string buffer.
  */
@@ -104,14 +95,23 @@ struct     roffkv {
        struct roffkv   *next; /* next in list */
 };
 
        struct roffkv   *next; /* next in list */
 };
 
+/*
+ * A single number register as part of a singly-linked list.
+ */
+struct roffreg {
+       struct roffstr   key;
+       int              val;
+       struct roffreg  *next;
+};
+
 struct roff {
 struct roff {
-       enum mparset     parsetype; /* requested parse type */
        struct mparse   *parse; /* parse point */
        struct mparse   *parse; /* parse point */
+       int              options; /* parse options */
        struct roffnode *last; /* leaf of stack */
        struct roffnode *last; /* leaf of stack */
-       enum roffrule    rstack[RSTACK_MAX]; /* stack of !`ie' rules */
+       int              rstack[RSTACK_MAX]; /* stack of !`ie' rules */
        char             control; /* control character */
        int              rstackpos; /* position in rstack */
        char             control; /* control character */
        int              rstackpos; /* position in rstack */
-       struct reg       regs[REG__MAX];
+       struct roffreg  *regtab; /* number registers */
        struct roffkv   *strtab; /* user-defined strings & macros */
        struct roffkv   *xmbtab; /* multi-byte trans table (`tr') */
        struct roffstr  *xtab; /* single-byte trans table (`tr') */
        struct roffkv   *strtab; /* user-defined strings & macros */
        struct roffkv   *xmbtab; /* multi-byte trans table (`tr') */
        struct roffstr  *xtab; /* single-byte trans table (`tr') */
@@ -132,12 +132,12 @@ struct    roffnode {
        char            *name; /* node name, e.g. macro name */
        char            *end; /* end-rules: custom token */
        int              endspan; /* end-rules: next-line or infty */
        char            *name; /* node name, e.g. macro name */
        char            *end; /* end-rules: custom token */
        int              endspan; /* end-rules: next-line or infty */
-       enum roffrule    rule; /* current evaluation rule */
+       int              rule; /* current evaluation rule */
 };
 
 #define        ROFF_ARGS        struct roff *r, /* parse ctx */ \
                         enum rofft tok, /* tok of macro */ \
 };
 
 #define        ROFF_ARGS        struct roff *r, /* parse ctx */ \
                         enum rofft tok, /* tok of macro */ \
-                        char **bufp, /* input buffer */ \
+                        char **bufp, /* input buffer */ \
                         size_t *szp, /* size of input buffer */ \
                         int ln, /* parse line */ \
                         int ppos, /* original pos in buffer */ \
                         size_t *szp, /* size of input buffer */ \
                         int ln, /* parse line */ \
                         int ppos, /* original pos in buffer */ \
@@ -175,29 +175,40 @@ static    enum rofferr     roff_block_text(ROFF_ARGS);
 static enum rofferr     roff_block_sub(ROFF_ARGS);
 static enum rofferr     roff_cblock(ROFF_ARGS);
 static enum rofferr     roff_cc(ROFF_ARGS);
 static enum rofferr     roff_block_sub(ROFF_ARGS);
 static enum rofferr     roff_cblock(ROFF_ARGS);
 static enum rofferr     roff_cc(ROFF_ARGS);
-static enum rofferr     roff_ccond(ROFF_ARGS);
+static void             roff_ccond(struct roff *, int, int);
 static enum rofferr     roff_cond(ROFF_ARGS);
 static enum rofferr     roff_cond_text(ROFF_ARGS);
 static enum rofferr     roff_cond_sub(ROFF_ARGS);
 static enum rofferr     roff_ds(ROFF_ARGS);
 static enum rofferr     roff_cond(ROFF_ARGS);
 static enum rofferr     roff_cond_text(ROFF_ARGS);
 static enum rofferr     roff_cond_sub(ROFF_ARGS);
 static enum rofferr     roff_ds(ROFF_ARGS);
-static enum roffrule    roff_evalcond(const char *, int *);
+static int              roff_evalcond(const char *, int *);
+static int              roff_evalnum(const char *, int *, int *, int);
+static int              roff_evalpar(const char *, int *, int *);
+static int              roff_evalstrcond(const char *, int *);
 static void             roff_free1(struct roff *);
 static void             roff_free1(struct roff *);
+static void             roff_freereg(struct roffreg *);
 static void             roff_freestr(struct roffkv *);
 static char            *roff_getname(struct roff *, char **, int, int);
 static void             roff_freestr(struct roffkv *);
 static char            *roff_getname(struct roff *, char **, int, int);
-static const char      *roff_getstrn(const struct roff *, 
+static int              roff_getnum(const char *, int *, int *);
+static int              roff_getop(const char *, int *, char *);
+static int              roff_getregn(const struct roff *,
+                               const char *, size_t);
+static int              roff_getregro(const char *name);
+static const char      *roff_getstrn(const struct roff *,
                                const char *, size_t);
                                const char *, size_t);
+static enum rofferr     roff_it(ROFF_ARGS);
 static enum rofferr     roff_line_ignore(ROFF_ARGS);
 static enum rofferr     roff_nr(ROFF_ARGS);
 static void             roff_openeqn(struct roff *, const char *,
                                int, int, const char *);
 static enum rofft       roff_parse(struct roff *, const char *, int *);
 static enum rofferr     roff_line_ignore(ROFF_ARGS);
 static enum rofferr     roff_nr(ROFF_ARGS);
 static void             roff_openeqn(struct roff *, const char *,
                                int, int, const char *);
 static enum rofft       roff_parse(struct roff *, const char *, int *);
-static enum rofferr     roff_parsetext(char *);
-static enum rofferr     roff_res(struct roff *, 
+static enum rofferr     roff_parsetext(char **, size_t *, int, int *);
+static enum rofferr     roff_res(struct roff *,
                                char **, size_t *, int, int);
 static enum rofferr     roff_rm(ROFF_ARGS);
                                char **, size_t *, int, int);
 static enum rofferr     roff_rm(ROFF_ARGS);
+static enum rofferr     roff_rr(ROFF_ARGS);
 static void             roff_setstr(struct roff *,
                                const char *, const char *, int);
 static void             roff_setstr(struct roff *,
                                const char *, const char *, int);
-static void             roff_setstrn(struct roffkv **, const char *, 
+static void             roff_setstrn(struct roffkv **, const char *,
                                size_t, const char *, size_t, int);
 static enum rofferr     roff_so(ROFF_ARGS);
 static enum rofferr     roff_tr(ROFF_ARGS);
                                size_t, const char *, size_t, int);
 static enum rofferr     roff_so(ROFF_ARGS);
 static enum rofferr     roff_tr(ROFF_ARGS);
@@ -223,23 +234,28 @@ static    struct roffmac   roffs[ROFF_MAX] = {
        { "am", roff_block, roff_block_text, roff_block_sub, 0, NULL },
        { "ami", roff_block, roff_block_text, roff_block_sub, 0, NULL },
        { "am1", roff_block, roff_block_text, roff_block_sub, 0, NULL },
        { "am", roff_block, roff_block_text, roff_block_sub, 0, NULL },
        { "ami", roff_block, roff_block_text, roff_block_sub, 0, NULL },
        { "am1", roff_block, roff_block_text, roff_block_sub, 0, NULL },
+       { "as", roff_ds, NULL, NULL, 0, NULL },
        { "cc", roff_cc, NULL, NULL, 0, NULL },
        { "cc", roff_cc, NULL, NULL, 0, NULL },
+       { "ce", roff_line_ignore, NULL, NULL, 0, NULL },
        { "de", roff_block, roff_block_text, roff_block_sub, 0, NULL },
        { "dei", roff_block, roff_block_text, roff_block_sub, 0, NULL },
        { "de1", roff_block, roff_block_text, roff_block_sub, 0, NULL },
        { "ds", roff_ds, NULL, NULL, 0, NULL },
        { "el", roff_cond, roff_cond_text, roff_cond_sub, ROFFMAC_STRUCT, NULL },
        { "de", roff_block, roff_block_text, roff_block_sub, 0, NULL },
        { "dei", roff_block, roff_block_text, roff_block_sub, 0, NULL },
        { "de1", roff_block, roff_block_text, roff_block_sub, 0, NULL },
        { "ds", roff_ds, NULL, NULL, 0, NULL },
        { "el", roff_cond, roff_cond_text, roff_cond_sub, ROFFMAC_STRUCT, NULL },
+       { "fam", roff_line_ignore, NULL, NULL, 0, NULL },
+       { "hw", roff_line_ignore, NULL, NULL, 0, NULL },
        { "hy", roff_line_ignore, NULL, NULL, 0, NULL },
        { "ie", roff_cond, roff_cond_text, roff_cond_sub, ROFFMAC_STRUCT, NULL },
        { "if", roff_cond, roff_cond_text, roff_cond_sub, ROFFMAC_STRUCT, NULL },
        { "ig", roff_block, roff_block_text, roff_block_sub, 0, NULL },
        { "hy", roff_line_ignore, NULL, NULL, 0, NULL },
        { "ie", roff_cond, roff_cond_text, roff_cond_sub, ROFFMAC_STRUCT, NULL },
        { "if", roff_cond, roff_cond_text, roff_cond_sub, ROFFMAC_STRUCT, NULL },
        { "ig", roff_block, roff_block_text, roff_block_sub, 0, NULL },
-       { "it", roff_line_ignore, NULL, NULL, 0, NULL },
+       { "it", roff_it, NULL, NULL, 0, NULL },
        { "ne", roff_line_ignore, NULL, NULL, 0, NULL },
        { "nh", roff_line_ignore, NULL, NULL, 0, NULL },
        { "nr", roff_nr, NULL, NULL, 0, NULL },
        { "ns", roff_line_ignore, NULL, NULL, 0, NULL },
        { "ps", roff_line_ignore, NULL, NULL, 0, NULL },
        { "rm", roff_rm, NULL, NULL, 0, NULL },
        { "ne", roff_line_ignore, NULL, NULL, 0, NULL },
        { "nh", roff_line_ignore, NULL, NULL, 0, NULL },
        { "nr", roff_nr, NULL, NULL, 0, NULL },
        { "ns", roff_line_ignore, NULL, NULL, 0, NULL },
        { "ps", roff_line_ignore, NULL, NULL, 0, NULL },
        { "rm", roff_rm, NULL, NULL, 0, NULL },
+       { "rr", roff_rr, NULL, NULL, 0, NULL },
        { "so", roff_so, NULL, NULL, 0, NULL },
        { "ta", roff_line_ignore, NULL, NULL, 0, NULL },
        { "tr", roff_tr, NULL, NULL, 0, NULL },
        { "so", roff_so, NULL, NULL, 0, NULL },
        { "ta", roff_line_ignore, NULL, NULL, 0, NULL },
        { "tr", roff_tr, NULL, NULL, 0, NULL },
@@ -251,38 +267,39 @@ static    struct roffmac   roffs[ROFF_MAX] = {
        { "EQ", roff_EQ, NULL, NULL, 0, NULL },
        { "EN", roff_EN, NULL, NULL, 0, NULL },
        { ".", roff_cblock, NULL, NULL, 0, NULL },
        { "EQ", roff_EQ, NULL, NULL, 0, NULL },
        { "EN", roff_EN, NULL, NULL, 0, NULL },
        { ".", roff_cblock, NULL, NULL, 0, NULL },
-       { "\\}", roff_ccond, NULL, NULL, 0, NULL },
        { NULL, roff_userdef, NULL, NULL, 0, NULL },
 };
 
        { NULL, roff_userdef, NULL, NULL, 0, NULL },
 };
 
+/* not currently implemented: Ds em Eq LP Me PP pp Or Rd Sf SH */
 const  char *const __mdoc_reserved[] = {
        "Ac", "Ad", "An", "Ao", "Ap", "Aq", "Ar", "At",
        "Bc", "Bd", "Bf", "Bk", "Bl", "Bo", "Bq",
        "Brc", "Bro", "Brq", "Bsx", "Bt", "Bx",
        "Cd", "Cm", "Db", "Dc", "Dd", "Dl", "Do", "Dq",
 const  char *const __mdoc_reserved[] = {
        "Ac", "Ad", "An", "Ao", "Ap", "Aq", "Ar", "At",
        "Bc", "Bd", "Bf", "Bk", "Bl", "Bo", "Bq",
        "Brc", "Bro", "Brq", "Bsx", "Bt", "Bx",
        "Cd", "Cm", "Db", "Dc", "Dd", "Dl", "Do", "Dq",
-       "Ds", "Dt", "Dv", "Dx", "D1",
-       "Ec", "Ed", "Ef", "Ek", "El", "Em", "em",
-       "En", "Eo", "Eq", "Er", "Es", "Ev", "Ex",
+       "Dt", "Dv", "Dx", "D1",
+       "Ec", "Ed", "Ef", "Ek", "El", "Em",
+       "En", "Eo", "Er", "Es", "Ev", "Ex",
        "Fa", "Fc", "Fd", "Fl", "Fn", "Fo", "Fr", "Ft", "Fx",
        "Fa", "Fc", "Fd", "Fl", "Fn", "Fo", "Fr", "Ft", "Fx",
-       "Hf", "Ic", "In", "It", "Lb", "Li", "Lk", "Lp", "LP",
-       "Me", "Ms", "Mt", "Nd", "Nm", "No", "Ns", "Nx",
+       "Hf", "Ic", "In", "It", "Lb", "Li", "Lk", "Lp",
+       "Ms", "Mt", "Nd", "Nm", "No", "Ns", "Nx",
        "Oc", "Oo", "Op", "Os", "Ot", "Ox",
        "Oc", "Oo", "Op", "Os", "Ot", "Ox",
-       "Pa", "Pc", "Pf", "Po", "Pp", "PP", "pp", "Pq",
-       "Qc", "Ql", "Qo", "Qq", "Or", "Rd", "Re", "Rs", "Rv",
-       "Sc", "Sf", "Sh", "SH", "Sm", "So", "Sq",
+       "Pa", "Pc", "Pf", "Po", "Pp", "Pq",
+       "Qc", "Ql", "Qo", "Qq", "Re", "Rs", "Rv",
+       "Sc", "Sh", "Sm", "So", "Sq",
        "Ss", "St", "Sx", "Sy",
        "Ta", "Tn", "Ud", "Ux", "Va", "Vt", "Xc", "Xo", "Xr",
        "Ss", "St", "Sx", "Sy",
        "Ta", "Tn", "Ud", "Ux", "Va", "Vt", "Xc", "Xo", "Xr",
-       "%A", "%B", "%D", "%I", "%J", "%N", "%O",
+       "%A", "%B", "%C", "%D", "%I", "%J", "%N", "%O",
        "%P", "%Q", "%R", "%T", "%U", "%V",
        NULL
 };
 
        "%P", "%Q", "%R", "%T", "%U", "%V",
        NULL
 };
 
+/* not currently implemented: BT DE DS ME MT PT SY TQ YS */
 const  char *const __man_reserved[] = {
 const  char *const __man_reserved[] = {
-       "AT", "B", "BI", "BR", "BT", "DE", "DS", "DT",
-       "EE", "EN", "EQ", "EX", "HF", "HP", "I", "IB", "IP", "IR",
-       "LP", "ME", "MT", "OP", "P", "PD", "PP", "PT",
-       "R", "RB", "RE", "RI", "RS", "SB", "SH", "SM", "SS", "SY",
-       "TE", "TH", "TP", "TQ", "TS", "T&", "UC", "UE", "UR", "YS",
+       "AT", "B", "BI", "BR", "DT",
+       "EE", "EN", "EQ", "EX", "HP", "I", "IB", "IP", "IR",
+       "LP", "OP", "P", "PD", "PP",
+       "R", "RB", "RE", "RI", "RS", "SB", "SH", "SM", "SS",
+       "TE", "TH", "TP", "TS", "T&", "UC", "UE", "UR",
        NULL
 };
 
        NULL
 };
 
@@ -295,6 +312,10 @@ static     const struct predef predefs[PREDEFS_MAX] = {
 /* See roffhash_find() */
 #define        ROFF_HASH(p)    (p[0] - ASCII_LO)
 
 /* See roffhash_find() */
 #define        ROFF_HASH(p)    (p[0] - ASCII_LO)
 
+static int      roffit_lines;  /* number of lines to delay */
+static char    *roffit_macro;  /* nil-terminated macro line */
+
+
 static void
 roffhash_init(void)
 {
 static void
 roffhash_init(void)
 {
@@ -347,7 +368,6 @@ roffhash_find(const char *p, size_t s)
        return(ROFF_MAX);
 }
 
        return(ROFF_MAX);
 }
 
-
 /*
  * Pop the current node off of the stack of roff instructions currently
  * pending.
 /*
  * Pop the current node off of the stack of roff instructions currently
  * pending.
@@ -358,7 +378,7 @@ roffnode_pop(struct roff *r)
        struct roffnode *p;
 
        assert(r->last);
        struct roffnode *p;
 
        assert(r->last);
-       p = r->last; 
+       p = r->last;
 
        r->last = r->last->parent;
        free(p->name);
 
        r->last = r->last->parent;
        free(p->name);
@@ -366,7 +386,6 @@ roffnode_pop(struct roff *r)
        free(p);
 }
 
        free(p);
 }
 
-
 /*
  * Push a roff node onto the instruction stack.  This must later be
  * removed with roffnode_pop().
 /*
  * Push a roff node onto the instruction stack.  This must later be
  * removed with roffnode_pop().
@@ -384,22 +403,21 @@ roffnode_push(struct roff *r, enum rofft tok, const char *name,
        p->parent = r->last;
        p->line = line;
        p->col = col;
        p->parent = r->last;
        p->line = line;
        p->col = col;
-       p->rule = p->parent ? p->parent->rule : ROFFRULE_DENY;
+       p->rule = p->parent ? p->parent->rule : 0;
 
        r->last = p;
 }
 
 
        r->last = p;
 }
 
-
 static void
 roff_free1(struct roff *r)
 {
 static void
 roff_free1(struct roff *r)
 {
-       struct tbl_node *t;
+       struct tbl_node *tbl;
        struct eqn_node *e;
        int              i;
 
        struct eqn_node *e;
        int              i;
 
-       while (NULL != (t = r->first_tbl)) {
-               r->first_tbl = t->next;
-               tbl_free(t);
+       while (NULL != (tbl = r->first_tbl)) {
+               r->first_tbl = tbl->next;
+               tbl_free(tbl);
        }
 
        r->first_tbl = r->last_tbl = r->tbl = NULL;
        }
 
        r->first_tbl = r->last_tbl = r->tbl = NULL;
@@ -419,6 +437,10 @@ roff_free1(struct roff *r)
 
        r->strtab = r->xmbtab = NULL;
 
 
        r->strtab = r->xmbtab = NULL;
 
+       roff_freereg(r->regtab);
+
+       r->regtab = NULL;
+
        if (r->xtab)
                for (i = 0; i < 128; i++)
                        free(r->xtab[i].p);
        if (r->xtab)
                for (i = 0; i < 128; i++)
                        free(r->xtab[i].p);
@@ -430,18 +452,11 @@ roff_free1(struct roff *r)
 void
 roff_reset(struct roff *r)
 {
 void
 roff_reset(struct roff *r)
 {
-       int              i;
 
        roff_free1(r);
 
        roff_free1(r);
-
        r->control = 0;
        r->control = 0;
-       memset(&r->regs, 0, sizeof(struct reg) * REG__MAX);
-
-       for (i = 0; i < PREDEFS_MAX; i++) 
-               roff_setstr(r, predefs[i].name, predefs[i].str, 0);
 }
 
 }
 
-
 void
 roff_free(struct roff *r)
 {
 void
 roff_free(struct roff *r)
 {
@@ -450,109 +465,135 @@ roff_free(struct roff *r)
        free(r);
 }
 
        free(r);
 }
 
-
 struct roff *
 struct roff *
-roff_alloc(enum mparset type, struct mparse *parse)
+roff_alloc(struct mparse *parse, int options)
 {
        struct roff     *r;
 {
        struct roff     *r;
-       int              i;
 
        r = mandoc_calloc(1, sizeof(struct roff));
 
        r = mandoc_calloc(1, sizeof(struct roff));
-       r->parsetype = type;
        r->parse = parse;
        r->parse = parse;
+       r->options = options;
        r->rstackpos = -1;
        r->rstackpos = -1;
-       
-       roffhash_init();
 
 
-       for (i = 0; i < PREDEFS_MAX; i++) 
-               roff_setstr(r, predefs[i].name, predefs[i].str, 0);
+       roffhash_init();
 
        return(r);
 }
 
 /*
 
        return(r);
 }
 
 /*
- * Pre-filter each and every line for reserved words (one beginning with
- * `\*', e.g., `\*(ab').  These must be handled before the actual line
- * is processed. 
- * This also checks the syntax of regular escapes.
+ * In the current line, expand escape sequences that tend to get
+ * used in numerical expressions and conditional requests.
+ * Also check the syntax of the remaining escape sequences.
  */
 static enum rofferr
 roff_res(struct roff *r, char **bufp, size_t *szp, int ln, int pos)
 {
  */
 static enum rofferr
 roff_res(struct roff *r, char **bufp, size_t *szp, int ln, int pos)
 {
-       enum mandoc_esc  esc;
-       const char      *stesc; /* start of an escape sequence ('\\') */
+       char             ubuf[24]; /* buffer to print the number */
+       const char      *start; /* start of the string to process */
+       char            *stesc; /* start of an escape sequence ('\\') */
        const char      *stnam; /* start of the name, after "[(*" */
        const char      *cp;    /* end of the name, e.g. before ']' */
        const char      *res;   /* the string to be substituted */
        const char      *stnam; /* start of the name, after "[(*" */
        const char      *cp;    /* end of the name, e.g. before ']' */
        const char      *res;   /* the string to be substituted */
-       int              i, maxl, expand_count;
-       size_t           nsz;
-       char            *n;
+       char            *nbuf;  /* new buffer to copy bufp to */
+       size_t           maxl;  /* expected length of the escape name */
+       size_t           naml;  /* actual length of the escape name */
+       int              expand_count;  /* to avoid infinite loops */
+       int              npos;  /* position in numeric expression */
+       int              irc;   /* return code from roff_evalnum() */
+       char             term;  /* character terminating the escape */
 
        expand_count = 0;
 
        expand_count = 0;
+       start = *bufp + pos;
+       stesc = strchr(start, '\0') - 1;
+       while (stesc-- > start) {
 
 
-again:
-       cp = *bufp + pos;
-       while (NULL != (cp = strchr(cp, '\\'))) {
-               stesc = cp++;
+               /* Search backwards for the next backslash. */
 
 
-               /*
-                * The second character must be an asterisk.
-                * If it isn't, skip it anyway:  It is escaped,
-                * so it can't start another escape sequence.
-                */
+               if ('\\' != *stesc)
+                       continue;
+
+               /* If it is escaped, skip it. */
+
+               for (cp = stesc - 1; cp >= start; cp--)
+                       if ('\\' != *cp)
+                               break;
 
 
-               if ('\0' == *cp)
-                       return(ROFF_CONT);
-
-               if ('*' != *cp) {
-                       res = cp;
-                       esc = mandoc_escape(&cp, NULL, NULL);
-                       if (ESCAPE_ERROR != esc)
-                               continue;
-                       cp = res;
-                       mandoc_msg
-                               (MANDOCERR_BADESCAPE, r->parse, 
-                                ln, (int)(stesc - *bufp), NULL);
-                       return(ROFF_CONT);
+               if (0 == (stesc - cp) % 2) {
+                       stesc = (char *)cp;
+                       continue;
                }
 
                }
 
-               cp++;
+               /* Decide whether to expand or to check only. */
+
+               term = '\0';
+               cp = stesc + 1;
+               switch (*cp) {
+               case '*':
+                       res = NULL;
+                       break;
+               case 'B':
+                       /* FALLTHROUGH */
+               case 'w':
+                       term = cp[1];
+                       /* FALLTHROUGH */
+               case 'n':
+                       res = ubuf;
+                       break;
+               default:
+                       if (ESCAPE_ERROR == mandoc_escape(&cp, NULL, NULL))
+                               mandoc_msg(MANDOCERR_BADESCAPE, r->parse,
+                                   ln, (int)(stesc - *bufp), NULL);
+                       continue;
+               }
+
+               if (EXPAND_LIMIT < ++expand_count) {
+                       mandoc_msg(MANDOCERR_ROFFLOOP, r->parse,
+                           ln, (int)(stesc - *bufp), NULL);
+                       return(ROFF_IGN);
+               }
 
                /*
                 * The third character decides the length
 
                /*
                 * The third character decides the length
-                * of the name of the string.
+                * of the name of the string or register.
                 * Save a pointer to the name.
                 */
 
                 * Save a pointer to the name.
                 */
 
-               switch (*cp) {
-               case ('\0'):
-                       return(ROFF_CONT);
-               case ('('):
-                       cp++;
-                       maxl = 2;
-                       break;
-               case ('['):
-                       cp++;
+               if ('\0' == term) {
+                       switch (*++cp) {
+                       case '\0':
+                               maxl = 0;
+                               break;
+                       case '(':
+                               cp++;
+                               maxl = 2;
+                               break;
+                       case '[':
+                               cp++;
+                               term = ']';
+                               maxl = 0;
+                               break;
+                       default:
+                               maxl = 1;
+                               break;
+                       }
+               } else {
+                       cp += 2;
                        maxl = 0;
                        maxl = 0;
-                       break;
-               default:
-                       maxl = 1;
-                       break;
                }
                stnam = cp;
 
                /* Advance to the end of the name. */
 
                }
                stnam = cp;
 
                /* Advance to the end of the name. */
 
-               for (i = 0; 0 == maxl || i < maxl; i++, cp++) {
+               for (naml = 0; 0 == maxl || naml < maxl; naml++, cp++) {
                        if ('\0' == *cp) {
                        if ('\0' == *cp) {
-                               mandoc_msg
-                                       (MANDOCERR_BADESCAPE, 
-                                        r->parse, ln, 
-                                        (int)(stesc - *bufp), NULL);
-                               return(ROFF_CONT);
+                               mandoc_msg(MANDOCERR_BADESCAPE, r->parse,
+                                   ln, (int)(stesc - *bufp), NULL);
+                               break;
                        }
                        }
-                       if (0 == maxl && ']' == *cp)
+                       if (0 == maxl && *cp == term) {
+                               cp++;
                                break;
                                break;
+                       }
                }
 
                /*
                }
 
                /*
@@ -560,52 +601,64 @@ again:
                 * undefined, resume searching for escapes.
                 */
 
                 * undefined, resume searching for escapes.
                 */
 
-               res = roff_getstrn(r, stnam, (size_t)i);
+               switch (stesc[1]) {
+               case '*':
+                       res = roff_getstrn(r, stnam, naml);
+                       break;
+               case 'B':
+                       npos = 0;
+                       irc = roff_evalnum(stnam, &npos, NULL, 0);
+                       ubuf[0] = irc && stnam + npos + 1 == cp
+                           ? '1' : '0';
+                       ubuf[1] = '\0';
+                       break;
+               case 'n':
+                       (void)snprintf(ubuf, sizeof(ubuf), "%d",
+                           roff_getregn(r, stnam, naml));
+                       break;
+               case 'w':
+                       (void)snprintf(ubuf, sizeof(ubuf), "%d",
+                           24 * (int)naml);
+                       break;
+               }
 
                if (NULL == res) {
 
                if (NULL == res) {
-                       mandoc_msg
-                               (MANDOCERR_BADESCAPE, r->parse, 
-                                ln, (int)(stesc - *bufp), NULL);
+                       mandoc_msg(MANDOCERR_BADESCAPE, r->parse,
+                           ln, (int)(stesc - *bufp), NULL);
                        res = "";
                }
 
                /* Replace the escape sequence by the string. */
 
                        res = "";
                }
 
                /* Replace the escape sequence by the string. */
 
-               pos = stesc - *bufp;
-
-               nsz = *szp + strlen(res) + 1;
-               n = mandoc_malloc(nsz);
+               *stesc = '\0';
+               *szp = mandoc_asprintf(&nbuf, "%s%s%s",
+                   *bufp, res, cp) + 1;
 
 
-               strlcpy(n, *bufp, (size_t)(stesc - *bufp + 1));
-               strlcat(n, res, nsz);
-               strlcat(n, cp + (maxl ? 0 : 1), nsz);
+               /* Prepare for the next replacement. */
 
 
+               start = nbuf + pos;
+               stesc = nbuf + (stesc - *bufp) + strlen(res);
                free(*bufp);
                free(*bufp);
-
-               *bufp = n;
-               *szp = nsz;
-
-               if (EXPAND_LIMIT >= ++expand_count)
-                       goto again;
-
-               /* Just leave the string unexpanded. */
-               mandoc_msg(MANDOCERR_ROFFLOOP, r->parse, ln, pos, NULL);
-               return(ROFF_IGN);
+               *bufp = nbuf;
        }
        return(ROFF_CONT);
 }
 
 /*
        }
        return(ROFF_CONT);
 }
 
 /*
- * Process text streams: convert all breakable hyphens into ASCII_HYPH.
+ * Process text streams:
+ * Convert all breakable hyphens into ASCII_HYPH.
+ * Decrement and spring input line trap.
  */
 static enum rofferr
  */
 static enum rofferr
-roff_parsetext(char *p)
+roff_parsetext(char **bufp, size_t *szp, int pos, int *offs)
 {
        size_t           sz;
        const char      *start;
 {
        size_t           sz;
        const char      *start;
+       char            *p;
+       int              isz;
        enum mandoc_esc  esc;
 
        enum mandoc_esc  esc;
 
-       start = p;
+       start = p = *bufp + pos;
 
        while ('\0' != *p) {
                sz = strcspn(p, "-\\");
 
        while ('\0' != *p) {
                sz = strcspn(p, "-\\");
@@ -617,8 +670,7 @@ roff_parsetext(char *p)
                if ('\\' == *p) {
                        /* Skip over escapes. */
                        p++;
                if ('\\' == *p) {
                        /* Skip over escapes. */
                        p++;
-                       esc = mandoc_escape
-                               ((const char **)&p, NULL, NULL);
+                       esc = mandoc_escape((const char **)&p, NULL, NULL);
                        if (ESCAPE_ERROR == esc)
                                break;
                        continue;
                        if (ESCAPE_ERROR == esc)
                                break;
                        continue;
@@ -633,11 +685,23 @@ roff_parsetext(char *p)
                p++;
        }
 
                p++;
        }
 
+       /* Spring the input line trap. */
+       if (1 == roffit_lines) {
+               isz = mandoc_asprintf(&p, "%s\n.%s", *bufp, roffit_macro);
+               free(*bufp);
+               *bufp = p;
+               *szp = isz + 1;
+               *offs = 0;
+               free(roffit_macro);
+               roffit_lines = 0;
+               return(ROFF_REPARSE);
+       } else if (1 < roffit_lines)
+               --roffit_lines;
        return(ROFF_CONT);
 }
 
 enum rofferr
        return(ROFF_CONT);
 }
 
 enum rofferr
-roff_parseln(struct roff *r, int ln, char **bufp, 
+roff_parseln(struct roff *r, int ln, char **bufp,
                size_t *szp, int pos, int *offs)
 {
        enum rofft       t;
                size_t *szp, int pos, int *offs)
 {
        enum rofft       t;
@@ -668,24 +732,18 @@ roff_parseln(struct roff *r, int ln, char **bufp,
        if (r->last && ! ctl) {
                t = r->last->tok;
                assert(roffs[t].text);
        if (r->last && ! ctl) {
                t = r->last->tok;
                assert(roffs[t].text);
-               e = (*roffs[t].text)
-                       (r, t, bufp, szp, ln, pos, pos, offs);
+               e = (*roffs[t].text)(r, t, bufp, szp, ln, pos, pos, offs);
                assert(ROFF_IGN == e || ROFF_CONT == e);
                if (ROFF_CONT != e)
                        return(e);
                assert(ROFF_IGN == e || ROFF_CONT == e);
                if (ROFF_CONT != e)
                        return(e);
-               if (r->eqn)
-                       return(eqn_read(&r->eqn, ln, *bufp, pos, offs));
-               if (r->tbl)
-                       return(tbl_read(r->tbl, ln, *bufp, pos));
-               return(roff_parsetext(*bufp + pos));
-       } else if ( ! ctl) {
-               if (r->eqn)
-                       return(eqn_read(&r->eqn, ln, *bufp, pos, offs));
+       }
+       if (r->eqn)
+               return(eqn_read(&r->eqn, ln, *bufp, ppos, offs));
+       if ( ! ctl) {
                if (r->tbl)
                        return(tbl_read(r->tbl, ln, *bufp, pos));
                if (r->tbl)
                        return(tbl_read(r->tbl, ln, *bufp, pos));
-               return(roff_parsetext(*bufp + pos));
-       } else if (r->eqn)
-               return(eqn_read(&r->eqn, ln, *bufp, ppos, offs));
+               return(roff_parsetext(bufp, szp, pos, offs));
+       }
 
        /*
         * If a scope is open, go to the child handler for that macro,
 
        /*
         * If a scope is open, go to the child handler for that macro,
@@ -696,9 +754,8 @@ roff_parseln(struct roff *r, int ln, char **bufp,
        if (r->last) {
                t = r->last->tok;
                assert(roffs[t].sub);
        if (r->last) {
                t = r->last->tok;
                assert(roffs[t].sub);
-               return((*roffs[t].sub)
-                               (r, t, bufp, szp, 
-                                ln, ppos, pos, offs));
+               return((*roffs[t].sub)(r, t, bufp, szp,
+                   ln, ppos, pos, offs));
        }
 
        /*
        }
 
        /*
@@ -711,29 +768,26 @@ roff_parseln(struct roff *r, int ln, char **bufp,
                return(ROFF_CONT);
 
        assert(roffs[t].proc);
                return(ROFF_CONT);
 
        assert(roffs[t].proc);
-       return((*roffs[t].proc)
-                       (r, t, bufp, szp, 
-                        ln, ppos, pos, offs));
+       return((*roffs[t].proc)(r, t, bufp, szp, ln, ppos, pos, offs));
 }
 
 }
 
-
 void
 roff_endparse(struct roff *r)
 {
 
        if (r->last)
                mandoc_msg(MANDOCERR_SCOPEEXIT, r->parse,
 void
 roff_endparse(struct roff *r)
 {
 
        if (r->last)
                mandoc_msg(MANDOCERR_SCOPEEXIT, r->parse,
-                               r->last->line, r->last->col, NULL);
+                   r->last->line, r->last->col, NULL);
 
        if (r->eqn) {
 
        if (r->eqn) {
-               mandoc_msg(MANDOCERR_SCOPEEXIT, r->parse, 
-                               r->eqn->eqn.ln, r->eqn->eqn.pos, NULL);
+               mandoc_msg(MANDOCERR_SCOPEEXIT, r->parse,
+                   r->eqn->eqn.ln, r->eqn->eqn.pos, NULL);
                eqn_end(&r->eqn);
        }
 
        if (r->tbl) {
                eqn_end(&r->eqn);
        }
 
        if (r->tbl) {
-               mandoc_msg(MANDOCERR_SCOPEEXIT, r->parse, 
-                               r->tbl->line, r->tbl->pos, NULL);
+               mandoc_msg(MANDOCERR_SCOPEEXIT, r->parse,
+                   r->tbl->line, r->tbl->pos, NULL);
                tbl_end(&r->tbl);
        }
 }
                tbl_end(&r->tbl);
        }
 }
@@ -749,18 +803,14 @@ roff_parse(struct roff *r, const char *buf, int *pos)
        size_t           maclen;
        enum rofft       t;
 
        size_t           maclen;
        enum rofft       t;
 
-       if ('\0' == buf[*pos] || '"' == buf[*pos] || 
-                       '\t' == buf[*pos] || ' ' == buf[*pos])
+       if ('\0' == buf[*pos] || '"' == buf[*pos] ||
+           '\t' == buf[*pos] || ' ' == buf[*pos])
                return(ROFF_MAX);
 
                return(ROFF_MAX);
 
-       /*
-        * We stop the macro parse at an escape, tab, space, or nil.
-        * However, `\}' is also a valid macro, so make sure we don't
-        * clobber it by seeing the `\' as the end of token.
-        */
+       /* We stop the macro parse at an escape, tab, space, or nil. */
 
        mac = buf + *pos;
 
        mac = buf + *pos;
-       maclen = strcspn(mac + 1, " \\\t\0") + 1;
+       maclen = strcspn(mac, " \\\t\0");
 
        t = (r->current_string = roff_getstrn(r, mac, maclen))
            ? ROFF_USERDEF : roffhash_find(mac, maclen);
 
        t = (r->current_string = roff_getstrn(r, mac, maclen))
            ? ROFF_USERDEF : roffhash_find(mac, maclen);
@@ -773,7 +823,6 @@ roff_parse(struct roff *r, const char *buf, int *pos)
        return(t);
 }
 
        return(t);
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_cblock(ROFF_ARGS)
 {
 static enum rofferr
 roff_cblock(ROFF_ARGS)
 {
@@ -789,18 +838,18 @@ roff_cblock(ROFF_ARGS)
        }
 
        switch (r->last->tok) {
        }
 
        switch (r->last->tok) {
-       case (ROFF_am):
+       case ROFF_am:
                /* FALLTHROUGH */
                /* FALLTHROUGH */
-       case (ROFF_ami):
+       case ROFF_ami:
                /* FALLTHROUGH */
                /* FALLTHROUGH */
-       case (ROFF_am1):
+       case ROFF_am1:
                /* FALLTHROUGH */
                /* FALLTHROUGH */
-       case (ROFF_de):
+       case ROFF_de:
                /* ROFF_de1 is remapped to ROFF_de in roff_block(). */
                /* FALLTHROUGH */
                /* ROFF_de1 is remapped to ROFF_de in roff_block(). */
                /* FALLTHROUGH */
-       case (ROFF_dei):
+       case ROFF_dei:
                /* FALLTHROUGH */
                /* FALLTHROUGH */
-       case (ROFF_ig):
+       case ROFF_ig:
                break;
        default:
                mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
                break;
        default:
                mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
@@ -816,7 +865,6 @@ roff_cblock(ROFF_ARGS)
 
 }
 
 
 }
 
-
 static void
 roffnode_cleanscope(struct roff *r)
 {
 static void
 roffnode_cleanscope(struct roff *r)
 {
@@ -828,44 +876,37 @@ roffnode_cleanscope(struct roff *r)
        }
 }
 
        }
 }
 
-
-/* ARGSUSED */
-static enum rofferr
-roff_ccond(ROFF_ARGS)
+static void
+roff_ccond(struct roff *r, int ln, int ppos)
 {
 
        if (NULL == r->last) {
                mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
 {
 
        if (NULL == r->last) {
                mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
-               return(ROFF_IGN);
+               return;
        }
 
        switch (r->last->tok) {
        }
 
        switch (r->last->tok) {
-       case (ROFF_el):
+       case ROFF_el:
                /* FALLTHROUGH */
                /* FALLTHROUGH */
-       case (ROFF_ie):
+       case ROFF_ie:
                /* FALLTHROUGH */
                /* FALLTHROUGH */
-       case (ROFF_if):
+       case ROFF_if:
                break;
        default:
                mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
                break;
        default:
                mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
-               return(ROFF_IGN);
+               return;
        }
 
        if (r->last->endspan > -1) {
                mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
        }
 
        if (r->last->endspan > -1) {
                mandoc_msg(MANDOCERR_NOSCOPE, r->parse, ln, ppos, NULL);
-               return(ROFF_IGN);
+               return;
        }
 
        }
 
-       if ((*bufp)[pos])
-               mandoc_msg(MANDOCERR_ARGSLOST, r->parse, ln, pos, NULL);
-
        roffnode_pop(r);
        roffnode_cleanscope(r);
        roffnode_pop(r);
        roffnode_cleanscope(r);
-       return(ROFF_IGN);
+       return;
 }
 
 }
 
-
-/* ARGSUSED */
 static enum rofferr
 roff_block(ROFF_ARGS)
 {
 static enum rofferr
 roff_block(ROFF_ARGS)
 {
@@ -906,7 +947,7 @@ roff_block(ROFF_ARGS)
        /*
         * At the beginning of a `de' macro, clear the existing string
         * with the same name, if there is one.  New content will be
        /*
         * At the beginning of a `de' macro, clear the existing string
         * with the same name, if there is one.  New content will be
-        * added from roff_block_text() in multiline mode.
+        * appended from roff_block_text() in multiline mode.
         */
 
        if (ROFF_de == tok)
         */
 
        if (ROFF_de == tok)
@@ -944,8 +985,6 @@ roff_block(ROFF_ARGS)
        return(ROFF_IGN);
 }
 
        return(ROFF_IGN);
 }
 
-
-/* ARGSUSED */
 static enum rofferr
 roff_block_sub(ROFF_ARGS)
 {
 static enum rofferr
 roff_block_sub(ROFF_ARGS)
 {
@@ -966,10 +1005,10 @@ roff_block_sub(ROFF_ARGS)
                        if ((*bufp)[i] != r->last->end[j])
                                break;
 
                        if ((*bufp)[i] != r->last->end[j])
                                break;
 
-               if ('\0' == r->last->end[j] && 
-                               ('\0' == (*bufp)[i] ||
-                                ' ' == (*bufp)[i] ||
-                                '\t' == (*bufp)[i])) {
+               if ('\0' == r->last->end[j] &&
+                   ('\0' == (*bufp)[i] ||
+                    ' '  == (*bufp)[i] ||
+                    '\t' == (*bufp)[i])) {
                        roffnode_pop(r);
                        roffnode_cleanscope(r);
 
                        roffnode_pop(r);
                        roffnode_cleanscope(r);
 
@@ -996,167 +1035,228 @@ roff_block_sub(ROFF_ARGS)
         */
        if (ROFF_cblock != t) {
                if (ROFF_de == tok)
         */
        if (ROFF_cblock != t) {
                if (ROFF_de == tok)
-                       roff_setstr(r, r->last->name, *bufp + ppos, 1);
+                       roff_setstr(r, r->last->name, *bufp + ppos, 2);
                return(ROFF_IGN);
        }
 
        assert(roffs[t].proc);
                return(ROFF_IGN);
        }
 
        assert(roffs[t].proc);
-       return((*roffs[t].proc)(r, t, bufp, szp, 
-                               ln, ppos, pos, offs));
+       return((*roffs[t].proc)(r, t, bufp, szp, ln, ppos, pos, offs));
 }
 
 }
 
-
-/* ARGSUSED */
 static enum rofferr
 roff_block_text(ROFF_ARGS)
 {
 
        if (ROFF_de == tok)
 static enum rofferr
 roff_block_text(ROFF_ARGS)
 {
 
        if (ROFF_de == tok)
-               roff_setstr(r, r->last->name, *bufp + pos, 1);
+               roff_setstr(r, r->last->name, *bufp + pos, 2);
 
        return(ROFF_IGN);
 }
 
 
        return(ROFF_IGN);
 }
 
-
-/* ARGSUSED */
 static enum rofferr
 roff_cond_sub(ROFF_ARGS)
 {
        enum rofft       t;
 static enum rofferr
 roff_cond_sub(ROFF_ARGS)
 {
        enum rofft       t;
-       enum roffrule    rr;
        char            *ep;
        char            *ep;
+       int              rr;
 
        rr = r->last->rule;
        roffnode_cleanscope(r);
 
        rr = r->last->rule;
        roffnode_cleanscope(r);
+       t = roff_parse(r, *bufp, &pos);
 
        /*
 
        /*
-        * If the macro is unknown, first check if it contains a closing
-        * delimiter `\}'.  If it does, close out our scope and return
-        * the currently-scoped rule (ignore or continue).  Else, drop
-        * into the currently-scoped rule.
+        * Fully handle known macros when they are structurally
+        * required or when the conditional evaluated to true.
         */
 
         */
 
-       if (ROFF_MAX == (t = roff_parse(r, *bufp, &pos))) {
-               ep = &(*bufp)[pos];
-               for ( ; NULL != (ep = strchr(ep, '\\')); ep++) {
-                       ep++;
-                       if ('}' != *ep)
-                               continue;
-
-                       /*
-                        * Make the \} go away.
-                        * This is a little haphazard, as it's not quite
-                        * clear how nroff does this.
-                        * If we're at the end of line, then just chop
-                        * off the \} and resize the buffer.
-                        * If we aren't, then conver it to spaces.
-                        */
-
-                       if ('\0' == *(ep + 1)) {
-                               *--ep = '\0';
-                               *szp -= 2;
-                       } else
-                               *(ep - 1) = *ep = ' ';
-
-                       roff_ccond(r, ROFF_ccond, bufp, szp, 
-                                       ln, pos, pos + 2, offs);
-                       break;
-               }
-               return(ROFFRULE_DENY == rr ? ROFF_IGN : ROFF_CONT);
+       if ((ROFF_MAX != t) &&
+           (rr || ROFFMAC_STRUCT & roffs[t].flags)) {
+               assert(roffs[t].proc);
+               return((*roffs[t].proc)(r, t, bufp, szp,
+                   ln, ppos, pos, offs));
        }
 
        /*
        }
 
        /*
-        * A denied conditional must evaluate its children if and only
-        * if they're either structurally required (such as loops and
-        * conditionals) or a closing macro.
+        * If `\}' occurs on a macro line without a preceding macro,
+        * drop the line completely.
         */
 
         */
 
-       if (ROFFRULE_DENY == rr)
-               if ( ! (ROFFMAC_STRUCT & roffs[t].flags))
-                       if (ROFF_ccond != t)
-                               return(ROFF_IGN);
+       ep = *bufp + pos;
+       if ('\\' == ep[0] && '}' == ep[1])
+               rr = 0;
 
 
-       assert(roffs[t].proc);
-       return((*roffs[t].proc)(r, t, bufp, szp, 
-                               ln, ppos, pos, offs));
+       /* Always check for the closing delimiter `\}'. */
+
+       while (NULL != (ep = strchr(ep, '\\'))) {
+               if ('}' == *(++ep)) {
+                       *ep = '&';
+                       roff_ccond(r, ln, ep - *bufp - 1);
+               }
+               ++ep;
+       }
+       return(rr ? ROFF_CONT : ROFF_IGN);
 }
 
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_cond_text(ROFF_ARGS)
 {
        char            *ep;
 static enum rofferr
 roff_cond_text(ROFF_ARGS)
 {
        char            *ep;
-       enum roffrule    rr;
+       int              rr;
 
        rr = r->last->rule;
        roffnode_cleanscope(r);
 
 
        rr = r->last->rule;
        roffnode_cleanscope(r);
 
-       ep = &(*bufp)[pos];
-       for ( ; NULL != (ep = strchr(ep, '\\')); ep++) {
-               ep++;
-               if ('}' != *ep)
-                       continue;
-               *ep = '&';
-               roff_ccond(r, ROFF_ccond, bufp, szp, 
-                               ln, pos, pos + 2, offs);
+       ep = *bufp + pos;
+       while (NULL != (ep = strchr(ep, '\\'))) {
+               if ('}' == *(++ep)) {
+                       *ep = '&';
+                       roff_ccond(r, ln, ep - *bufp - 1);
+               }
+               ++ep;
+       }
+       return(rr ? ROFF_CONT : ROFF_IGN);
+}
+
+/*
+ * Parse a single signed integer number.  Stop at the first non-digit.
+ * If there is at least one digit, return success and advance the
+ * parse point, else return failure and let the parse point unchanged.
+ * Ignore overflows, treat them just like the C language.
+ */
+static int
+roff_getnum(const char *v, int *pos, int *res)
+{
+       int      myres, n, p;
+
+       if (NULL == res)
+               res = &myres;
+
+       p = *pos;
+       n = v[p] == '-';
+       if (n)
+               p++;
+
+       for (*res = 0; isdigit((unsigned char)v[p]); p++)
+               *res = 10 * *res + v[p] - '0';
+       if (p == *pos + n)
+               return 0;
+
+       if (n)
+               *res = -*res;
+
+       *pos = p;
+       return 1;
+}
+
+/*
+ * Evaluate a string comparison condition.
+ * The first character is the delimiter.
+ * Succeed if the string up to its second occurrence
+ * matches the string up to its third occurence.
+ * Advance the cursor after the third occurrence
+ * or lacking that, to the end of the line.
+ */
+static int
+roff_evalstrcond(const char *v, int *pos)
+{
+       const char      *s1, *s2, *s3;
+       int              match;
+
+       match = 0;
+       s1 = v + *pos;          /* initial delimiter */
+       s2 = s1 + 1;            /* for scanning the first string */
+       s3 = strchr(s2, *s1);   /* for scanning the second string */
+
+       if (NULL == s3)         /* found no middle delimiter */
+               goto out;
+
+       while ('\0' != *++s3) {
+               if (*s2 != *s3) {  /* mismatch */
+                       s3 = strchr(s3, *s1);
+                       break;
+               }
+               if (*s3 == *s1) {  /* found the final delimiter */
+                       match = 1;
+                       break;
+               }
+               s2++;
        }
        }
-       return(ROFFRULE_DENY == rr ? ROFF_IGN : ROFF_CONT);
+
+out:
+       if (NULL == s3)
+               s3 = strchr(s2, '\0');
+       else
+               s3++;
+       *pos = s3 - v;
+       return(match);
 }
 
 }
 
-static enum roffrule
+/*
+ * Evaluate an optionally negated single character, numerical,
+ * or string condition.
+ */
+static int
 roff_evalcond(const char *v, int *pos)
 {
 roff_evalcond(const char *v, int *pos)
 {
+       int      wanttrue, number;
+
+       if ('!' == v[*pos]) {
+               wanttrue = 0;
+               (*pos)++;
+       } else
+               wanttrue = 1;
 
        switch (v[*pos]) {
 
        switch (v[*pos]) {
-       case ('n'):
+       case 'n':
+               /* FALLTHROUGH */
+       case 'o':
                (*pos)++;
                (*pos)++;
-               return(ROFFRULE_ALLOW);
-       case ('e'):
+               return(wanttrue);
+       case 'c':
+               /* FALLTHROUGH */
+       case 'd':
                /* FALLTHROUGH */
                /* FALLTHROUGH */
-       case ('o'):
+       case 'e':
                /* FALLTHROUGH */
                /* FALLTHROUGH */
-       case ('t'):
+       case 'r':
+               /* FALLTHROUGH */
+       case 't':
                (*pos)++;
                (*pos)++;
-               return(ROFFRULE_DENY);
+               return(!wanttrue);
        default:
                break;
        }
 
        default:
                break;
        }
 
-       while (v[*pos] && ' ' != v[*pos])
-               (*pos)++;
-       return(ROFFRULE_DENY);
+       if (roff_evalnum(v, pos, &number, 0))
+               return((number > 0) == wanttrue);
+       else
+               return(roff_evalstrcond(v, pos) == wanttrue);
 }
 
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_line_ignore(ROFF_ARGS)
 {
 
 static enum rofferr
 roff_line_ignore(ROFF_ARGS)
 {
 
-       if (ROFF_it == tok)
-               mandoc_msg(MANDOCERR_REQUEST, r->parse, ln, ppos, "it");
-
        return(ROFF_IGN);
 }
 
        return(ROFF_IGN);
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_cond(ROFF_ARGS)
 {
 
        roffnode_push(r, tok, NULL, ln, ppos);
 
 static enum rofferr
 roff_cond(ROFF_ARGS)
 {
 
        roffnode_push(r, tok, NULL, ln, ppos);
 
-       /* 
+       /*
         * An `.el' has no conditional body: it will consume the value
         * of the current rstack entry set in prior `ie' calls or
         * An `.el' has no conditional body: it will consume the value
         * of the current rstack entry set in prior `ie' calls or
-        * defaults to DENY.  
+        * defaults to DENY.
         *
         * If we're not an `el', however, then evaluate the conditional.
         */
 
        r->last->rule = ROFF_el == tok ?
         *
         * If we're not an `el', however, then evaluate the conditional.
         */
 
        r->last->rule = ROFF_el == tok ?
-               (r->rstackpos < 0 ? 
-                ROFFRULE_DENY : r->rstack[r->rstackpos--]) :
-               roff_evalcond(*bufp, &pos);
+           (r->rstackpos < 0 ? 0 : r->rstack[r->rstackpos--]) :
+           roff_evalcond(*bufp, &pos);
 
        /*
         * An if-else will put the NEGATION of the current evaluated
 
        /*
         * An if-else will put the NEGATION of the current evaluated
@@ -1165,19 +1265,17 @@ roff_cond(ROFF_ARGS)
 
        if (ROFF_ie == tok) {
                if (r->rstackpos == RSTACK_MAX - 1) {
 
        if (ROFF_ie == tok) {
                if (r->rstackpos == RSTACK_MAX - 1) {
-                       mandoc_msg(MANDOCERR_MEM, 
-                               r->parse, ln, ppos, NULL);
+                       mandoc_msg(MANDOCERR_MEM,
+                           r->parse, ln, ppos, NULL);
                        return(ROFF_ERR);
                }
                        return(ROFF_ERR);
                }
-               r->rstack[++r->rstackpos] = 
-                       ROFFRULE_DENY == r->last->rule ?
-                       ROFFRULE_ALLOW : ROFFRULE_DENY;
+               r->rstack[++r->rstackpos] = !r->last->rule;
        }
 
        /* If the parent has false as its rule, then so do we. */
 
        }
 
        /* If the parent has false as its rule, then so do we. */
 
-       if (r->last->parent && ROFFRULE_DENY == r->last->parent->rule)
-               r->last->rule = ROFFRULE_DENY;
+       if (r->last->parent && !r->last->parent->rule)
+               r->last->rule = 0;
 
        /*
         * Determine scope.
 
        /*
         * Determine scope.
@@ -1199,7 +1297,7 @@ roff_cond(ROFF_ARGS)
                r->last->endspan = -1;
                pos += 2;
                goto out;
                r->last->endspan = -1;
                pos += 2;
                goto out;
-       } 
+       }
 
        /*
         * Anything else following the conditional causes
 
        /*
         * Anything else following the conditional causes
@@ -1217,8 +1315,6 @@ out:
        return(ROFF_RERUN);
 }
 
        return(ROFF_RERUN);
 }
 
-
-/* ARGSUSED */
 static enum rofferr
 roff_ds(ROFF_ARGS)
 {
 static enum rofferr
 roff_ds(ROFF_ARGS)
 {
@@ -1244,54 +1340,363 @@ roff_ds(ROFF_ARGS)
                string++;
 
        /* The rest is the value. */
                string++;
 
        /* The rest is the value. */
-       roff_setstr(r, name, string, 0);
+       roff_setstr(r, name, string, ROFF_as == tok);
        return(ROFF_IGN);
 }
 
        return(ROFF_IGN);
 }
 
-int
-roff_regisset(const struct roff *r, enum regs reg)
+/*
+ * Parse a single operator, one or two characters long.
+ * If the operator is recognized, return success and advance the
+ * parse point, else return failure and let the parse point unchanged.
+ */
+static int
+roff_getop(const char *v, int *pos, char *res)
 {
 
 {
 
-       return(r->regs[(int)reg].set);
+       *res = v[*pos];
+
+       switch (*res) {
+       case '+':
+               /* FALLTHROUGH */
+       case '-':
+               /* FALLTHROUGH */
+       case '*':
+               /* FALLTHROUGH */
+       case '/':
+               /* FALLTHROUGH */
+       case '%':
+               /* FALLTHROUGH */
+       case '&':
+               /* FALLTHROUGH */
+       case ':':
+               break;
+       case '<':
+               switch (v[*pos + 1]) {
+               case '=':
+                       *res = 'l';
+                       (*pos)++;
+                       break;
+               case '>':
+                       *res = '!';
+                       (*pos)++;
+                       break;
+               case '?':
+                       *res = 'i';
+                       (*pos)++;
+                       break;
+               default:
+                       break;
+               }
+               break;
+       case '>':
+               switch (v[*pos + 1]) {
+               case '=':
+                       *res = 'g';
+                       (*pos)++;
+                       break;
+               case '?':
+                       *res = 'a';
+                       (*pos)++;
+                       break;
+               default:
+                       break;
+               }
+               break;
+       case '=':
+               if ('=' == v[*pos + 1])
+                       (*pos)++;
+               break;
+       default:
+               return(0);
+       }
+       (*pos)++;
+
+       return(*res);
 }
 
 }
 
-unsigned int
-roff_regget(const struct roff *r, enum regs reg)
+/*
+ * Evaluate either a parenthesized numeric expression
+ * or a single signed integer number.
+ */
+static int
+roff_evalpar(const char *v, int *pos, int *res)
 {
 
 {
 
-       return(r->regs[(int)reg].u);
+       if ('(' != v[*pos])
+               return(roff_getnum(v, pos, res));
+
+       (*pos)++;
+       if ( ! roff_evalnum(v, pos, res, 1))
+               return(0);
+
+       /*
+        * Omission of the closing parenthesis
+        * is an error in validation mode,
+        * but ignored in evaluation mode.
+        */
+
+       if (')' == v[*pos])
+               (*pos)++;
+       else if (NULL == res)
+               return(0);
+
+       return(1);
+}
+
+/*
+ * Evaluate a complete numeric expression.
+ * Proceed left to right, there is no concept of precedence.
+ */
+static int
+roff_evalnum(const char *v, int *pos, int *res, int skipwhite)
+{
+       int              mypos, operand2;
+       char             operator;
+
+       if (NULL == pos) {
+               mypos = 0;
+               pos = &mypos;
+       }
+
+       if (skipwhite)
+               while (isspace((unsigned char)v[*pos]))
+                       (*pos)++;
+
+       if ( ! roff_evalpar(v, pos, res))
+               return(0);
+
+       while (1) {
+               if (skipwhite)
+                       while (isspace((unsigned char)v[*pos]))
+                               (*pos)++;
+
+               if ( ! roff_getop(v, pos, &operator))
+                       break;
+
+               if (skipwhite)
+                       while (isspace((unsigned char)v[*pos]))
+                               (*pos)++;
+
+               if ( ! roff_evalpar(v, pos, &operand2))
+                       return(0);
+
+               if (skipwhite)
+                       while (isspace((unsigned char)v[*pos]))
+                               (*pos)++;
+
+               if (NULL == res)
+                       continue;
+
+               switch (operator) {
+               case '+':
+                       *res += operand2;
+                       break;
+               case '-':
+                       *res -= operand2;
+                       break;
+               case '*':
+                       *res *= operand2;
+                       break;
+               case '/':
+                       *res /= operand2;
+                       break;
+               case '%':
+                       *res %= operand2;
+                       break;
+               case '<':
+                       *res = *res < operand2;
+                       break;
+               case '>':
+                       *res = *res > operand2;
+                       break;
+               case 'l':
+                       *res = *res <= operand2;
+                       break;
+               case 'g':
+                       *res = *res >= operand2;
+                       break;
+               case '=':
+                       *res = *res == operand2;
+                       break;
+               case '!':
+                       *res = *res != operand2;
+                       break;
+               case '&':
+                       *res = *res && operand2;
+                       break;
+               case ':':
+                       *res = *res || operand2;
+                       break;
+               case 'i':
+                       if (operand2 < *res)
+                               *res = operand2;
+                       break;
+               case 'a':
+                       if (operand2 > *res)
+                               *res = operand2;
+                       break;
+               default:
+                       abort();
+               }
+       }
+       return(1);
 }
 
 void
 }
 
 void
-roff_regunset(struct roff *r, enum regs reg)
+roff_setreg(struct roff *r, const char *name, int val, char sign)
+{
+       struct roffreg  *reg;
+
+       /* Search for an existing register with the same name. */
+       reg = r->regtab;
+
+       while (reg && strcmp(name, reg->key.p))
+               reg = reg->next;
+
+       if (NULL == reg) {
+               /* Create a new register. */
+               reg = mandoc_malloc(sizeof(struct roffreg));
+               reg->key.p = mandoc_strdup(name);
+               reg->key.sz = strlen(name);
+               reg->val = 0;
+               reg->next = r->regtab;
+               r->regtab = reg;
+       }
+
+       if ('+' == sign)
+               reg->val += val;
+       else if ('-' == sign)
+               reg->val -= val;
+       else
+               reg->val = val;
+}
+
+/*
+ * Handle some predefined read-only number registers.
+ * For now, return -1 if the requested register is not predefined;
+ * in case a predefined read-only register having the value -1
+ * were to turn up, another special value would have to be chosen.
+ */
+static int
+roff_getregro(const char *name)
 {
 
 {
 
-       r->regs[(int)reg].set = 0;
+       switch (*name) {
+       case 'A':  /* ASCII approximation mode is always off. */
+               return(0);
+       case 'g':  /* Groff compatibility mode is always on. */
+               return(1);
+       case 'H':  /* Fixed horizontal resolution. */
+               return (24);
+       case 'j':  /* Always adjust left margin only. */
+               return(0);
+       case 'T':  /* Some output device is always defined. */
+               return(1);
+       case 'V':  /* Fixed vertical resolution. */
+               return (40);
+       default:
+               return (-1);
+       }
+}
+
+int
+roff_getreg(const struct roff *r, const char *name)
+{
+       struct roffreg  *reg;
+       int              val;
+
+       if ('.' == name[0] && '\0' != name[1] && '\0' == name[2]) {
+               val = roff_getregro(name + 1);
+               if (-1 != val)
+                       return (val);
+       }
+
+       for (reg = r->regtab; reg; reg = reg->next)
+               if (0 == strcmp(name, reg->key.p))
+                       return(reg->val);
+
+       return(0);
+}
+
+static int
+roff_getregn(const struct roff *r, const char *name, size_t len)
+{
+       struct roffreg  *reg;
+       int              val;
+
+       if ('.' == name[0] && 2 == len) {
+               val = roff_getregro(name + 1);
+               if (-1 != val)
+                       return (val);
+       }
+
+       for (reg = r->regtab; reg; reg = reg->next)
+               if (len == reg->key.sz &&
+                   0 == strncmp(name, reg->key.p, len))
+                       return(reg->val);
+
+       return(0);
+}
+
+static void
+roff_freereg(struct roffreg *reg)
+{
+       struct roffreg  *old_reg;
+
+       while (NULL != reg) {
+               free(reg->key.p);
+               old_reg = reg;
+               reg = reg->next;
+               free(old_reg);
+       }
 }
 
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_nr(ROFF_ARGS)
 {
        const char      *key;
        char            *val;
        int              iv;
 static enum rofferr
 roff_nr(ROFF_ARGS)
 {
        const char      *key;
        char            *val;
        int              iv;
+       char             sign;
 
        val = *bufp + pos;
        key = roff_getname(r, &val, ln, pos);
 
 
        val = *bufp + pos;
        key = roff_getname(r, &val, ln, pos);
 
-       if (0 == strcmp(key, "nS")) {
-               r->regs[(int)REG_nS].set = 1;
-               if ((iv = mandoc_strntoi(val, strlen(val), 10)) >= 0)
-                       r->regs[(int)REG_nS].u = (unsigned)iv;
-               else
-                       r->regs[(int)REG_nS].u = 0u;
-       }
+       sign = *val;
+       if ('+' == sign || '-' == sign)
+               val++;
+
+       if (roff_evalnum(val, NULL, &iv, 0))
+               roff_setreg(r, key, iv, sign);
 
        return(ROFF_IGN);
 }
 
 
        return(ROFF_IGN);
 }
 
-/* ARGSUSED */
+static enum rofferr
+roff_rr(ROFF_ARGS)
+{
+       struct roffreg  *reg, **prev;
+       const char      *name;
+       char            *cp;
+
+       cp = *bufp + pos;
+       name = roff_getname(r, &cp, ln, pos);
+
+       prev = &r->regtab;
+       while (1) {
+               reg = *prev;
+               if (NULL == reg || !strcmp(name, reg->key.p))
+                       break;
+               prev = &reg->next;
+       }
+       if (NULL != reg) {
+               *prev = reg->next;
+               free(reg->key.p);
+               free(reg);
+       }
+       return(ROFF_IGN);
+}
+
 static enum rofferr
 roff_rm(ROFF_ARGS)
 {
 static enum rofferr
 roff_rm(ROFF_ARGS)
 {
@@ -1307,33 +1712,54 @@ roff_rm(ROFF_ARGS)
        return(ROFF_IGN);
 }
 
        return(ROFF_IGN);
 }
 
-/* ARGSUSED */
+static enum rofferr
+roff_it(ROFF_ARGS)
+{
+       char            *cp;
+       size_t           len;
+       int              iv;
+
+       /* Parse the number of lines. */
+       cp = *bufp + pos;
+       len = strcspn(cp, " \t");
+       cp[len] = '\0';
+       if ((iv = mandoc_strntoi(cp, len, 10)) <= 0) {
+               mandoc_msg(MANDOCERR_NUMERIC, r->parse,
+                   ln, ppos, *bufp + 1);
+               return(ROFF_IGN);
+       }
+       cp += len + 1;
+
+       /* Arm the input line trap. */
+       roffit_lines = iv;
+       roffit_macro = mandoc_strdup(cp);
+       return(ROFF_IGN);
+}
+
 static enum rofferr
 roff_Dd(ROFF_ARGS)
 {
        const char *const       *cp;
 
 static enum rofferr
 roff_Dd(ROFF_ARGS)
 {
        const char *const       *cp;
 
-       if (MPARSE_MDOC != r->parsetype)
+       if (0 == ((MPARSE_MDOC | MPARSE_QUICK) & r->options))
                for (cp = __mdoc_reserved; *cp; cp++)
                        roff_setstr(r, *cp, NULL, 0);
 
        return(ROFF_CONT);
 }
 
                for (cp = __mdoc_reserved; *cp; cp++)
                        roff_setstr(r, *cp, NULL, 0);
 
        return(ROFF_CONT);
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_TH(ROFF_ARGS)
 {
        const char *const       *cp;
 
 static enum rofferr
 roff_TH(ROFF_ARGS)
 {
        const char *const       *cp;
 
-       if (MPARSE_MDOC != r->parsetype)
+       if (0 == (MPARSE_QUICK & r->options))
                for (cp = __man_reserved; *cp; cp++)
                        roff_setstr(r, *cp, NULL, 0);
 
        return(ROFF_CONT);
 }
 
                for (cp = __man_reserved; *cp; cp++)
                        roff_setstr(r, *cp, NULL, 0);
 
        return(ROFF_CONT);
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_TE(ROFF_ARGS)
 {
 static enum rofferr
 roff_TE(ROFF_ARGS)
 {
@@ -1346,7 +1772,6 @@ roff_TE(ROFF_ARGS)
        return(ROFF_IGN);
 }
 
        return(ROFF_IGN);
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_T_(ROFF_ARGS)
 {
 static enum rofferr
 roff_T_(ROFF_ARGS)
 {
@@ -1369,7 +1794,7 @@ roff_closeeqn(struct roff *r)
 #endif
 
 static void
 #endif
 
 static void
-roff_openeqn(struct roff *r, const char *name, int line, 
+roff_openeqn(struct roff *r, const char *name, int line,
                int offs, const char *buf)
 {
        struct eqn_node *e;
                int offs, const char *buf)
 {
        struct eqn_node *e;
@@ -1391,7 +1816,6 @@ roff_openeqn(struct roff *r, const char *name, int line,
        }
 }
 
        }
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_EQ(ROFF_ARGS)
 {
 static enum rofferr
 roff_EQ(ROFF_ARGS)
 {
@@ -1400,7 +1824,6 @@ roff_EQ(ROFF_ARGS)
        return(ROFF_IGN);
 }
 
        return(ROFF_IGN);
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_EN(ROFF_ARGS)
 {
 static enum rofferr
 roff_EN(ROFF_ARGS)
 {
@@ -1409,29 +1832,27 @@ roff_EN(ROFF_ARGS)
        return(ROFF_IGN);
 }
 
        return(ROFF_IGN);
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_TS(ROFF_ARGS)
 {
 static enum rofferr
 roff_TS(ROFF_ARGS)
 {
-       struct tbl_node *t;
+       struct tbl_node *tbl;
 
        if (r->tbl) {
                mandoc_msg(MANDOCERR_SCOPEBROKEN, r->parse, ln, ppos, NULL);
                tbl_end(&r->tbl);
        }
 
 
        if (r->tbl) {
                mandoc_msg(MANDOCERR_SCOPEBROKEN, r->parse, ln, ppos, NULL);
                tbl_end(&r->tbl);
        }
 
-       t = tbl_alloc(ppos, ln, r->parse);
+       tbl = tbl_alloc(ppos, ln, r->parse);
 
        if (r->last_tbl)
 
        if (r->last_tbl)
-               r->last_tbl->next = t;
+               r->last_tbl->next = tbl;
        else
        else
-               r->first_tbl = r->last_tbl = t;
+               r->first_tbl = r->last_tbl = tbl;
 
 
-       r->tbl = r->last_tbl = t;
+       r->tbl = r->last_tbl = tbl;
        return(ROFF_IGN);
 }
 
        return(ROFF_IGN);
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_cc(ROFF_ARGS)
 {
 static enum rofferr
 roff_cc(ROFF_ARGS)
 {
@@ -1448,7 +1869,6 @@ roff_cc(ROFF_ARGS)
        return(ROFF_IGN);
 }
 
        return(ROFF_IGN);
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_tr(ROFF_ARGS)
 {
 static enum rofferr
 roff_tr(ROFF_ARGS)
 {
@@ -1470,9 +1890,9 @@ roff_tr(ROFF_ARGS)
                if ('\\' == *first) {
                        esc = mandoc_escape(&p, NULL, NULL);
                        if (ESCAPE_ERROR == esc) {
                if ('\\' == *first) {
                        esc = mandoc_escape(&p, NULL, NULL);
                        if (ESCAPE_ERROR == esc) {
-                               mandoc_msg
-                                       (MANDOCERR_BADESCAPE, r->parse, 
-                                        ln, (int)(p - *bufp), NULL);
+                               mandoc_msg(MANDOCERR_BADESCAPE,
+                                   r->parse, ln,
+                                   (int)(p - *bufp), NULL);
                                return(ROFF_IGN);
                        }
                        fsz = (size_t)(p - first);
                                return(ROFF_IGN);
                        }
                        fsz = (size_t)(p - first);
@@ -1482,28 +1902,28 @@ roff_tr(ROFF_ARGS)
                if ('\\' == *second) {
                        esc = mandoc_escape(&p, NULL, NULL);
                        if (ESCAPE_ERROR == esc) {
                if ('\\' == *second) {
                        esc = mandoc_escape(&p, NULL, NULL);
                        if (ESCAPE_ERROR == esc) {
-                               mandoc_msg
-                                       (MANDOCERR_BADESCAPE, r->parse, 
-                                        ln, (int)(p - *bufp), NULL);
+                               mandoc_msg(MANDOCERR_BADESCAPE,
+                                   r->parse, ln,
+                                   (int)(p - *bufp), NULL);
                                return(ROFF_IGN);
                        }
                        ssz = (size_t)(p - second);
                } else if ('\0' == *second) {
                                return(ROFF_IGN);
                        }
                        ssz = (size_t)(p - second);
                } else if ('\0' == *second) {
-                       mandoc_msg(MANDOCERR_ARGCOUNT, r->parse, 
-                                       ln, (int)(p - *bufp), NULL);
+                       mandoc_msg(MANDOCERR_ARGCOUNT, r->parse,
+                           ln, (int)(p - *bufp), NULL);
                        second = " ";
                        p--;
                }
 
                if (fsz > 1) {
                        second = " ";
                        p--;
                }
 
                if (fsz > 1) {
-                       roff_setstrn(&r->xmbtab, first, 
-                                       fsz, second, ssz, 0);
+                       roff_setstrn(&r->xmbtab, first, fsz,
+                           second, ssz, 0);
                        continue;
                }
 
                if (NULL == r->xtab)
                        continue;
                }
 
                if (NULL == r->xtab)
-                       r->xtab = mandoc_calloc
-                               (128, sizeof(struct roffstr));
+                       r->xtab = mandoc_calloc(128,
+                           sizeof(struct roffstr));
 
                free(r->xtab[(int)*first].p);
                r->xtab[(int)*first].p = mandoc_strndup(second, ssz);
 
                free(r->xtab[(int)*first].p);
                r->xtab[(int)*first].p = mandoc_strndup(second, ssz);
@@ -1513,13 +1933,13 @@ roff_tr(ROFF_ARGS)
        return(ROFF_IGN);
 }
 
        return(ROFF_IGN);
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_so(ROFF_ARGS)
 {
        char *name;
 
 static enum rofferr
 roff_so(ROFF_ARGS)
 {
        char *name;
 
-       mandoc_msg(MANDOCERR_SO, r->parse, ln, ppos, NULL);
+       name = *bufp + pos;
+       mandoc_vmsg(MANDOCERR_SO, r->parse, ln, ppos, ".so %s", name);
 
        /*
         * Handle `so'.  Be EXTREMELY careful, as we shouldn't be
 
        /*
         * Handle `so'.  Be EXTREMELY careful, as we shouldn't be
@@ -1528,9 +1948,9 @@ roff_so(ROFF_ARGS)
         * or using absolute paths.
         */
 
         * or using absolute paths.
         */
 
-       name = *bufp + pos;
        if ('/' == *name || strstr(name, "../") || strstr(name, "/..")) {
        if ('/' == *name || strstr(name, "../") || strstr(name, "/..")) {
-               mandoc_msg(MANDOCERR_SOPATH, r->parse, ln, pos, NULL);
+               mandoc_vmsg(MANDOCERR_SO_PATH, r->parse, ln, ppos,
+                   ".so %s", name);
                return(ROFF_ERR);
        }
 
                return(ROFF_ERR);
        }
 
@@ -1538,7 +1958,6 @@ roff_so(ROFF_ARGS)
        return(ROFF_SO);
 }
 
        return(ROFF_SO);
 }
 
-/* ARGSUSED */
 static enum rofferr
 roff_userdef(ROFF_ARGS)
 {
 static enum rofferr
 roff_userdef(ROFF_ARGS)
 {
@@ -1548,7 +1967,7 @@ roff_userdef(ROFF_ARGS)
 
        /*
         * Collect pointers to macro argument strings
 
        /*
         * Collect pointers to macro argument strings
-        * and null-terminate them.
+        * and NUL-terminate them.
         */
        cp = *bufp + pos;
        for (i = 0; i < 9; i++)
         */
        cp = *bufp + pos;
        for (i = 0; i < 9; i++)
@@ -1567,14 +1986,9 @@ roff_userdef(ROFF_ARGS)
                        cp += 2;
                        continue;
                }
                        cp += 2;
                        continue;
                }
-
-               *szp = strlen(n1) - 3 + strlen(arg[i]) + 1;
-               n2 = mandoc_malloc(*szp);
-
-               strlcpy(n2, n1, (size_t)(cp - n1 + 1));
-               strlcat(n2, arg[i], *szp);
-               strlcat(n2, cp + 3, *szp);
-
+               *cp = '\0';
+               *szp = mandoc_asprintf(&n2, "%s%s%s",
+                   n1, arg[i], cp + 3) + 1;
                cp = n2 + (cp - n1);
                free(n1);
                n1 = n2;
                cp = n2 + (cp - n1);
                free(n1);
                n1 = n2;
@@ -1628,22 +2042,23 @@ roff_getname(struct roff *r, char **cpp, int ln, int pos)
 
 /*
  * Store *string into the user-defined string called *name.
 
 /*
  * Store *string into the user-defined string called *name.
- * In multiline mode, append to an existing entry and append '\n';
- * else replace the existing entry, if there is one.
  * To clear an existing entry, call with (*r, *name, NULL, 0).
  * To clear an existing entry, call with (*r, *name, NULL, 0).
+ * append == 0: replace mode
+ * append == 1: single-line append mode
+ * append == 2: multiline append mode, append '\n' after each call
  */
 static void
 roff_setstr(struct roff *r, const char *name, const char *string,
  */
 static void
 roff_setstr(struct roff *r, const char *name, const char *string,
-       int multiline)
+       int append)
 {
 
        roff_setstrn(&r->strtab, name, strlen(name), string,
 {
 
        roff_setstrn(&r->strtab, name, strlen(name), string,
-                       string ? strlen(string) : 0, multiline);
+           string ? strlen(string) : 0, append);
 }
 
 static void
 roff_setstrn(struct roffkv **r, const char *name, size_t namesz,
 }
 
 static void
 roff_setstrn(struct roffkv **r, const char *name, size_t namesz,
-               const char *string, size_t stringsz, int multiline)
+               const char *string, size_t stringsz, int append)
 {
        struct roffkv   *n;
        char            *c;
 {
        struct roffkv   *n;
        char            *c;
@@ -1653,7 +2068,8 @@ roff_setstrn(struct roffkv **r, const char *name, size_t namesz,
        /* Search for an existing string with the same name. */
        n = *r;
 
        /* Search for an existing string with the same name. */
        n = *r;
 
-       while (n && strcmp(name, n->key.p))
+       while (n && (namesz != n->key.sz ||
+                       strncmp(n->key.p, name, namesz)))
                n = n->next;
 
        if (NULL == n) {
                n = n->next;
 
        if (NULL == n) {
@@ -1665,8 +2081,7 @@ roff_setstrn(struct roffkv **r, const char *name, size_t namesz,
                n->val.sz = 0;
                n->next = *r;
                *r = n;
                n->val.sz = 0;
                n->next = *r;
                *r = n;
-       } else if (0 == multiline) {
-               /* In multiline mode, append; else replace. */
+       } else if (0 == append) {
                free(n->val.p);
                n->val.p = NULL;
                n->val.sz = 0;
                free(n->val.p);
                n->val.p = NULL;
                n->val.sz = 0;
@@ -1679,7 +2094,7 @@ roff_setstrn(struct roffkv **r, const char *name, size_t namesz,
         * One additional byte for the '\n' in multiline mode,
         * and one for the terminating '\0'.
         */
         * One additional byte for the '\n' in multiline mode,
         * and one for the terminating '\0'.
         */
-       newch = stringsz + (multiline ? 2u : 1u);
+       newch = stringsz + (1 < append ? 2u : 1u);
 
        if (NULL == n->val.p) {
                n->val.p = mandoc_malloc(newch);
 
        if (NULL == n->val.p) {
                n->val.p = mandoc_malloc(newch);
@@ -1706,7 +2121,7 @@ roff_setstrn(struct roffkv **r, const char *name, size_t namesz,
        }
 
        /* Append terminating bytes. */
        }
 
        /* Append terminating bytes. */
-       if (multiline)
+       if (1 < append)
                *c++ = '\n';
 
        *c = '\0';
                *c++ = '\n';
 
        *c = '\0';
@@ -1717,12 +2132,18 @@ static const char *
 roff_getstrn(const struct roff *r, const char *name, size_t len)
 {
        const struct roffkv *n;
 roff_getstrn(const struct roff *r, const char *name, size_t len)
 {
        const struct roffkv *n;
+       int i;
 
        for (n = r->strtab; n; n = n->next)
 
        for (n = r->strtab; n; n = n->next)
-               if (0 == strncmp(name, n->key.p, len) && 
-                               '\0' == n->key.p[(int)len])
+               if (0 == strncmp(name, n->key.p, len) &&
+                   '\0' == n->key.p[(int)len])
                        return(n->val.p);
 
                        return(n->val.p);
 
+       for (i = 0; i < PREDEFS_MAX; i++)
+               if (0 == strncmp(name, predefs[i].name, len) &&
+                               '\0' == predefs[i].name[(int)len])
+                       return(predefs[i].str);
+
        return(NULL);
 }
 
        return(NULL);
 }
 
@@ -1742,14 +2163,14 @@ roff_freestr(struct roffkv *r)
 const struct tbl_span *
 roff_span(const struct roff *r)
 {
 const struct tbl_span *
 roff_span(const struct roff *r)
 {
-       
+
        return(r->tbl ? tbl_span(r->tbl) : NULL);
 }
 
 const struct eqn *
 roff_eqn(const struct roff *r)
 {
        return(r->tbl ? tbl_span(r->tbl) : NULL);
 }
 
 const struct eqn *
 roff_eqn(const struct roff *r)
 {
-       
+
        return(r->last_eqn ? &r->last_eqn->eqn : NULL);
 }
 
        return(r->last_eqn ? &r->last_eqn->eqn : NULL);
 }
 
@@ -1808,8 +2229,8 @@ roff_strdup(const struct roff *r, const char *p)
                         * Append the match to the array and move
                         * forward by its keysize.
                         */
                         * Append the match to the array and move
                         * forward by its keysize.
                         */
-                       res = mandoc_realloc
-                               (res, ssz + cp->val.sz + 1);
+                       res = mandoc_realloc(res,
+                           ssz + cp->val.sz + 1);
                        memcpy(res + ssz, cp->val.p, cp->val.sz);
                        ssz += cp->val.sz;
                        p += (int)cp->key.sz;
                        memcpy(res + ssz, cp->val.p, cp->val.sz);
                        ssz += cp->val.sz;
                        p += (int)cp->key.sz;
@@ -1830,8 +2251,8 @@ roff_strdup(const struct roff *r, const char *p)
                        memcpy(res + ssz, pp, sz);
                        break;
                }
                        memcpy(res + ssz, pp, sz);
                        break;
                }
-               /* 
-                * We bail out on bad escapes. 
+               /*
+                * We bail out on bad escapes.
                 * No need to warn: we already did so when
                 * roff_res() was called.
                 */
                 * No need to warn: we already did so when
                 * roff_res() was called.
                 */
@@ -1846,7 +2267,7 @@ roff_strdup(const struct roff *r, const char *p)
 }
 
 /*
 }
 
 /*
- * Find out whether a line is a macro line or not.  
+ * Find out whether a line is a macro line or not.
  * If it is, adjust the current position and return one; if it isn't,
  * return zero and don't change the current position.
  * If the control character has been set with `.cc', then let that grain
  * If it is, adjust the current position and return one; if it isn't,
  * return zero and don't change the current position.
  * If the control character has been set with `.cc', then let that grain