]> git.cameronkatri.com Git - mandoc.git/blobdiff - mdoc_html.c
Do text production for .Bt, .Ex, .Rv, .Ud at the validation stage
[mandoc.git] / mdoc_html.c
index 1d34a39d4e3bfc597bff0fe9a1132c13f0fb5efc..81b60cb5bf77be5a071811c0a50193557f826b2d 100644 (file)
@@ -1,7 +1,7 @@
-/*     $Id: mdoc_html.c,v 1.235 2015/09/26 00:32:35 schwarze Exp $ */
+/*     $Id: mdoc_html.c,v 1.247 2017/01/11 17:39:53 schwarze Exp $ */
 /*
  * Copyright (c) 2008-2011, 2014 Kristaps Dzonsons <kristaps@bsd.lv>
- * Copyright (c) 2014, 2015 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2014, 2015, 2016, 2017 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
@@ -70,8 +70,6 @@ static        int               mdoc_bf_pre(MDOC_ARGS);
 static void              mdoc_bk_post(MDOC_ARGS);
 static int               mdoc_bk_pre(MDOC_ARGS);
 static int               mdoc_bl_pre(MDOC_ARGS);
-static int               mdoc_bt_pre(MDOC_ARGS);
-static int               mdoc_bx_pre(MDOC_ARGS);
 static int               mdoc_cd_pre(MDOC_ARGS);
 static int               mdoc_d1_pre(MDOC_ARGS);
 static int               mdoc_dv_pre(MDOC_ARGS);
@@ -107,7 +105,6 @@ static      int               mdoc_pp_pre(MDOC_ARGS);
 static void              mdoc_quote_post(MDOC_ARGS);
 static int               mdoc_quote_pre(MDOC_ARGS);
 static int               mdoc_rs_pre(MDOC_ARGS);
-static int               mdoc_rv_pre(MDOC_ARGS);
 static int               mdoc_sh_pre(MDOC_ARGS);
 static int               mdoc_skip_pre(MDOC_ARGS);
 static int               mdoc_sm_pre(MDOC_ARGS);
@@ -115,7 +112,6 @@ static      int               mdoc_sp_pre(MDOC_ARGS);
 static int               mdoc_ss_pre(MDOC_ARGS);
 static int               mdoc_sx_pre(MDOC_ARGS);
 static int               mdoc_sy_pre(MDOC_ARGS);
-static int               mdoc_ud_pre(MDOC_ARGS);
 static int               mdoc_va_pre(MDOC_ARGS);
 static int               mdoc_vt_pre(MDOC_ARGS);
 static int               mdoc_xr_pre(MDOC_ARGS);
@@ -158,7 +154,7 @@ static      const struct htmlmdoc mdocs[MDOC_MAX] = {
        {mdoc_quote_pre, mdoc_quote_post}, /* Op */
        {mdoc_ft_pre, NULL}, /* Ot */
        {mdoc_pa_pre, NULL}, /* Pa */
-       {mdoc_rv_pre, NULL}, /* Rv */
+       {mdoc_ex_pre, NULL}, /* Rv */
        {NULL, NULL}, /* St */
        {mdoc_va_pre, NULL}, /* Va */
        {mdoc_vt_pre, NULL}, /* Vt */
@@ -183,7 +179,7 @@ static      const struct htmlmdoc mdocs[MDOC_MAX] = {
        {mdoc_quote_pre, mdoc_quote_post}, /* Bo */
        {mdoc_quote_pre, mdoc_quote_post}, /* Bq */
        {mdoc_xx_pre, NULL}, /* Bsx */
-       {mdoc_bx_pre, NULL}, /* Bx */
+       {mdoc_xx_pre, NULL}, /* Bx */
        {mdoc_skip_pre, NULL}, /* Db */
        {NULL, NULL}, /* Dc */
        {mdoc_quote_pre, mdoc_quote_post}, /* Do */
@@ -224,10 +220,10 @@ static    const struct htmlmdoc mdocs[MDOC_MAX] = {
        {NULL, NULL}, /* Oc */
        {mdoc_bk_pre, mdoc_bk_post}, /* Bk */
        {NULL, NULL}, /* Ek */
-       {mdoc_bt_pre, NULL}, /* Bt */
+       {NULL, NULL}, /* Bt */
        {NULL, NULL}, /* Hf */
        {mdoc_em_pre, NULL}, /* Fr */
-       {mdoc_ud_pre, NULL}, /* Ud */
+       {NULL, NULL}, /* Ud */
        {mdoc_lb_pre, NULL}, /* Lb */
        {mdoc_pp_pre, NULL}, /* Lp */
        {mdoc_lk_pre, NULL}, /* Lk */
@@ -286,7 +282,7 @@ static void
 synopsis_pre(struct html *h, const struct roff_node *n)
 {
 
-       if (NULL == n->prev || ! (MDOC_SYNPRETTY & n->flags))
+       if (NULL == n->prev || ! (NODE_SYNPRETTY & n->flags))
                return;
 
        if (n->prev->tok == n->tok &&
@@ -299,13 +295,9 @@ synopsis_pre(struct html *h, const struct roff_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:
                print_paragraph(h);
                break;
@@ -381,9 +373,12 @@ print_mdoc_node(MDOC_ARGS)
        int              child;
        struct tag      *t;
 
+       if (n->flags & NODE_NOPRT)
+               return;
+
        child = 1;
        t = h->tags.head;
-       n->flags &= ~MDOC_ENDED;
+       n->flags &= ~NODE_ENDED;
 
        switch (n->type) {
        case ROFFT_TEXT:
@@ -394,17 +389,17 @@ print_mdoc_node(MDOC_ARGS)
                 * Make sure that if we're in a literal mode already
                 * (i.e., within a <PRE>) don't print the newline.
                 */
-               if (' ' == *n->string && MDOC_LINE & n->flags)
+               if (' ' == *n->string && NODE_LINE & n->flags)
                        if ( ! (HTML_LITERAL & h->flags))
                                print_otag(h, TAG_BR, 0, NULL);
-               if (MDOC_DELIMC & n->flags)
+               if (NODE_DELIMC & n->flags)
                        h->flags |= HTML_NOSPACE;
                print_text(h, n->string);
-               if (MDOC_DELIMO & n->flags)
+               if (NODE_DELIMO & n->flags)
                        h->flags |= HTML_NOSPACE;
                return;
        case ROFFT_EQN:
-               if (n->flags & MDOC_LINE)
+               if (n->flags & NODE_LINE)
                        putchar('\n');
                print_eqn(h, n->eqn);
                break;
@@ -432,7 +427,7 @@ print_mdoc_node(MDOC_ARGS)
                break;
        }
 
-       if (h->flags & HTML_KEEP && n->flags & MDOC_LINE) {
+       if (h->flags & HTML_KEEP && n->flags & NODE_LINE) {
                h->flags &= ~HTML_KEEP;
                h->flags |= HTML_PREKEEP;
        }
@@ -446,11 +441,11 @@ print_mdoc_node(MDOC_ARGS)
        case ROFFT_EQN:
                break;
        default:
-               if ( ! mdocs[n->tok].post || n->flags & MDOC_ENDED)
+               if ( ! mdocs[n->tok].post || n->flags & NODE_ENDED)
                        break;
                (*mdocs[n->tok].post)(meta, n, h);
                if (n->end != ENDBODY_NOT)
-                       n->body->flags |= MDOC_ENDED;
+                       n->body->flags |= NODE_ENDED;
                if (n->end == ENDBODY_NOSPACE)
                        h->flags |= HTML_NOSPACE;
                break;
@@ -524,7 +519,7 @@ mdoc_root_pre(MDOC_ARGS)
 
        free(title);
        free(volume);
-       return(1);
+       return 1;
 }
 
 static int
@@ -536,17 +531,16 @@ mdoc_sh_pre(MDOC_ARGS)
        case ROFFT_BLOCK:
                PAIR_CLASS_INIT(&tag, "section");
                print_otag(h, TAG_DIV, 1, &tag);
-               return(1);
+               return 1;
        case ROFFT_BODY:
                if (n->sec == SEC_AUTHORS)
                        h->flags &= ~(HTML_SPLIT|HTML_NOSPLIT);
-               return(1);
+               return 1;
        default:
                break;
        }
 
        bufinit(h);
-       bufcat(h, "x");
 
        for (n = n->child; n != NULL && n->type == ROFFT_TEXT; ) {
                bufcat_id(h, n->string);
@@ -560,7 +554,7 @@ mdoc_sh_pre(MDOC_ARGS)
        } else
                print_otag(h, TAG_H1, 0, NULL);
 
-       return(1);
+       return 1;
 }
 
 static int
@@ -571,12 +565,11 @@ mdoc_ss_pre(MDOC_ARGS)
        if (n->type == ROFFT_BLOCK) {
                PAIR_CLASS_INIT(&tag, "subsection");
                print_otag(h, TAG_DIV, 1, &tag);
-               return(1);
+               return 1;
        } else if (n->type == ROFFT_BODY)
-               return(1);
+               return 1;
 
        bufinit(h);
-       bufcat(h, "x");
 
        for (n = n->child; n != NULL && n->type == ROFFT_TEXT; ) {
                bufcat_id(h, n->string);
@@ -590,7 +583,7 @@ mdoc_ss_pre(MDOC_ARGS)
        } else
                print_otag(h, TAG_H2, 0, NULL);
 
-       return(1);
+       return 1;
 }
 
 static int
@@ -604,17 +597,17 @@ mdoc_fl_pre(MDOC_ARGS)
        /* `Cm' has no leading hyphen. */
 
        if (MDOC_Cm == n->tok)
-               return(1);
+               return 1;
 
        print_text(h, "\\-");
 
-       if ( ! (n->nchild == 0 &&
+       if (!(n->child == NULL &&
            (n->next == NULL ||
             n->next->type == ROFFT_TEXT ||
-            n->next->flags & MDOC_LINE)))
+            n->next->flags & NODE_LINE)))
                h->flags |= HTML_NOSPACE;
 
-       return(1);
+       return 1;
 }
 
 static int
@@ -623,14 +616,14 @@ mdoc_nd_pre(MDOC_ARGS)
        struct htmlpair  tag;
 
        if (n->type != ROFFT_BODY)
-               return(1);
+               return 1;
 
        /* XXX: this tag in theory can contain block elements. */
 
        print_text(h, "\\(em");
        PAIR_CLASS_INIT(&tag, "desc");
        print_otag(h, TAG_SPAN, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -649,10 +642,10 @@ mdoc_nm_pre(MDOC_ARGS)
                print_otag(h, TAG_B, 1, &tag);
                if (n->child == NULL && meta->name != NULL)
                        print_text(h, meta->name);
-               return(1);
+               return 1;
        case ROFFT_BODY:
                print_otag(h, TAG_TD, 0, NULL);
-               return(1);
+               return 1;
        default:
                break;
        }
@@ -676,7 +669,7 @@ mdoc_nm_pre(MDOC_ARGS)
        print_otag(h, TAG_COL, 0, NULL);
        print_otag(h, TAG_TBODY, 0, NULL);
        print_otag(h, TAG_TR, 0, NULL);
-       return(1);
+       return 1;
 }
 
 static int
@@ -685,7 +678,7 @@ mdoc_xr_pre(MDOC_ARGS)
        struct htmlpair  tag[2];
 
        if (NULL == n->child)
-               return(0);
+               return 0;
 
        PAIR_CLASS_INIT(&tag[0], "link-man");
 
@@ -702,7 +695,7 @@ mdoc_xr_pre(MDOC_ARGS)
        print_text(h, n->string);
 
        if (NULL == (n = n->next))
-               return(0);
+               return 0;
 
        h->flags |= HTML_NOSPACE;
        print_text(h, "(");
@@ -710,16 +703,16 @@ mdoc_xr_pre(MDOC_ARGS)
        print_text(h, n->string);
        h->flags |= HTML_NOSPACE;
        print_text(h, ")");
-       return(0);
+       return 0;
 }
 
 static int
 mdoc_ns_pre(MDOC_ARGS)
 {
 
-       if ( ! (MDOC_LINE & n->flags))
+       if ( ! (NODE_LINE & n->flags))
                h->flags |= HTML_NOSPACE;
-       return(1);
+       return 1;
 }
 
 static int
@@ -729,77 +722,17 @@ mdoc_ar_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "arg");
        print_otag(h, TAG_I, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
 mdoc_xx_pre(MDOC_ARGS)
 {
-       const char      *pp;
        struct htmlpair  tag;
-       int              flags;
-
-       switch (n->tok) {
-       case MDOC_Bsx:
-               pp = "BSD/OS";
-               break;
-       case MDOC_Dx:
-               pp = "DragonFly";
-               break;
-       case MDOC_Fx:
-               pp = "FreeBSD";
-               break;
-       case MDOC_Nx:
-               pp = "NetBSD";
-               break;
-       case MDOC_Ox:
-               pp = "OpenBSD";
-               break;
-       case MDOC_Ux:
-               pp = "UNIX";
-               break;
-       default:
-               return(1);
-       }
 
        PAIR_CLASS_INIT(&tag, "unix");
        print_otag(h, TAG_SPAN, 1, &tag);
-
-       print_text(h, pp);
-       if (n->child) {
-               flags = h->flags;
-               h->flags |= HTML_KEEP;
-               print_text(h, n->child->string);
-               h->flags = flags;
-       }
-       return(0);
-}
-
-static int
-mdoc_bx_pre(MDOC_ARGS)
-{
-       struct htmlpair  tag;
-
-       PAIR_CLASS_INIT(&tag, "unix");
-       print_otag(h, TAG_SPAN, 1, &tag);
-
-       if (NULL != (n = n->child)) {
-               print_text(h, n->string);
-               h->flags |= HTML_NOSPACE;
-               print_text(h, "BSD");
-       } else {
-               print_text(h, "BSD");
-               return(0);
-       }
-
-       if (NULL != (n = n->next)) {
-               h->flags |= HTML_NOSPACE;
-               print_text(h, "-");
-               h->flags |= HTML_NOSPACE;
-               print_text(h, n->string);
-       }
-
-       return(0);
+       return 1;
 }
 
 static int
@@ -826,23 +759,15 @@ mdoc_it_pre(MDOC_ARGS)
        if (n->type == ROFFT_HEAD) {
                switch (type) {
                case LIST_bullet:
-                       /* FALLTHROUGH */
                case LIST_dash:
-                       /* FALLTHROUGH */
                case LIST_item:
-                       /* FALLTHROUGH */
                case LIST_hyphen:
-                       /* FALLTHROUGH */
                case LIST_enum:
-                       return(0);
+                       return 0;
                case LIST_diag:
-                       /* FALLTHROUGH */
                case LIST_hang:
-                       /* FALLTHROUGH */
                case LIST_inset:
-                       /* FALLTHROUGH */
                case LIST_ohang:
-                       /* FALLTHROUGH */
                case LIST_tag:
                        SCALE_VS_INIT(&su, ! bl->norm->Bl.comp);
                        bufcat_su(h, "margin-top", &su);
@@ -861,13 +786,9 @@ mdoc_it_pre(MDOC_ARGS)
        } else if (n->type == ROFFT_BODY) {
                switch (type) {
                case LIST_bullet:
-                       /* FALLTHROUGH */
                case LIST_hyphen:
-                       /* FALLTHROUGH */
                case LIST_dash:
-                       /* FALLTHROUGH */
                case LIST_enum:
-                       /* FALLTHROUGH */
                case LIST_item:
                        SCALE_VS_INIT(&su, ! bl->norm->Bl.comp);
                        bufcat_su(h, "margin-top", &su);
@@ -875,13 +796,9 @@ mdoc_it_pre(MDOC_ARGS)
                        print_otag(h, TAG_LI, 2, tag);
                        break;
                case LIST_diag:
-                       /* FALLTHROUGH */
                case LIST_hang:
-                       /* FALLTHROUGH */
                case LIST_inset:
-                       /* FALLTHROUGH */
                case LIST_ohang:
-                       /* FALLTHROUGH */
                case LIST_tag:
                        if (NULL == bl->norm->Bl.width) {
                                print_otag(h, TAG_DD, 1, tag);
@@ -911,7 +828,7 @@ mdoc_it_pre(MDOC_ARGS)
                }
        }
 
-       return(1);
+       return 1;
 }
 
 static int
@@ -925,12 +842,12 @@ mdoc_bl_pre(MDOC_ARGS)
        if (n->type == ROFFT_BODY) {
                if (LIST_column == n->norm->Bl.type)
                        print_otag(h, TAG_TBODY, 0, NULL);
-               return(1);
+               return 1;
        }
 
        if (n->type == ROFFT_HEAD) {
                if (LIST_column != n->norm->Bl.type)
-                       return(0);
+                       return 0;
 
                /*
                 * For each column, print out the <COL> tag with our
@@ -950,7 +867,7 @@ mdoc_bl_pre(MDOC_ARGS)
                        print_otag(h, TAG_COL, 1, tag);
                }
 
-               return(0);
+               return 0;
        }
 
        SCALE_VS_INIT(&su, 0);
@@ -973,11 +890,8 @@ mdoc_bl_pre(MDOC_ARGS)
 
        switch (n->norm->Bl.type) {
        case LIST_bullet:
-               /* FALLTHROUGH */
        case LIST_dash:
-               /* FALLTHROUGH */
        case LIST_hyphen:
-               /* FALLTHROUGH */
        case LIST_item:
                print_otag(h, TAG_UL, 2, tag);
                break;
@@ -985,13 +899,9 @@ mdoc_bl_pre(MDOC_ARGS)
                print_otag(h, TAG_OL, 2, tag);
                break;
        case LIST_diag:
-               /* FALLTHROUGH */
        case LIST_hang:
-               /* FALLTHROUGH */
        case LIST_inset:
-               /* FALLTHROUGH */
        case LIST_ohang:
-               /* FALLTHROUGH */
        case LIST_tag:
                print_otag(h, TAG_DL, 2, tag);
                break;
@@ -1000,50 +910,17 @@ mdoc_bl_pre(MDOC_ARGS)
                break;
        default:
                abort();
-               /* NOTREACHED */
        }
 
-       return(1);
+       return 1;
 }
 
 static int
 mdoc_ex_pre(MDOC_ARGS)
 {
-       struct tag      *t;
-       struct htmlpair  tag;
-       int              nchild;
-
        if (n->prev)
                print_otag(h, TAG_BR, 0, NULL);
-
-       PAIR_CLASS_INIT(&tag, "utility");
-
-       print_text(h, "The");
-
-       nchild = n->nchild;
-       for (n = n->child; n; n = n->next) {
-               assert(n->type == ROFFT_TEXT);
-
-               t = print_otag(h, TAG_B, 1, &tag);
-               print_text(h, n->string);
-               print_tagq(h, t);
-
-               if (nchild > 2 && n->next) {
-                       h->flags |= HTML_NOSPACE;
-                       print_text(h, ",");
-               }
-
-               if (n->next && NULL == n->next->next)
-                       print_text(h, "and");
-       }
-
-       if (nchild > 1)
-               print_text(h, "utilities exit\\~0");
-       else
-               print_text(h, "utility exits\\~0");
-
-       print_text(h, "on success, and\\~>0 if an error occurs.");
-       return(0);
+       return 1;
 }
 
 static int
@@ -1053,7 +930,7 @@ mdoc_em_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "emph");
        print_otag(h, TAG_SPAN, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1063,7 +940,7 @@ mdoc_d1_pre(MDOC_ARGS)
        struct roffsu    su;
 
        if (n->type != ROFFT_BLOCK)
-               return(1);
+               return 1;
 
        SCALE_VS_INIT(&su, 0);
        bufinit(h);
@@ -1082,7 +959,7 @@ mdoc_d1_pre(MDOC_ARGS)
                print_otag(h, TAG_CODE, 1, tag);
        }
 
-       return(1);
+       return 1;
 }
 
 static int
@@ -1091,7 +968,7 @@ mdoc_sx_pre(MDOC_ARGS)
        struct htmlpair  tag[2];
 
        bufinit(h);
-       bufcat(h, "#x");
+       bufcat(h, "#");
 
        for (n = n->child; n; ) {
                bufcat_id(h, n->string);
@@ -1104,7 +981,7 @@ mdoc_sx_pre(MDOC_ARGS)
 
        print_otag(h, TAG_I, 1, tag);
        print_otag(h, TAG_A, 2, tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1116,7 +993,7 @@ mdoc_bd_pre(MDOC_ARGS)
        struct roffsu            su;
 
        if (n->type == ROFFT_HEAD)
-               return(0);
+               return 0;
 
        if (n->type == ROFFT_BLOCK) {
                comp = n->norm->Bd.comp;
@@ -1130,7 +1007,7 @@ mdoc_bd_pre(MDOC_ARGS)
                }
                if ( ! comp)
                        print_paragraph(h);
-               return(1);
+               return 1;
        }
 
        /* Handle the -offset argument. */
@@ -1149,15 +1026,15 @@ mdoc_bd_pre(MDOC_ARGS)
        bufcat_su(h, "margin-left", &su);
        PAIR_STYLE_INIT(&tag[0], h);
 
-       if (DISP_unfilled != n->norm->Bd.type &&
-           DISP_literal != n->norm->Bd.type) {
-               PAIR_CLASS_INIT(&tag[1], "display");
-               print_otag(h, TAG_DIV, 2, tag);
-               return(1);
-       }
+       PAIR_CLASS_INIT(&tag[1], "display");
+       print_otag(h, TAG_DIV, 2, tag);
+
+       if (n->norm->Bd.type != DISP_unfilled &&
+           n->norm->Bd.type != DISP_literal)
+               return 1;
 
-       PAIR_CLASS_INIT(&tag[1], "lit display");
-       print_otag(h, TAG_PRE, 2, tag);
+       PAIR_CLASS_INIT(&tag[0], "lit");
+       print_otag(h, TAG_PRE, 1, tag);
 
        /* This can be recursive: save & set our literal state. */
 
@@ -1174,26 +1051,19 @@ mdoc_bd_pre(MDOC_ARGS)
                 */
                switch (nn->tok) {
                case MDOC_Sm:
-                       /* FALLTHROUGH */
                case MDOC_br:
-                       /* FALLTHROUGH */
                case MDOC_sp:
-                       /* FALLTHROUGH */
                case MDOC_Bl:
-                       /* FALLTHROUGH */
                case MDOC_D1:
-                       /* FALLTHROUGH */
                case MDOC_Dl:
-                       /* FALLTHROUGH */
                case MDOC_Lp:
-                       /* FALLTHROUGH */
                case MDOC_Pp:
                        continue;
                default:
                        break;
                }
                if (h->flags & HTML_NONEWLINE ||
-                   (nn->next && ! (nn->next->flags & MDOC_LINE)))
+                   (nn->next && ! (nn->next->flags & NODE_LINE)))
                        continue;
                else if (nn->next)
                        print_text(h, "\n");
@@ -1204,7 +1074,7 @@ mdoc_bd_pre(MDOC_ARGS)
        if (0 == sv)
                h->flags &= ~HTML_LITERAL;
 
-       return(0);
+       return 0;
 }
 
 static int
@@ -1214,7 +1084,7 @@ mdoc_pa_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "file");
        print_otag(h, TAG_I, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1224,7 +1094,7 @@ mdoc_ad_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "addr");
        print_otag(h, TAG_I, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1235,12 +1105,12 @@ mdoc_an_pre(MDOC_ARGS)
        if (n->norm->An.auth == AUTH_split) {
                h->flags &= ~HTML_NOSPLIT;
                h->flags |= HTML_SPLIT;
-               return(0);
+               return 0;
        }
        if (n->norm->An.auth == AUTH_nosplit) {
                h->flags &= ~HTML_SPLIT;
                h->flags |= HTML_NOSPLIT;
-               return(0);
+               return 0;
        }
 
        if (h->flags & HTML_SPLIT)
@@ -1251,7 +1121,7 @@ mdoc_an_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "author");
        print_otag(h, TAG_SPAN, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1262,7 +1132,7 @@ mdoc_cd_pre(MDOC_ARGS)
        synopsis_pre(h, n);
        PAIR_CLASS_INIT(&tag, "config");
        print_otag(h, TAG_B, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1272,7 +1142,7 @@ mdoc_dv_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "define");
        print_otag(h, TAG_SPAN, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1282,7 +1152,7 @@ mdoc_ev_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "env");
        print_otag(h, TAG_SPAN, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1292,7 +1162,7 @@ mdoc_er_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "errno");
        print_otag(h, TAG_SPAN, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1305,7 +1175,7 @@ mdoc_fa_pre(MDOC_ARGS)
        PAIR_CLASS_INIT(&tag, "farg");
        if (n->parent->tok != MDOC_Fo) {
                print_otag(h, TAG_I, 1, &tag);
-               return(1);
+               return 1;
        }
 
        for (nn = n->child; nn; nn = nn->next) {
@@ -1323,7 +1193,7 @@ mdoc_fa_pre(MDOC_ARGS)
                print_text(h, ",");
        }
 
-       return(0);
+       return 0;
 }
 
 static int
@@ -1338,14 +1208,14 @@ mdoc_fd_pre(MDOC_ARGS)
        synopsis_pre(h, n);
 
        if (NULL == (n = n->child))
-               return(0);
+               return 0;
 
        assert(n->type == ROFFT_TEXT);
 
        if (strcmp(n->string, "#include")) {
                PAIR_CLASS_INIT(&tag[0], "macro");
                print_otag(h, TAG_B, 1, tag);
-               return(1);
+               return 1;
        }
 
        PAIR_CLASS_INIT(&tag[0], "includes");
@@ -1391,7 +1261,7 @@ mdoc_fd_pre(MDOC_ARGS)
                print_text(h, n->string);
        }
 
-       return(0);
+       return 0;
 }
 
 static int
@@ -1401,15 +1271,15 @@ mdoc_vt_pre(MDOC_ARGS)
 
        if (n->type == ROFFT_BLOCK) {
                synopsis_pre(h, n);
-               return(1);
+               return 1;
        } else if (n->type == ROFFT_ELEM) {
                synopsis_pre(h, n);
        } else if (n->type == ROFFT_HEAD)
-               return(0);
+               return 0;
 
        PAIR_CLASS_INIT(&tag, "type");
        print_otag(h, TAG_SPAN, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1420,7 +1290,7 @@ mdoc_ft_pre(MDOC_ARGS)
        synopsis_pre(h, n);
        PAIR_CLASS_INIT(&tag, "ftype");
        print_otag(h, TAG_I, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1432,7 +1302,7 @@ mdoc_fn_pre(MDOC_ARGS)
        const char      *sp, *ep;
        int              sz, i, pretty;
 
-       pretty = MDOC_SYNPRETTY & n->flags;
+       pretty = NODE_SYNPRETTY & n->flags;
        synopsis_pre(h, n);
 
        /* Split apart into type and name. */
@@ -1462,7 +1332,7 @@ mdoc_fn_pre(MDOC_ARGS)
         */
 
 #if 0
-       if (MDOC_SYNPRETTY & n->flags) {
+       if (NODE_SYNPRETTY & n->flags) {
                nbuf[0] = '\0';
                html_idcat(nbuf, sp, BUFSIZ);
                PAIR_ID_INIT(&tag[1], nbuf);
@@ -1491,7 +1361,7 @@ mdoc_fn_pre(MDOC_ARGS)
 
        for (n = n->child->next; n; n = n->next) {
                i = 1;
-               if (MDOC_SYNPRETTY & n->flags)
+               if (NODE_SYNPRETTY & n->flags)
                        i = 2;
                t = print_otag(h, TAG_I, i, tag);
                print_text(h, n->string);
@@ -1510,7 +1380,7 @@ mdoc_fn_pre(MDOC_ARGS)
                print_text(h, ";");
        }
 
-       return(0);
+       return 0;
 }
 
 static int
@@ -1527,14 +1397,14 @@ mdoc_sm_pre(MDOC_ARGS)
        if ( ! (HTML_NONOSPACE & h->flags))
                h->flags &= ~HTML_NOSPACE;
 
-       return(0);
+       return 0;
 }
 
 static int
 mdoc_skip_pre(MDOC_ARGS)
 {
 
-       return(0);
+       return 0;
 }
 
 static int
@@ -1542,7 +1412,7 @@ mdoc_pp_pre(MDOC_ARGS)
 {
 
        print_paragraph(h);
-       return(0);
+       return 0;
 }
 
 static int
@@ -1571,7 +1441,7 @@ mdoc_sp_pre(MDOC_ARGS)
        /* So the div isn't empty: */
        print_text(h, "\\~");
 
-       return(0);
+       return 0;
 
 }
 
@@ -1581,7 +1451,7 @@ mdoc_lk_pre(MDOC_ARGS)
        struct htmlpair  tag[2];
 
        if (NULL == (n = n->child))
-               return(0);
+               return 0;
 
        assert(n->type == ROFFT_TEXT);
 
@@ -1596,7 +1466,7 @@ mdoc_lk_pre(MDOC_ARGS)
        for (n = n->next; n; n = n->next)
                print_text(h, n->string);
 
-       return(0);
+       return 0;
 }
 
 static int
@@ -1620,7 +1490,7 @@ mdoc_mt_pre(MDOC_ARGS)
                print_tagq(h, t);
        }
 
-       return(0);
+       return 0;
 }
 
 static int
@@ -1633,21 +1503,21 @@ mdoc_fo_pre(MDOC_ARGS)
                h->flags |= HTML_NOSPACE;
                print_text(h, "(");
                h->flags |= HTML_NOSPACE;
-               return(1);
+               return 1;
        } else if (n->type == ROFFT_BLOCK) {
                synopsis_pre(h, n);
-               return(1);
+               return 1;
        }
 
        if (n->child == NULL)
-               return(0);
+               return 0;
 
        assert(n->child->string);
        PAIR_CLASS_INIT(&tag, "fname");
        t = print_otag(h, TAG_B, 1, &tag);
        print_text(h, n->child->string);
        print_tagq(h, t);
-       return(0);
+       return 0;
 }
 
 static void
@@ -1681,7 +1551,7 @@ mdoc_in_pre(MDOC_ARGS)
         * of no children.
         */
 
-       if (MDOC_SYNPRETTY & n->flags && MDOC_LINE & n->flags)
+       if (NODE_SYNPRETTY & n->flags && NODE_LINE & n->flags)
                print_text(h, "#include");
 
        print_text(h, "<");
@@ -1714,7 +1584,7 @@ mdoc_in_pre(MDOC_ARGS)
                print_text(h, n->string);
        }
 
-       return(0);
+       return 0;
 }
 
 static int
@@ -1724,63 +1594,7 @@ mdoc_ic_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "cmd");
        print_otag(h, TAG_B, 1, &tag);
-       return(1);
-}
-
-static int
-mdoc_rv_pre(MDOC_ARGS)
-{
-       struct htmlpair  tag;
-       struct tag      *t;
-       int              nchild;
-
-       if (n->prev)
-               print_otag(h, TAG_BR, 0, NULL);
-
-       PAIR_CLASS_INIT(&tag, "fname");
-
-       nchild = n->nchild;
-       if (nchild > 0) {
-               print_text(h, "The");
-
-               for (n = n->child; n; n = n->next) {
-                       t = print_otag(h, TAG_B, 1, &tag);
-                       print_text(h, n->string);
-                       print_tagq(h, t);
-
-                       h->flags |= HTML_NOSPACE;
-                       print_text(h, "()");
-
-                       if (n->next == NULL)
-                               continue;
-
-                       if (nchild > 2) {
-                               h->flags |= HTML_NOSPACE;
-                               print_text(h, ",");
-                       }
-                       if (n->next->next == NULL)
-                               print_text(h, "and");
-               }
-
-               if (nchild > 1)
-                       print_text(h, "functions return");
-               else
-                       print_text(h, "function returns");
-
-               print_text(h, "the value\\~0 if successful;");
-       } else
-               print_text(h, "Upon successful completion,"
-                    " the value\\~0 is returned;");
-
-       print_text(h, "otherwise the value\\~\\-1 is returned"
-          " and the global variable");
-
-       PAIR_CLASS_INIT(&tag, "var");
-       t = print_otag(h, TAG_B, 1, &tag);
-       print_text(h, "errno");
-       print_tagq(h, t);
-       print_text(h, "is set to indicate the error.");
-       return(0);
+       return 1;
 }
 
 static int
@@ -1790,7 +1604,7 @@ mdoc_va_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "var");
        print_otag(h, TAG_B, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1800,7 +1614,7 @@ mdoc_ap_pre(MDOC_ARGS)
        h->flags |= HTML_NOSPACE;
        print_text(h, "\\(aq");
        h->flags |= HTML_NOSPACE;
-       return(1);
+       return 1;
 }
 
 static int
@@ -1810,9 +1624,9 @@ mdoc_bf_pre(MDOC_ARGS)
        struct roffsu    su;
 
        if (n->type == ROFFT_HEAD)
-               return(0);
+               return 0;
        else if (n->type != ROFFT_BODY)
-               return(1);
+               return 1;
 
        if (FONT_Em == n->norm->Bf.font)
                PAIR_CLASS_INIT(&tag[0], "emph");
@@ -1834,7 +1648,7 @@ mdoc_bf_pre(MDOC_ARGS)
        bufcat_su(h, "margin-left", &su);
        PAIR_STYLE_INIT(&tag[1], h);
        print_otag(h, TAG_DIV, 2, tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1844,7 +1658,7 @@ mdoc_ms_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "symb");
        print_otag(h, TAG_SPAN, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1852,14 +1666,14 @@ mdoc_igndelim_pre(MDOC_ARGS)
 {
 
        h->flags |= HTML_IGNDELIM;
-       return(1);
+       return 1;
 }
 
 static void
 mdoc_pf_post(MDOC_ARGS)
 {
 
-       if ( ! (n->next == NULL || n->next->flags & MDOC_LINE))
+       if ( ! (n->next == NULL || n->next->flags & NODE_LINE))
                h->flags |= HTML_NOSPACE;
 }
 
@@ -1869,14 +1683,14 @@ mdoc_rs_pre(MDOC_ARGS)
        struct htmlpair  tag;
 
        if (n->type != ROFFT_BLOCK)
-               return(1);
+               return 1;
 
        if (n->prev && SEC_SEE_ALSO == n->sec)
                print_paragraph(h);
 
        PAIR_CLASS_INIT(&tag, "ref");
        print_otag(h, TAG_SPAN, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1885,8 +1699,8 @@ mdoc_no_pre(MDOC_ARGS)
        struct htmlpair tag;
 
        PAIR_CLASS_INIT(&tag, "none");
-       print_otag(h, TAG_CODE, 1, &tag);
-       return(1);
+       print_otag(h, TAG_SPAN, 1, &tag);
+       return 1;
 }
 
 static int
@@ -1896,7 +1710,7 @@ mdoc_li_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "lit");
        print_otag(h, TAG_CODE, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1906,23 +1720,7 @@ mdoc_sy_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "symb");
        print_otag(h, TAG_SPAN, 1, &tag);
-       return(1);
-}
-
-static int
-mdoc_bt_pre(MDOC_ARGS)
-{
-
-       print_text(h, "is currently in beta test.");
-       return(0);
-}
-
-static int
-mdoc_ud_pre(MDOC_ARGS)
-{
-
-       print_text(h, "currently under development.");
-       return(0);
+       return 1;
 }
 
 static int
@@ -1930,12 +1728,12 @@ mdoc_lb_pre(MDOC_ARGS)
 {
        struct htmlpair tag;
 
-       if (SEC_LIBRARY == n->sec && MDOC_LINE & n->flags && n->prev)
+       if (SEC_LIBRARY == n->sec && NODE_LINE & n->flags && n->prev)
                print_otag(h, TAG_BR, 0, NULL);
 
        PAIR_CLASS_INIT(&tag, "lib");
        print_otag(h, TAG_SPAN, 1, &tag);
-       return(1);
+       return 1;
 }
 
 static int
@@ -1997,18 +1795,17 @@ mdoc__x_pre(MDOC_ARGS)
                break;
        default:
                abort();
-               /* NOTREACHED */
        }
 
        if (MDOC__U != n->tok) {
                print_otag(h, t, 1, tag);
-               return(1);
+               return 1;
        }
 
        PAIR_HREF_INIT(&tag[1], n->child->string);
        print_otag(h, TAG_A, 2, tag);
 
-       return(1);
+       return 1;
 }
 
 static void
@@ -2037,17 +1834,16 @@ mdoc_bk_pre(MDOC_ARGS)
        case ROFFT_BLOCK:
                break;
        case ROFFT_HEAD:
-               return(0);
+               return 0;
        case ROFFT_BODY:
-               if (n->parent->args || 0 == n->prev->nchild)
+               if (n->parent->args != NULL || n->prev->child == NULL)
                        h->flags |= HTML_PREKEEP;
                break;
        default:
                abort();
-               /* NOTREACHED */
        }
 
-       return(1);
+       return 1;
 }
 
 static void
@@ -2064,27 +1860,23 @@ mdoc_quote_pre(MDOC_ARGS)
        struct htmlpair tag;
 
        if (n->type != ROFFT_BODY)
-               return(1);
+               return 1;
 
        switch (n->tok) {
        case MDOC_Ao:
-               /* FALLTHROUGH */
        case MDOC_Aq:
-               print_text(h, n->nchild == 1 &&
+               print_text(h, n->child != NULL && n->child->next == NULL &&
                    n->child->tok == MDOC_Mt ?  "<" : "\\(la");
                break;
        case MDOC_Bro:
-               /* FALLTHROUGH */
        case MDOC_Brq:
                print_text(h, "\\(lC");
                break;
        case MDOC_Bo:
-               /* FALLTHROUGH */
        case MDOC_Bq:
                print_text(h, "\\(lB");
                break;
        case MDOC_Oo:
-               /* FALLTHROUGH */
        case MDOC_Op:
                print_text(h, "\\(lB");
                h->flags |= HTML_NOSPACE;
@@ -2094,20 +1886,16 @@ mdoc_quote_pre(MDOC_ARGS)
        case MDOC_En:
                if (NULL == n->norm->Es ||
                    NULL == n->norm->Es->child)
-                       return(1);
+                       return 1;
                print_text(h, n->norm->Es->child->string);
                break;
        case MDOC_Do:
-               /* FALLTHROUGH */
        case MDOC_Dq:
-               /* FALLTHROUGH */
        case MDOC_Qo:
-               /* FALLTHROUGH */
        case MDOC_Qq:
                print_text(h, "\\(lq");
                break;
        case MDOC_Po:
-               /* FALLTHROUGH */
        case MDOC_Pq:
                print_text(h, "(");
                break;
@@ -2118,17 +1906,15 @@ mdoc_quote_pre(MDOC_ARGS)
                print_otag(h, TAG_CODE, 1, &tag);
                break;
        case MDOC_So:
-               /* FALLTHROUGH */
        case MDOC_Sq:
                print_text(h, "\\(oq");
                break;
        default:
                abort();
-               /* NOTREACHED */
        }
 
        h->flags |= HTML_NOSPACE;
-       return(1);
+       return 1;
 }
 
 static void
@@ -2142,22 +1928,17 @@ mdoc_quote_post(MDOC_ARGS)
 
        switch (n->tok) {
        case MDOC_Ao:
-               /* FALLTHROUGH */
        case MDOC_Aq:
-               print_text(h, n->nchild == 1 &&
+               print_text(h, n->child != NULL && n->child->next == NULL &&
                    n->child->tok == MDOC_Mt ?  ">" : "\\(ra");
                break;
        case MDOC_Bro:
-               /* FALLTHROUGH */
        case MDOC_Brq:
                print_text(h, "\\(rC");
                break;
        case MDOC_Oo:
-               /* FALLTHROUGH */
        case MDOC_Op:
-               /* FALLTHROUGH */
        case MDOC_Bo:
-               /* FALLTHROUGH */
        case MDOC_Bq:
                print_text(h, "\\(rB");
                break;
@@ -2170,29 +1951,22 @@ mdoc_quote_post(MDOC_ARGS)
                        print_text(h, n->norm->Es->child->next->string);
                break;
        case MDOC_Qo:
-               /* FALLTHROUGH */
        case MDOC_Qq:
-               /* FALLTHROUGH */
        case MDOC_Do:
-               /* FALLTHROUGH */
        case MDOC_Dq:
                print_text(h, "\\(rq");
                break;
        case MDOC_Po:
-               /* FALLTHROUGH */
        case MDOC_Pq:
                print_text(h, ")");
                break;
        case MDOC_Ql:
-               /* FALLTHROUGH */
        case MDOC_So:
-               /* FALLTHROUGH */
        case MDOC_Sq:
                print_text(h, "\\(cq");
                break;
        default:
                abort();
-               /* NOTREACHED */
        }
 }
 
@@ -2201,7 +1975,7 @@ mdoc_eo_pre(MDOC_ARGS)
 {
 
        if (n->type != ROFFT_BODY)
-               return(1);
+               return 1;
 
        if (n->end == ENDBODY_NOT &&
            n->parent->head->child == NULL &&
@@ -2212,7 +1986,7 @@ mdoc_eo_pre(MDOC_ARGS)
            n->parent->head->child != NULL && (n->child != NULL ||
            (n->parent->tail != NULL && n->parent->tail->child != NULL)))
                h->flags |= HTML_NOSPACE;
-       return(1);
+       return 1;
 }
 
 static void