]> git.cameronkatri.com Git - mandoc.git/blobdiff - mdoc_man.c
support more than one tag entry for the same search term;
[mandoc.git] / mdoc_man.c
index 974cfbd46f58cd9505bf843e47a39e34200469d9..ab245313492a452f3056a6ed262bc672079f3ed2 100644 (file)
@@ -1,6 +1,6 @@
-/*     $Id: mdoc_man.c,v 1.77 2014/11/30 05:29:00 schwarze Exp $ */
+/*     $Id: mdoc_man.c,v 1.96 2016/01/08 17:48:09 schwarze Exp $ */
 /*
- * Copyright (c) 2011, 2012, 2013, 2014 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2011-2016 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
 #include <stdio.h>
 #include <string.h>
 
-#include "mandoc.h"
 #include "mandoc_aux.h"
-#include "out.h"
-#include "man.h"
+#include "mandoc.h"
+#include "roff.h"
 #include "mdoc.h"
+#include "man.h"
+#include "out.h"
 #include "main.h"
 
-#define        DECL_ARGS const struct mdoc_meta *meta, \
-                 const struct mdoc_node *n
+#define        DECL_ARGS const struct roff_meta *meta, struct roff_node *n
 
 struct manact {
        int             (*cond)(DECL_ARGS); /* DON'T run actions */
@@ -108,7 +108,7 @@ static      int       pre_sm(DECL_ARGS);
 static int       pre_sp(DECL_ARGS);
 static int       pre_sect(DECL_ARGS);
 static int       pre_sy(DECL_ARGS);
-static void      pre_syn(const struct mdoc_node *);
+static void      pre_syn(const struct roff_node *);
 static int       pre_vt(DECL_ARGS);
 static int       pre_ux(DECL_ARGS);
 static int       pre_xr(DECL_ARGS);
@@ -116,8 +116,8 @@ static      void      print_word(const char *);
 static void      print_line(const char *, int);
 static void      print_block(const char *, int);
 static void      print_offs(const char *, int);
-static void      print_width(const char *,
-                               const struct mdoc_node *, size_t);
+static void      print_width(const struct mdoc_bl *,
+                       const struct roff_node *);
 static void      print_count(int *);
 static void      print_node(DECL_ARGS);
 
@@ -186,8 +186,8 @@ static      const struct manact manacts[MDOC_MAX + 1] = {
        { NULL, pre_bx, NULL, NULL, NULL }, /* Bx */
        { NULL, pre_skip, NULL, NULL, NULL }, /* Db */
        { NULL, NULL, NULL, NULL, NULL }, /* Dc */
-       { cond_body, pre_enc, post_enc, "\\(lq", "\\(rq" }, /* Do */
-       { cond_body, pre_enc, post_enc, "\\(lq", "\\(rq" }, /* Dq */
+       { cond_body, pre_enc, post_enc, "\\(Lq", "\\(Rq" }, /* Do */
+       { cond_body, pre_enc, post_enc, "\\(Lq", "\\(Rq" }, /* Dq */
        { NULL, NULL, NULL, NULL, NULL }, /* Ec */
        { NULL, NULL, NULL, NULL, NULL }, /* Ef */
        { NULL, pre_em, post_font, NULL, NULL }, /* Em */
@@ -265,7 +265,7 @@ static      int             outflags;
 
 #define        BL_STACK_MAX    32
 
-static size_t          Bl_stack[BL_STACK_MAX];  /* offsets [chars] */
+static int             Bl_stack[BL_STACK_MAX];  /* offsets [chars] */
 static int             Bl_stack_post[BL_STACK_MAX];  /* add final .RE */
 static int             Bl_stack_len;  /* number of nested Bl blocks */
 static int             TPremain;  /* characters before tag is full */
@@ -423,7 +423,7 @@ print_offs(const char *v, int keywords)
 {
        char              buf[24];
        struct roffsu     su;
-       size_t            sz;
+       int               sz;
 
        print_line(".RS", MMAN_Bk_susp);
 
@@ -434,7 +434,7 @@ print_offs(const char *v, int keywords)
                sz = 6;
        else if (keywords && !strcmp(v, "indent-two"))
                sz = 12;
-       else if (a2roffsu(v, &su, SCALE_MAX)) {
+       else if (a2roffsu(v, &su, SCALE_EN) > 1) {
                if (SCALE_EN == su.unit)
                        sz = su.scale;
                else {
@@ -459,7 +459,7 @@ print_offs(const char *v, int keywords)
        if (Bl_stack_len)
                sz += Bl_stack[Bl_stack_len - 1];
 
-       (void)snprintf(buf, sizeof(buf), "%zun", sz);
+       (void)snprintf(buf, sizeof(buf), "%dn", sz);
        print_word(buf);
        outflags |= MMAN_nl;
 }
@@ -468,20 +468,19 @@ print_offs(const char *v, int keywords)
  * Set up the indentation for a list item; used from pre_it().
  */
 static void
-print_width(const char *v, const struct mdoc_node *child, size_t defsz)
+print_width(const struct mdoc_bl *bl, const struct roff_node *child)
 {
        char              buf[24];
        struct roffsu     su;
-       size_t            sz, chsz;
-       int               numeric, remain;
+       int               numeric, remain, sz, chsz;
 
        numeric = 1;
        remain = 0;
 
-       /* Convert v into a number (of characters). */
-       if (NULL == v)
-               sz = defsz;
-       else if (a2roffsu(v, &su, SCALE_MAX)) {
+       /* Convert the width into a number (of characters). */
+       if (bl->width == NULL)
+               sz = (bl->type == LIST_hang) ? 6 : 0;
+       else if (a2roffsu(bl->width, &su, SCALE_MAX) > 1) {
                if (SCALE_EN == su.unit)
                        sz = su.scale;
                else {
@@ -489,11 +488,15 @@ print_width(const char *v, const struct mdoc_node *child, size_t defsz)
                        numeric = 0;
                }
        } else
-               sz = strlen(v);
+               sz = strlen(bl->width);
 
        /* XXX Rough estimation, might have multiple parts. */
-       chsz = (NULL != child && MDOC_TEXT == child->type) ?
-           strlen(child->string) : 0;
+       if (bl->type == LIST_enum)
+               chsz = (bl->count > 8) + 1;
+       else if (child != NULL && child->type == ROFFT_TEXT)
+               chsz = strlen(child->string);
+       else
+               chsz = 0;
 
        /* Maybe we are inside an enclosing list? */
        mid_it();
@@ -505,17 +508,17 @@ print_width(const char *v, const struct mdoc_node *child, size_t defsz)
        Bl_stack[Bl_stack_len++] = sz + 2;
 
        /* Set up the current list. */
-       if (defsz && chsz > sz)
+       if (chsz > sz && bl->type != LIST_tag)
                print_block(".HP", 0);
        else {
                print_block(".TP", 0);
                remain = sz + 2;
        }
        if (numeric) {
-               (void)snprintf(buf, sizeof(buf), "%zun", sz + 2);
+               (void)snprintf(buf, sizeof(buf), "%dn", sz + 2);
                print_word(buf);
        } else
-               print_word(v);
+               print_word(bl->width);
        TPremain = remain;
 }
 
@@ -524,12 +527,12 @@ print_count(int *count)
 {
        char              buf[24];
 
-       (void)snprintf(buf, sizeof(buf), "%d.", ++*count);
+       (void)snprintf(buf, sizeof(buf), "%d.\\&", ++*count);
        print_word(buf);
 }
 
 void
-man_man(void *arg, const struct man *man)
+man_man(void *arg, const struct roff_man *man)
 {
 
        /*
@@ -542,18 +545,14 @@ man_man(void *arg, const struct man *man)
 }
 
 void
-man_mdoc(void *arg, const struct mdoc *mdoc)
+man_mdoc(void *arg, const struct roff_man *mdoc)
 {
-       const struct mdoc_meta *meta;
-       const struct mdoc_node *n;
-
-       meta = mdoc_meta(mdoc);
-       n = mdoc_node(mdoc);
+       struct roff_node *n;
 
        printf(".TH \"%s\" \"%s\" \"%s\" \"%s\" \"%s\"\n",
-           meta->title,
-           (meta->msec == NULL ? "" : meta->msec),
-           meta->date, meta->os, meta->vol);
+           mdoc->meta.title,
+           (mdoc->meta.msec == NULL ? "" : mdoc->meta.msec),
+           mdoc->meta.date, mdoc->meta.os, mdoc->meta.vol);
 
        /* Disable hyphenation and if nroff, disable justification. */
        printf(".nh\n.if n .ad l");
@@ -564,15 +563,16 @@ man_mdoc(void *arg, const struct mdoc *mdoc)
                fontqueue.head = fontqueue.tail = mandoc_malloc(8);
                *fontqueue.tail = 'R';
        }
-       print_node(meta, n);
+       for (n = mdoc->first->child; n != NULL; n = n->next)
+               print_node(&mdoc->meta, n);
        putchar('\n');
 }
 
 static void
 print_node(DECL_ARGS)
 {
-       const struct mdoc_node  *sub;
        const struct manact     *act;
+       struct roff_node        *sub;
        int                      cond, do_sub;
 
        /*
@@ -585,8 +585,9 @@ print_node(DECL_ARGS)
        act = NULL;
        cond = 0;
        do_sub = 1;
+       n->flags &= ~MDOC_ENDED;
 
-       if (MDOC_TEXT == n->type) {
+       if (n->type == ROFFT_TEXT) {
                /*
                 * Make sure that we don't happen to start with a
                 * control character at the start of a line.
@@ -609,7 +610,8 @@ print_node(DECL_ARGS)
                 */
                act = manacts + n->tok;
                cond = act->cond == NULL || (*act->cond)(meta, n);
-               if (cond && act->pre && (n->end == ENDBODY_NOT || n->nchild))
+               if (cond && act->pre != NULL &&
+                   (n->end == ENDBODY_NOT || n->child != NULL))
                        do_sub = (*act->pre)(meta, n);
        }
 
@@ -632,7 +634,7 @@ print_node(DECL_ARGS)
                (*act->post)(meta, n);
 
        if (ENDBODY_NOT != n->end)
-               n->pending->flags |= MDOC_ENDED;
+               n->body->flags |= MDOC_ENDED;
 
        if (ENDBODY_NOSPACE == n->end)
                outflags &= ~(MMAN_spc | MMAN_nl);
@@ -642,14 +644,14 @@ static int
 cond_head(DECL_ARGS)
 {
 
-       return(MDOC_HEAD == n->type);
+       return n->type == ROFFT_HEAD;
 }
 
 static int
 cond_body(DECL_ARGS)
 {
 
-       return(MDOC_BODY == n->type);
+       return n->type == ROFFT_BODY;
 }
 
 static int
@@ -659,10 +661,10 @@ pre_enc(DECL_ARGS)
 
        prefix = manacts[n->tok].prefix;
        if (NULL == prefix)
-               return(1);
+               return 1;
        print_word(prefix);
        outflags &= ~MMAN_spc;
-       return(1);
+       return 1;
 }
 
 static void
@@ -680,37 +682,36 @@ post_enc(DECL_ARGS)
 static int
 pre_ex(DECL_ARGS)
 {
-       int      nchild;
+       struct roff_node *nch;
 
        outflags |= MMAN_br | MMAN_nl;
 
        print_word("The");
 
-       nchild = n->nchild;
-       for (n = n->child; n; n = n->next) {
+       for (nch = n->child; nch != NULL; nch = nch->next) {
                font_push('B');
-               print_word(n->string);
+               print_word(nch->string);
                font_pop();
 
-               if (n->next == NULL)
+               if (nch->next == NULL)
                        continue;
 
-               if (nchild > 2) {
+               if (nch->prev != NULL || nch->next->next != NULL) {
                        outflags &= ~MMAN_spc;
                        print_word(",");
                }
-               if (n->next->next == NULL)
+               if (nch->next->next == NULL)
                        print_word("and");
        }
 
-       if (nchild > 1)
+       if (n->child != NULL && n->child->next != NULL)
                print_word("utilities exit\\~0");
        else
                print_word("utility exits\\~0");
 
        print_word("on success, and\\~>0 if an error occurs.");
        outflags |= MMAN_nl;
-       return(0);
+       return 0;
 }
 
 static void
@@ -748,7 +749,7 @@ pre__t(DECL_ARGS)
                outflags &= ~MMAN_spc;
        } else
                font_push('I');
-       return(1);
+       return 1;
 }
 
 static void
@@ -772,14 +773,14 @@ static int
 pre_sect(DECL_ARGS)
 {
 
-       if (MDOC_HEAD == n->type) {
+       if (n->type == ROFFT_HEAD) {
                outflags |= MMAN_sp;
                print_block(manacts[n->tok].prefix, 0);
                print_word("");
                putchar('\"');
                outflags &= ~MMAN_spc;
        }
-       return(1);
+       return 1;
 }
 
 /*
@@ -789,7 +790,7 @@ static void
 post_sect(DECL_ARGS)
 {
 
-       if (MDOC_HEAD != n->type)
+       if (n->type != ROFFT_HEAD)
                return;
        outflags &= ~MMAN_spc;
        print_word("");
@@ -801,7 +802,7 @@ post_sect(DECL_ARGS)
 
 /* See mdoc_term.c, synopsis_pre() for comments. */
 static void
-pre_syn(const struct mdoc_node *n)
+pre_syn(const struct roff_node *n)
 {
 
        if (NULL == n->prev || ! (MDOC_SYNPRETTY & n->flags))
@@ -817,13 +818,9 @@ pre_syn(const struct mdoc_node *n)
 
        switch (n->prev->tok) {
        case MDOC_Fd:
-               /* FALLTHROUGH */
        case MDOC_Fn:
-               /* FALLTHROUGH */
        case MDOC_Fo:
-               /* FALLTHROUGH */
        case MDOC_In:
-               /* FALLTHROUGH */
        case MDOC_Vt:
                outflags |= MMAN_sp;
                break;
@@ -847,18 +844,18 @@ pre_an(DECL_ARGS)
        case AUTH_split:
                outflags &= ~MMAN_An_nosplit;
                outflags |= MMAN_An_split;
-               return(0);
+               return 0;
        case AUTH_nosplit:
                outflags &= ~MMAN_An_split;
                outflags |= MMAN_An_nosplit;
-               return(0);
+               return 0;
        default:
                if (MMAN_An_split & outflags)
                        outflags |= MMAN_br;
                else if (SEC_AUTHORS == n->sec &&
                    ! (MMAN_An_nosplit & outflags))
                        outflags |= MMAN_An_split;
-               return(1);
+               return 1;
        }
 }
 
@@ -869,17 +866,17 @@ pre_ap(DECL_ARGS)
        outflags &= ~MMAN_spc;
        print_word("'");
        outflags &= ~MMAN_spc;
-       return(0);
+       return 0;
 }
 
 static int
 pre_aq(DECL_ARGS)
 {
 
-       print_word(n->parent->prev != NULL &&
-           n->parent->prev->tok == MDOC_An ?  "<" : "\\(la");
+       print_word(n->child != NULL && n->child->next == NULL &&
+           n->child->tok == MDOC_Mt ?  "<" : "\\(la");
        outflags &= ~MMAN_spc;
-       return(1);
+       return 1;
 }
 
 static void
@@ -887,8 +884,8 @@ post_aq(DECL_ARGS)
 {
 
        outflags &= ~(MMAN_spc | MMAN_nl);
-       print_word(n->parent->prev != NULL &&
-           n->parent->prev->tok == MDOC_An ?  ">" : "\\(ra");
+       print_word(n->child != NULL && n->child->next == NULL &&
+           n->child->tok == MDOC_Mt ?  ">" : "\\(ra");
 }
 
 static int
@@ -903,7 +900,7 @@ pre_bd(DECL_ARGS)
        if (0 == n->norm->Bd.comp && NULL != n->parent->prev)
                outflags |= MMAN_sp;
        print_offs(n->norm->Bd.offs, 1);
-       return(1);
+       return 1;
 }
 
 static void
@@ -926,12 +923,12 @@ pre_bf(DECL_ARGS)
 {
 
        switch (n->type) {
-       case MDOC_BLOCK:
-               return(1);
-       case MDOC_BODY:
+       case ROFFT_BLOCK:
+               return 1;
+       case ROFFT_BODY:
                break;
        default:
-               return(0);
+               return 0;
        }
        switch (n->norm->Bf.font) {
        case FONT_Em:
@@ -944,14 +941,14 @@ pre_bf(DECL_ARGS)
                font_push('R');
                break;
        }
-       return(1);
+       return 1;
 }
 
 static void
 post_bf(DECL_ARGS)
 {
 
-       if (MDOC_BODY == n->type)
+       if (n->type == ROFFT_BODY)
                font_pop();
 }
 
@@ -960,13 +957,13 @@ pre_bk(DECL_ARGS)
 {
 
        switch (n->type) {
-       case MDOC_BLOCK:
-               return(1);
-       case MDOC_BODY:
+       case ROFFT_BLOCK:
+               return 1;
+       case ROFFT_BODY:
                outflags |= MMAN_Bk;
-               return(1);
+               return 1;
        default:
-               return(0);
+               return 0;
        }
 }
 
@@ -974,7 +971,7 @@ static void
 post_bk(DECL_ARGS)
 {
 
-       if (MDOC_BODY == n->type)
+       if (n->type == ROFFT_BODY)
                outflags &= ~MMAN_Bk;
 }
 
@@ -996,19 +993,21 @@ pre_bl(DECL_ARGS)
        switch (n->norm->Bl.type) {
        case LIST_enum:
                n->norm->Bl.count = 0;
-               return(1);
+               return 1;
        case LIST_column:
                break;
        default:
-               return(1);
+               return 1;
        }
 
-       print_line(".TS", MMAN_nl);
-       for (icol = 0; icol < n->norm->Bl.ncols; icol++)
-               print_word("l");
-       print_word(".");
+       if (n->child != NULL) {
+               print_line(".TS", MMAN_nl);
+               for (icol = 0; icol < n->norm->Bl.ncols; icol++)
+                       print_word("l");
+               print_word(".");
+       }
        outflags |= MMAN_nl;
-       return(1);
+       return 1;
 }
 
 static void
@@ -1017,7 +1016,8 @@ post_bl(DECL_ARGS)
 
        switch (n->norm->Bl.type) {
        case LIST_column:
-               print_line(".TE", 0);
+               if (n->child != NULL)
+                       print_line(".TE", 0);
                break;
        case LIST_enum:
                n->norm->Bl.count = 0;
@@ -1047,7 +1047,7 @@ pre_br(DECL_ARGS)
 {
 
        outflags |= MMAN_br;
-       return(0);
+       return 0;
 }
 
 static int
@@ -1062,12 +1062,12 @@ pre_bx(DECL_ARGS)
        }
        print_word("BSD");
        if (NULL == n)
-               return(0);
+               return 0;
        outflags &= ~MMAN_spc;
        print_word("-");
        outflags &= ~MMAN_spc;
        print_word(n->string);
-       return(0);
+       return 0;
 }
 
 static int
@@ -1075,7 +1075,7 @@ pre_dl(DECL_ARGS)
 {
 
        print_offs("6n", 0);
-       return(1);
+       return 1;
 }
 
 static void
@@ -1094,7 +1094,7 @@ pre_em(DECL_ARGS)
 {
 
        font_push('I');
-       return(1);
+       return 1;
 }
 
 static int
@@ -1103,11 +1103,11 @@ pre_en(DECL_ARGS)
 
        if (NULL == n->norm->Es ||
            NULL == n->norm->Es->child)
-               return(1);
+               return 1;
 
        print_word(n->norm->Es->child->string);
        outflags &= ~MMAN_spc;
-       return(1);
+       return 1;
 }
 
 static void
@@ -1128,16 +1128,37 @@ static int
 pre_eo(DECL_ARGS)
 {
 
-       outflags &= ~(MMAN_spc | MMAN_nl);
-       return(1);
+       if (n->end == ENDBODY_NOT &&
+           n->parent->head->child == NULL &&
+           n->child != NULL &&
+           n->child->end != ENDBODY_NOT)
+               print_word("\\&");
+       else if (n->end != ENDBODY_NOT ? n->child != NULL :
+           n->parent->head->child != NULL && (n->child != NULL ||
+           (n->parent->tail != NULL && n->parent->tail->child != NULL)))
+               outflags &= ~(MMAN_spc | MMAN_nl);
+       return 1;
 }
 
 static void
 post_eo(DECL_ARGS)
 {
+       int      body, tail;
 
-       if (n->end != ENDBODY_SPACE)
+       if (n->end != ENDBODY_NOT) {
+               outflags |= MMAN_spc;
+               return;
+       }
+
+       body = n->child != NULL || n->parent->head->child != NULL;
+       tail = n->parent->tail != NULL && n->parent->tail->child != NULL;
+
+       if (body && tail)
                outflags &= ~MMAN_spc;
+       else if ( ! (body || tail))
+               print_word("\\&");
+       else if ( ! tail)
+               outflags |= MMAN_spc;
 }
 
 static int
@@ -1159,7 +1180,7 @@ pre_fa(DECL_ARGS)
                if (NULL != (n = n->next))
                        print_word(",");
        }
-       return(0);
+       return 0;
 }
 
 static void
@@ -1176,7 +1197,7 @@ pre_fd(DECL_ARGS)
 
        pre_syn(n);
        font_push('B');
-       return(1);
+       return 1;
 }
 
 static void
@@ -1193,9 +1214,9 @@ pre_fl(DECL_ARGS)
 
        font_push('B');
        print_word("\\-");
-       if (n->nchild)
+       if (n->child != NULL)
                outflags &= ~MMAN_spc;
-       return(1);
+       return 1;
 }
 
 static void
@@ -1203,9 +1224,9 @@ post_fl(DECL_ARGS)
 {
 
        font_pop();
-       if ( ! (n->nchild ||
+       if (!(n->child != NULL ||
            n->next == NULL ||
-           n->next->type == MDOC_TEXT ||
+           n->next->type == ROFFT_TEXT ||
            n->next->flags & MDOC_LINE))
                outflags &= ~MMAN_spc;
 }
@@ -1218,7 +1239,7 @@ pre_fn(DECL_ARGS)
 
        n = n->child;
        if (NULL == n)
-               return(0);
+               return 0;
 
        if (MDOC_SYNPRETTY & n->flags)
                print_block(".HP 4n", MMAN_nl);
@@ -1233,7 +1254,7 @@ pre_fn(DECL_ARGS)
        n = n->next;
        if (NULL != n)
                pre_fa(meta, n);
-       return(0);
+       return 0;
 }
 
 static void
@@ -1252,23 +1273,25 @@ pre_fo(DECL_ARGS)
 {
 
        switch (n->type) {
-       case MDOC_BLOCK:
+       case ROFFT_BLOCK:
                pre_syn(n);
                break;
-       case MDOC_HEAD:
+       case ROFFT_HEAD:
+               if (n->child == NULL)
+                       return 0;
                if (MDOC_SYNPRETTY & n->flags)
                        print_block(".HP 4n", MMAN_nl);
                font_push('B');
                break;
-       case MDOC_BODY:
-               outflags &= ~MMAN_spc;
+       case ROFFT_BODY:
+               outflags &= ~(MMAN_spc | MMAN_nl);
                print_word("(");
                outflags &= ~MMAN_spc;
                break;
        default:
                break;
        }
-       return(1);
+       return 1;
 }
 
 static void
@@ -1276,10 +1299,11 @@ post_fo(DECL_ARGS)
 {
 
        switch (n->type) {
-       case MDOC_HEAD:
-               font_pop();
+       case ROFFT_HEAD:
+               if (n->child != NULL)
+                       font_pop();
                break;
-       case MDOC_BODY:
+       case ROFFT_BODY:
                post_fn(meta, n);
                break;
        default:
@@ -1293,7 +1317,7 @@ pre_ft(DECL_ARGS)
 
        pre_syn(n);
        font_push('I');
-       return(1);
+       return 1;
 }
 
 static int
@@ -1310,7 +1334,7 @@ pre_in(DECL_ARGS)
                outflags &= ~MMAN_spc;
                font_push('I');
        }
-       return(1);
+       return 1;
 }
 
 static void
@@ -1332,10 +1356,10 @@ post_in(DECL_ARGS)
 static int
 pre_it(DECL_ARGS)
 {
-       const struct mdoc_node *bln;
+       const struct roff_node *bln;
 
        switch (n->type) {
-       case MDOC_HEAD:
+       case ROFFT_HEAD:
                outflags |= MMAN_PP | MMAN_nl;
                bln = n->parent->parent;
                if (0 == bln->norm->Bl.comp ||
@@ -1345,24 +1369,20 @@ pre_it(DECL_ARGS)
                outflags &= ~MMAN_br;
                switch (bln->norm->Bl.type) {
                case LIST_item:
-                       return(0);
+                       return 0;
                case LIST_inset:
-                       /* FALLTHROUGH */
                case LIST_diag:
-                       /* FALLTHROUGH */
                case LIST_ohang:
                        if (bln->norm->Bl.type == LIST_diag)
                                print_line(".B \"", 0);
                        else
                                print_line(".R \"", 0);
                        outflags &= ~MMAN_spc;
-                       return(1);
+                       return 1;
                case LIST_bullet:
-                       /* FALLTHROUGH */
                case LIST_dash:
-                       /* FALLTHROUGH */
                case LIST_hyphen:
-                       print_width(bln->norm->Bl.width, NULL, 0);
+                       print_width(&bln->norm->Bl, NULL);
                        TPremain = 0;
                        outflags |= MMAN_nl;
                        font_push('B');
@@ -1372,31 +1392,31 @@ pre_it(DECL_ARGS)
                                print_word("-");
                        font_pop();
                        outflags |= MMAN_nl;
-                       return(0);
+                       return 0;
                case LIST_enum:
-                       print_width(bln->norm->Bl.width, NULL, 0);
+                       print_width(&bln->norm->Bl, NULL);
                        TPremain = 0;
                        outflags |= MMAN_nl;
                        print_count(&bln->norm->Bl.count);
                        outflags |= MMAN_nl;
-                       return(0);
+                       return 0;
                case LIST_hang:
-                       print_width(bln->norm->Bl.width, n->child, 6);
+                       print_width(&bln->norm->Bl, n->child);
                        TPremain = 0;
                        outflags |= MMAN_nl;
-                       return(1);
+                       return 1;
                case LIST_tag:
-                       print_width(bln->norm->Bl.width, n->child, 0);
+                       print_width(&bln->norm->Bl, n->child);
                        putchar('\n');
                        outflags &= ~MMAN_spc;
-                       return(1);
+                       return 1;
                default:
-                       return(1);
+                       return 1;
                }
        default:
                break;
        }
-       return(1);
+       return 1;
 }
 
 /*
@@ -1418,7 +1438,7 @@ mid_it(void)
 
        /* Restore the indentation of the enclosing list. */
        print_line(".RS", MMAN_Bk_susp);
-       (void)snprintf(buf, sizeof(buf), "%zun",
+       (void)snprintf(buf, sizeof(buf), "%dn",
            Bl_stack[Bl_stack_len - 1]);
        print_word(buf);
 
@@ -1429,12 +1449,12 @@ mid_it(void)
 static void
 post_it(DECL_ARGS)
 {
-       const struct mdoc_node *bln;
+       const struct roff_node *bln;
 
        bln = n->parent->parent;
 
        switch (n->type) {
-       case MDOC_HEAD:
+       case ROFFT_HEAD:
                switch (bln->norm->Bl.type) {
                case LIST_diag:
                        outflags &= ~MMAN_spc;
@@ -1447,18 +1467,13 @@ post_it(DECL_ARGS)
                        break;
                }
                break;
-       case MDOC_BODY:
+       case ROFFT_BODY:
                switch (bln->norm->Bl.type) {
                case LIST_bullet:
-                       /* FALLTHROUGH */
                case LIST_dash:
-                       /* FALLTHROUGH */
                case LIST_hyphen:
-                       /* FALLTHROUGH */
                case LIST_enum:
-                       /* FALLTHROUGH */
                case LIST_hang:
-                       /* FALLTHROUGH */
                case LIST_tag:
                        assert(Bl_stack_len);
                        Bl_stack[--Bl_stack_len] = 0;
@@ -1499,10 +1514,10 @@ post_lb(DECL_ARGS)
 static int
 pre_lk(DECL_ARGS)
 {
-       const struct mdoc_node *link, *descr;
+       const struct roff_node *link, *descr;
 
        if (NULL == (link = n->child))
-               return(0);
+               return 0;
 
        if (NULL != (descr = link->next)) {
                font_push('I');
@@ -1517,7 +1532,7 @@ pre_lk(DECL_ARGS)
        font_push('B');
        print_word(link->string);
        font_pop();
-       return(0);
+       return 0;
 }
 
 static int
@@ -1525,7 +1540,7 @@ pre_ll(DECL_ARGS)
 {
 
        print_line(".ll", 0);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1533,7 +1548,7 @@ pre_li(DECL_ARGS)
 {
 
        font_push('R');
-       return(1);
+       return 1;
 }
 
 static int
@@ -1541,16 +1556,16 @@ pre_nm(DECL_ARGS)
 {
        char    *name;
 
-       if (MDOC_BLOCK == n->type) {
+       if (n->type == ROFFT_BLOCK) {
                outflags |= MMAN_Bk;
                pre_syn(n);
        }
-       if (MDOC_ELEM != n->type && MDOC_HEAD != n->type)
-               return(1);
+       if (n->type != ROFFT_ELEM && n->type != ROFFT_HEAD)
+               return 1;
        name = n->child ? n->child->string : meta->name;
        if (NULL == name)
-               return(0);
-       if (MDOC_HEAD == n->type) {
+               return 0;
+       if (n->type == ROFFT_HEAD) {
                if (NULL == n->parent->prev)
                        outflags |= MMAN_sp;
                print_block(".HP", 0);
@@ -1560,7 +1575,7 @@ pre_nm(DECL_ARGS)
        font_push('B');
        if (NULL == n->child)
                print_word(meta->name);
-       return(1);
+       return 1;
 }
 
 static void
@@ -1568,12 +1583,11 @@ post_nm(DECL_ARGS)
 {
 
        switch (n->type) {
-       case MDOC_BLOCK:
+       case ROFFT_BLOCK:
                outflags &= ~MMAN_Bk;
                break;
-       case MDOC_HEAD:
-               /* FALLTHROUGH */
-       case MDOC_ELEM:
+       case ROFFT_HEAD:
+       case ROFFT_ELEM:
                if (n->child != NULL || meta->name != NULL)
                        font_pop();
                break;
@@ -1587,7 +1601,7 @@ pre_no(DECL_ARGS)
 {
 
        outflags |= MMAN_spc_force;
-       return(1);
+       return 1;
 }
 
 static int
@@ -1595,7 +1609,7 @@ pre_ns(DECL_ARGS)
 {
 
        outflags &= ~MMAN_spc;
-       return(0);
+       return 0;
 }
 
 static void
@@ -1614,7 +1628,7 @@ pre_pp(DECL_ARGS)
                outflags |= MMAN_PP;
        outflags |= MMAN_sp | MMAN_nl;
        outflags &= ~MMAN_br;
-       return(0);
+       return 0;
 }
 
 static int
@@ -1625,40 +1639,39 @@ pre_rs(DECL_ARGS)
                outflags |= MMAN_PP | MMAN_sp | MMAN_nl;
                outflags &= ~MMAN_br;
        }
-       return(1);
+       return 1;
 }
 
 static int
 pre_rv(DECL_ARGS)
 {
-       int      nchild;
+       struct roff_node *nch;
 
        outflags |= MMAN_br | MMAN_nl;
 
-       nchild = n->nchild;
-       if (nchild > 0) {
+       if (n->child != NULL) {
                print_word("The");
 
-               for (n = n->child; n; n = n->next) {
+               for (nch = n->child; nch != NULL; nch = nch->next) {
                        font_push('B');
-                       print_word(n->string);
+                       print_word(nch->string);
                        font_pop();
 
                        outflags &= ~MMAN_spc;
                        print_word("()");
 
-                       if (n->next == NULL)
+                       if (nch->next == NULL)
                                continue;
 
-                       if (nchild > 2) {
+                       if (nch->prev != NULL || nch->next->next != NULL) {
                                outflags &= ~MMAN_spc;
                                print_word(",");
                        }
-                       if (n->next->next == NULL)
+                       if (nch->next->next == NULL)
                                print_word("and");
                }
 
-               if (nchild > 1)
+               if (n->child != NULL && n->child->next != NULL)
                        print_word("functions return");
                else
                        print_word("function returns");
@@ -1677,14 +1690,14 @@ pre_rv(DECL_ARGS)
 
        print_word("is set to indicate the error.");
        outflags |= MMAN_nl;
-       return(0);
+       return 0;
 }
 
 static int
 pre_skip(DECL_ARGS)
 {
 
-       return(0);
+       return 0;
 }
 
 static int
@@ -1701,7 +1714,7 @@ pre_sm(DECL_ARGS)
        if (MMAN_Sm & outflags)
                outflags |= MMAN_spc;
 
-       return(0);
+       return 0;
 }
 
 static int
@@ -1713,7 +1726,7 @@ pre_sp(DECL_ARGS)
                print_line(".PP", 0);
        } else
                print_line(".sp", 0);
-       return(1);
+       return 1;
 }
 
 static void
@@ -1728,7 +1741,7 @@ pre_sy(DECL_ARGS)
 {
 
        font_push('B');
-       return(1);
+       return 1;
 }
 
 static int
@@ -1737,24 +1750,24 @@ pre_vt(DECL_ARGS)
 
        if (MDOC_SYNPRETTY & n->flags) {
                switch (n->type) {
-               case MDOC_BLOCK:
+               case ROFFT_BLOCK:
                        pre_syn(n);
-                       return(1);
-               case MDOC_BODY:
+                       return 1;
+               case ROFFT_BODY:
                        break;
                default:
-                       return(0);
+                       return 0;
                }
        }
        font_push('I');
-       return(1);
+       return 1;
 }
 
 static void
 post_vt(DECL_ARGS)
 {
 
-       if (MDOC_SYNPRETTY & n->flags && MDOC_BODY != n->type)
+       if (n->flags & MDOC_SYNPRETTY && n->type != ROFFT_BODY)
                return;
        font_pop();
 }
@@ -1765,16 +1778,16 @@ pre_xr(DECL_ARGS)
 
        n = n->child;
        if (NULL == n)
-               return(0);
+               return 0;
        print_node(meta, n);
        n = n->next;
        if (NULL == n)
-               return(0);
+               return 0;
        outflags &= ~MMAN_spc;
        print_word("(");
        print_node(meta, n);
        print_word(")");
-       return(0);
+       return 0;
 }
 
 static int
@@ -1783,9 +1796,9 @@ pre_ux(DECL_ARGS)
 
        print_word(manacts[n->tok].prefix);
        if (NULL == n->child)
-               return(0);
+               return 0;
        outflags &= ~MMAN_spc;
        print_word("\\ ");
        outflags &= ~MMAN_spc;
-       return(1);
+       return 1;
 }