]> git.cameronkatri.com Git - mandoc.git/blobdiff - mdoc_html.c
Use C99 uint32_t, not BSD-style u_int32_t.
[mandoc.git] / mdoc_html.c
index 15d87f7e8bd371dc10cd787965d403126c07afb9..a7aa722d94c43e737192d8ebd7b1fb4eb0757531 100644 (file)
@@ -1,6 +1,6 @@
-/*     $Id: mdoc_html.c,v 1.144 2011/01/13 14:30:13 kristaps Exp $ */
+/*     $Id: mdoc_html.c,v 1.186 2013/12/24 20:45:27 schwarze Exp $ */
 /*
- * Copyright (c) 2008, 2009, 2010 Kristaps Dzonsons <kristaps@bsd.lv>
+ * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  *
  * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
@@ -34,9 +34,8 @@
 #include "main.h"
 
 #define        INDENT           5
-#define        HALFINDENT       3
 
-#define        MDOC_ARGS         const struct mdoc_meta *m, \
+#define        MDOC_ARGS         const struct mdoc_meta *meta, \
                          const struct mdoc_node *n, \
                          struct html *h
 
@@ -190,7 +189,7 @@ static      const struct htmlmdoc mdocs[MDOC_MAX] = {
        {NULL, NULL}, /* Ec */ /* FIXME: no space */
        {NULL, NULL}, /* Ef */
        {mdoc_em_pre, NULL}, /* Em */ 
-       {NULL, NULL}, /* Eo */
+       {mdoc_quote_pre, mdoc_quote_post}, /* Eo */
        {mdoc_xx_pre, NULL}, /* Fx */
        {mdoc_ms_pre, NULL}, /* Ms */
        {mdoc_igndelim_pre, NULL}, /* No */
@@ -261,19 +260,12 @@ static    const char * const lists[LIST_MAX] = {
 };
 
 void
-html_mdoc(void *arg, const struct mdoc *m)
+html_mdoc(void *arg, const struct mdoc *mdoc)
 {
-       struct html     *h;
-       struct tag      *t;
-
-       h = (struct html *)arg;
-
-       print_gen_decls(h);
-       t = print_otag(h, TAG_HTML, 0, NULL);
-       print_mdoc(mdoc_meta(m), mdoc_node(m), h);
-       print_tagq(h, t);
 
-       printf("\n");
+       print_mdoc(mdoc_meta(mdoc), mdoc_node(mdoc),
+                       (struct html *)arg);
+       putchar('\n');
 }
 
 
@@ -288,7 +280,7 @@ a2width(const char *p, struct roffsu *su)
 
        if ( ! a2roffsu(p, su, SCALE_MAX)) {
                su->unit = SCALE_BU;
-               su->scale = (int)strlen(p);
+               su->scale = html_strlen(p);
        }
 }
 
@@ -353,24 +345,31 @@ a2offs(const char *p, struct roffsu *su)
                SCALE_HS_INIT(su, INDENT);
        else if (0 == strcmp(p, "indent-two"))
                SCALE_HS_INIT(su, INDENT * 2);
-       else if ( ! a2roffsu(p, su, SCALE_MAX)) {
-               su->unit = SCALE_BU;
-               su->scale = (int)strlen(p);
-       }
+       else if ( ! a2roffsu(p, su, SCALE_MAX))
+               SCALE_HS_INIT(su, html_strlen(p));
 }
 
 
 static void
 print_mdoc(MDOC_ARGS)
 {
-       struct tag      *t;
+       struct tag      *t, *tt;
+       struct htmlpair  tag;
 
-       t = print_otag(h, TAG_HEAD, 0, NULL);
-       print_mdoc_head(m, n, h);
-       print_tagq(h, t);
+       PAIR_CLASS_INIT(&tag, "mandoc");
 
-       t = print_otag(h, TAG_BODY, 0, NULL);
-       print_mdoc_nodelist(m, n, h);
+       if ( ! (HTML_FRAGMENT & h->oflags)) {
+               print_gen_decls(h);
+               t = print_otag(h, TAG_HTML, 0, NULL);
+               tt = print_otag(h, TAG_HEAD, 0, NULL);
+               print_mdoc_head(meta, n, h);
+               print_tagq(h, tt);
+               print_otag(h, TAG_BODY, 0, NULL);
+               print_otag(h, TAG_DIV, 1, &tag);
+       } else 
+               t = print_otag(h, TAG_DIV, 1, &tag);
+
+       print_mdoc_nodelist(meta, n, h);
        print_tagq(h, t);
 }
 
@@ -382,13 +381,10 @@ print_mdoc_head(MDOC_ARGS)
 
        print_gen_head(h);
        bufinit(h);
-       buffmt(h, "%s(%s)", m->title, m->msec);
+       bufcat_fmt(h, "%s(%s)", meta->title, meta->msec);
 
-       if (m->arch) {
-               bufcat(h, " (");
-               bufcat(h, m->arch);
-               bufcat(h, ")");
-       }
+       if (meta->arch)
+               bufcat_fmt(h, " (%s)", meta->arch);
 
        print_otag(h, TAG_TITLE, 0, NULL);
        print_text(h, h->buf);
@@ -399,9 +395,9 @@ static void
 print_mdoc_nodelist(MDOC_ARGS)
 {
 
-       print_mdoc_node(m, n, h);
+       print_mdoc_node(meta, n, h);
        if (n->next)
-               print_mdoc_nodelist(m, n->next, h);
+               print_mdoc_nodelist(meta, n->next, h);
 }
 
 
@@ -414,18 +410,30 @@ print_mdoc_node(MDOC_ARGS)
        child = 1;
        t = h->tags.head;
 
-       bufinit(h);
        switch (n->type) {
        case (MDOC_ROOT):
-               child = mdoc_root_pre(m, n, h);
+               child = mdoc_root_pre(meta, n, h);
                break;
        case (MDOC_TEXT):
                /* No tables in this mode... */
                assert(NULL == h->tblt);
+
+               /*
+                * 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)
-                       print_otag(h, TAG_BR, 0, NULL);
+                       if ( ! (HTML_LITERAL & h->flags))
+                               print_otag(h, TAG_BR, 0, NULL);
+               if (MDOC_DELIMC & n->flags)
+                       h->flags |= HTML_NOSPACE;
                print_text(h, n->string);
+               if (MDOC_DELIMO & n->flags)
+                       h->flags |= HTML_NOSPACE;
                return;
+       case (MDOC_EQN):
+               print_eqn(h, n->eqn);
+               break;
        case (MDOC_TBL):
                /*
                 * This will take care of initialising all of the table
@@ -447,35 +455,32 @@ print_mdoc_node(MDOC_ARGS)
 
                assert(NULL == h->tblt);
                if (mdocs[n->tok].pre && ENDBODY_NOT == n->end)
-                       child = (*mdocs[n->tok].pre)(m, n, h);
+                       child = (*mdocs[n->tok].pre)(meta, n, h);
                break;
        }
 
        if (HTML_KEEP & h->flags) {
-               if (n->prev && n->prev->line != n->line) {
+               if (n->prev ? (n->prev->lastline != n->line) :
+                   (n->parent && n->parent->line != n->line)) {
                        h->flags &= ~HTML_KEEP;
                        h->flags |= HTML_PREKEEP;
-               } else if (NULL == n->prev) {
-                       if (n->parent && n->parent->line != n->line) {
-                               h->flags &= ~HTML_KEEP;
-                               h->flags |= HTML_PREKEEP;
-                       }
                }
        }
 
        if (child && n->child)
-               print_mdoc_nodelist(m, n->child, h);
+               print_mdoc_nodelist(meta, n->child, h);
 
        print_stagq(h, t);
 
-       bufinit(h);
        switch (n->type) {
        case (MDOC_ROOT):
-               mdoc_root_post(m, n, h);
+               mdoc_root_post(meta, n, h);
+               break;
+       case (MDOC_EQN):
                break;
        default:
                if (mdocs[n->tok].post && ENDBODY_NOT == n->end)
-                       (*mdocs[n->tok].post)(m, n, h);
+                       (*mdocs[n->tok].post)(meta, n, h);
                break;
        }
 }
@@ -486,39 +491,28 @@ mdoc_root_post(MDOC_ARGS)
 {
        struct htmlpair  tag[3];
        struct tag      *t, *tt;
-       char             b[DATESIZ];
-
-       time2a(m->date, b, DATESIZ);
 
        PAIR_SUMMARY_INIT(&tag[0], "Document Footer");
        PAIR_CLASS_INIT(&tag[1], "foot");
-       if (NULL == h->style) {
-               PAIR_INIT(&tag[2], ATTR_WIDTH, "100%");
-               t = print_otag(h, TAG_TABLE, 3, tag);
-               PAIR_INIT(&tag[0], ATTR_WIDTH, "50%");
-               print_otag(h, TAG_COL, 1, tag);
-               print_otag(h, TAG_COL, 1, tag);
-       } else
-               t = print_otag(h, TAG_TABLE, 2, tag);
+       PAIR_INIT(&tag[2], ATTR_WIDTH, "100%");
+       t = print_otag(h, TAG_TABLE, 3, tag);
+       PAIR_INIT(&tag[0], ATTR_WIDTH, "50%");
+       print_otag(h, TAG_COL, 1, tag);
+       print_otag(h, TAG_COL, 1, tag);
 
-       t = print_otag(h, TAG_TBODY, 0, NULL);
+       print_otag(h, TAG_TBODY, 0, NULL);
 
        tt = print_otag(h, TAG_TR, 0, NULL);
 
        PAIR_CLASS_INIT(&tag[0], "foot-date");
        print_otag(h, TAG_TD, 1, tag);
-
-       print_text(h, b);
+       print_text(h, meta->date);
        print_stagq(h, tt);
 
        PAIR_CLASS_INIT(&tag[0], "foot-os");
-       if (NULL == h->style) {
-               PAIR_INIT(&tag[1], ATTR_ALIGN, "right");
-               print_otag(h, TAG_TD, 2, tag);
-       } else 
-               print_otag(h, TAG_TD, 1, tag);
-
-       print_text(h, m->os);
+       PAIR_INIT(&tag[1], ATTR_ALIGN, "right");
+       print_otag(h, TAG_TD, 2, tag);
+       print_text(h, meta->os);
        print_tagq(h, t);
 }
 
@@ -531,27 +525,24 @@ mdoc_root_pre(MDOC_ARGS)
        struct tag      *t, *tt;
        char             b[BUFSIZ], title[BUFSIZ];
 
-       strlcpy(b, m->vol, BUFSIZ);
+       strlcpy(b, meta->vol, BUFSIZ);
 
-       if (m->arch) {
+       if (meta->arch) {
                strlcat(b, " (", BUFSIZ);
-               strlcat(b, m->arch, BUFSIZ);
+               strlcat(b, meta->arch, BUFSIZ);
                strlcat(b, ")", BUFSIZ);
        }
 
-       snprintf(title, BUFSIZ - 1, "%s(%s)", m->title, m->msec);
+       snprintf(title, BUFSIZ - 1, "%s(%s)", meta->title, meta->msec);
 
        PAIR_SUMMARY_INIT(&tag[0], "Document Header");
        PAIR_CLASS_INIT(&tag[1], "head");
-       if (NULL == h->style) {
-               PAIR_INIT(&tag[2], ATTR_WIDTH, "100%");
-               t = print_otag(h, TAG_TABLE, 3, tag);
-               PAIR_INIT(&tag[0], ATTR_WIDTH, "30%");
-               print_otag(h, TAG_COL, 1, tag);
-               print_otag(h, TAG_COL, 1, tag);
-               print_otag(h, TAG_COL, 1, tag);
-       } else
-               t = print_otag(h, TAG_TABLE, 2, tag);
+       PAIR_INIT(&tag[2], ATTR_WIDTH, "100%");
+       t = print_otag(h, TAG_TABLE, 3, tag);
+       PAIR_INIT(&tag[0], ATTR_WIDTH, "30%");
+       print_otag(h, TAG_COL, 1, tag);
+       print_otag(h, TAG_COL, 1, tag);
+       print_otag(h, TAG_COL, 1, tag);
 
        print_otag(h, TAG_TBODY, 0, NULL);
 
@@ -559,27 +550,18 @@ mdoc_root_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag[0], "head-ltitle");
        print_otag(h, TAG_TD, 1, tag);
-
        print_text(h, title);
        print_stagq(h, tt);
 
        PAIR_CLASS_INIT(&tag[0], "head-vol");
-       if (NULL == h->style) {
-               PAIR_INIT(&tag[1], ATTR_ALIGN, "center");
-               print_otag(h, TAG_TD, 2, tag);
-       } else 
-               print_otag(h, TAG_TD, 1, tag);
-
+       PAIR_INIT(&tag[1], ATTR_ALIGN, "center");
+       print_otag(h, TAG_TD, 2, tag);
        print_text(h, b);
        print_stagq(h, tt);
 
        PAIR_CLASS_INIT(&tag[0], "head-rtitle");
-       if (NULL == h->style) {
-               PAIR_INIT(&tag[1], ATTR_ALIGN, "right");
-               print_otag(h, TAG_TD, 2, tag);
-       } else 
-               print_otag(h, TAG_TD, 1, tag);
-
+       PAIR_INIT(&tag[1], ATTR_ALIGN, "right");
+       print_otag(h, TAG_TD, 2, tag);
        print_text(h, title);
        print_tagq(h, t);
        return(1);
@@ -591,7 +573,6 @@ static int
 mdoc_sh_pre(MDOC_ARGS)
 {
        struct htmlpair  tag;
-       char             buf[BUFSIZ];
 
        if (MDOC_BLOCK == n->type) {
                PAIR_CLASS_INIT(&tag, "section");
@@ -600,25 +581,29 @@ mdoc_sh_pre(MDOC_ARGS)
        } else if (MDOC_BODY == n->type)
                return(1);
 
-       buf[0] = '\0';
-       for (n = n->child; n; n = n->next) {
-               html_idcat(buf, n->string, BUFSIZ);
-               if (n->next)
-                       html_idcat(buf, " ", BUFSIZ);
+       bufinit(h);
+       bufcat(h, "x");
+
+       for (n = n->child; n && MDOC_TEXT == n->type; ) {
+               bufcat_id(h, n->string);
+               if (NULL != (n = n->next))
+                       bufcat_id(h, " ");
        }
 
-       PAIR_ID_INIT(&tag, buf);
-       print_otag(h, TAG_H1, 1, &tag);
+       if (NULL == n) {
+               PAIR_ID_INIT(&tag, h->buf);
+               print_otag(h, TAG_H1, 1, &tag);
+       } else
+               print_otag(h, TAG_H1, 0, NULL);
+
        return(1);
 }
 
-
 /* ARGSUSED */
 static int
 mdoc_ss_pre(MDOC_ARGS)
 {
        struct htmlpair  tag;
-       char             buf[BUFSIZ];
 
        if (MDOC_BLOCK == n->type) {
                PAIR_CLASS_INIT(&tag, "subsection");
@@ -627,15 +612,21 @@ mdoc_ss_pre(MDOC_ARGS)
        } else if (MDOC_BODY == n->type)
                return(1);
 
-       buf[0] = '\0';
-       for (n = n->child; n; n = n->next) {
-               html_idcat(buf, n->string, BUFSIZ);
-               if (n->next)
-                       html_idcat(buf, " ", BUFSIZ);
+       bufinit(h);
+       bufcat(h, "x");
+
+       for (n = n->child; n && MDOC_TEXT == n->type; ) {
+               bufcat_id(h, n->string);
+               if (NULL != (n = n->next))
+                       bufcat_id(h, " ");
        }
 
-       PAIR_ID_INIT(&tag, buf);
-       print_otag(h, TAG_H2, 1, &tag);
+       if (NULL == n) {
+               PAIR_ID_INIT(&tag, h->buf);
+               print_otag(h, TAG_H2, 1, &tag);
+       } else
+               print_otag(h, TAG_H2, 0, NULL);
+
        return(1);
 }
 
@@ -688,20 +679,20 @@ mdoc_nm_pre(MDOC_ARGS)
 {
        struct htmlpair  tag;
        struct roffsu    su;
-       size_t           len;
+       int              len;
 
        switch (n->type) {
        case (MDOC_ELEM):
                synopsis_pre(h, n);
                PAIR_CLASS_INIT(&tag, "name");
                print_otag(h, TAG_B, 1, &tag);
-               if (NULL == n->child && m->name)
-                       print_text(h, m->name);
+               if (NULL == n->child && meta->name)
+                       print_text(h, meta->name);
                return(1);
        case (MDOC_HEAD):
                print_otag(h, TAG_TD, 0, NULL);
-               if (NULL == n->child && m->name)
-                       print_text(h, m->name);
+               if (NULL == n->child && meta->name)
+                       print_text(h, meta->name);
                return(1);
        case (MDOC_BODY):
                print_otag(h, TAG_TD, 0, NULL);
@@ -716,12 +707,13 @@ mdoc_nm_pre(MDOC_ARGS)
 
        for (len = 0, n = n->child; n; n = n->next)
                if (MDOC_TEXT == n->type)
-                       len += strlen(n->string);
+                       len += html_strlen(n->string);
 
-       if (0 == len && m->name)
-               len = strlen(m->name);
+       if (0 == len && meta->name)
+               len = html_strlen(meta->name);
 
        SCALE_HS_INIT(&su, (double)len);
+       bufinit(h);
        bufcat_su(h, "width", &su);
        PAIR_STYLE_INIT(&tag, h);
        print_otag(h, TAG_COL, 1, &tag);
@@ -736,8 +728,7 @@ mdoc_nm_pre(MDOC_ARGS)
 static int
 mdoc_xr_pre(MDOC_ARGS)
 {
-       struct htmlpair          tag[2];
-       const struct mdoc_node  *nn;
+       struct htmlpair  tag[2];
 
        if (NULL == n->child)
                return(0);
@@ -753,16 +744,16 @@ mdoc_xr_pre(MDOC_ARGS)
        } else
                print_otag(h, TAG_A, 1, tag);
 
-       nn = n->child;
-       print_text(h, nn->string);
+       n = n->child;
+       print_text(h, n->string);
 
-       if (NULL == (nn = nn->next))
+       if (NULL == (n = n->next))
                return(0);
 
        h->flags |= HTML_NOSPACE;
        print_text(h, "(");
        h->flags |= HTML_NOSPACE;
-       print_text(h, nn->string);
+       print_text(h, n->string);
        h->flags |= HTML_NOSPACE;
        print_text(h, ")");
        return(0);
@@ -774,7 +765,8 @@ static int
 mdoc_ns_pre(MDOC_ARGS)
 {
 
-       h->flags |= HTML_NOSPACE;
+       if ( ! (MDOC_LINE & n->flags))
+               h->flags |= HTML_NOSPACE;
        return(1);
 }
 
@@ -797,6 +789,7 @@ mdoc_xx_pre(MDOC_ARGS)
 {
        const char      *pp;
        struct htmlpair  tag;
+       int              flags;
 
        switch (n->tok) {
        case (MDOC_Bsx):
@@ -823,8 +816,15 @@ mdoc_xx_pre(MDOC_ARGS)
 
        PAIR_CLASS_INIT(&tag, "unix");
        print_otag(h, TAG_SPAN, 1, &tag);
+
        print_text(h, pp);
-       return(1);
+       if (n->child) {
+               flags = h->flags;
+               h->flags |= HTML_KEEP;
+               print_text(h, n->child->string);
+               h->flags = flags;
+       }
+       return(0);
 }
 
 
@@ -832,19 +832,27 @@ mdoc_xx_pre(MDOC_ARGS)
 static int
 mdoc_bx_pre(MDOC_ARGS)
 {
-       const struct mdoc_node  *nn;
-       struct htmlpair          tag;
+       struct htmlpair  tag;
 
        PAIR_CLASS_INIT(&tag, "unix");
        print_otag(h, TAG_SPAN, 1, &tag);
 
-       for (nn = n->child; nn; nn = nn->next)
-               print_mdoc_node(m, nn, h);
+       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 (n->child)
+       if (NULL != (n = n->next)) {
                h->flags |= HTML_NOSPACE;
+               print_text(h, "-");
+               h->flags |= HTML_NOSPACE;
+               print_text(h, n->string);
+       }
 
-       print_text(h, "BSD");
        return(0);
 }
 
@@ -868,6 +876,8 @@ mdoc_it_pre(MDOC_ARGS)
        assert(lists[type]);
        PAIR_CLASS_INIT(&tag[0], lists[type]);
 
+       bufinit(h);
+
        if (MDOC_HEAD == n->type) {
                switch (type) {
                case(LIST_bullet):
@@ -986,8 +996,8 @@ mdoc_bl_pre(MDOC_ARGS)
                 */
 
                for (i = 0; i < (int)n->norm->Bl.ncols; i++) {
-                       a2width(n->norm->Bl.cols[i], &su);
                        bufinit(h);
+                       a2width(n->norm->Bl.cols[i], &su);
                        if (i < (int)n->norm->Bl.ncols - 1)
                                bufcat_su(h, "width", &su);
                        else
@@ -1000,6 +1010,7 @@ mdoc_bl_pre(MDOC_ARGS)
        }
 
        SCALE_VS_INIT(&su, 0);
+       bufinit(h);
        bufcat_su(h, "margin-top", &su);
        bufcat_su(h, "margin-bottom", &su);
        PAIR_STYLE_INIT(&tag[0], h);
@@ -1055,9 +1066,9 @@ mdoc_bl_pre(MDOC_ARGS)
 static int
 mdoc_ex_pre(MDOC_ARGS)
 {
-       const struct mdoc_node  *nn;
-       struct tag              *t;
-       struct htmlpair          tag;
+       struct tag      *t;
+       struct htmlpair  tag;
+       int              nchild;
 
        if (n->prev)
                print_otag(h, TAG_BR, 0, NULL);
@@ -1065,22 +1076,25 @@ mdoc_ex_pre(MDOC_ARGS)
        PAIR_CLASS_INIT(&tag, "utility");
 
        print_text(h, "The");
-       for (nn = n->child; nn; nn = nn->next) {
+
+       nchild = n->nchild;
+       for (n = n->child; n; n = n->next) {
+               assert(MDOC_TEXT == n->type);
+
                t = print_otag(h, TAG_B, 1, &tag);
-               print_text(h, nn->string);
+               print_text(h, n->string);
                print_tagq(h, t);
 
-               h->flags |= HTML_NOSPACE;
-
-               if (nn->next && NULL == nn->next->next)
-                       print_text(h, ", and");
-               else if (nn->next)
+               if (nchild > 2 && n->next) {
+                       h->flags |= HTML_NOSPACE;
                        print_text(h, ",");
-               else
-                       h->flags &= ~HTML_NOSPACE;
+               }
+
+               if (n->next && NULL == n->next->next)
+                       print_text(h, "and");
        }
 
-       if (n->child && n->child->next)
+       if (nchild > 1)
                print_text(h, "utilities exit");
        else
                print_text(h, "utility exits");
@@ -1113,6 +1127,7 @@ mdoc_d1_pre(MDOC_ARGS)
                return(1);
 
        SCALE_VS_INIT(&su, 0);
+       bufinit(h);
        bufcat_su(h, "margin-top", &su);
        bufcat_su(h, "margin-bottom", &su);
        PAIR_STYLE_INIT(&tag[0], h);
@@ -1136,19 +1151,19 @@ mdoc_d1_pre(MDOC_ARGS)
 static int
 mdoc_sx_pre(MDOC_ARGS)
 {
-       struct htmlpair          tag[2];
-       const struct mdoc_node  *nn;
-       char                     buf[BUFSIZ];
+       struct htmlpair  tag[2];
 
-       strlcpy(buf, "#", BUFSIZ);
-       for (nn = n->child; nn; nn = nn->next) {
-               html_idcat(buf, nn->string, BUFSIZ);
-               if (nn->next)
-                       html_idcat(buf, " ", BUFSIZ);
+       bufinit(h);
+       bufcat(h, "#x");
+
+       for (n = n->child; n; ) {
+               bufcat_id(h, n->string);
+               if (NULL != (n = n->next))
+                       bufcat_id(h, " ");
        }
 
        PAIR_CLASS_INIT(&tag[0], "link-sec");
-       PAIR_HREF_INIT(&tag[1], buf);
+       PAIR_HREF_INIT(&tag[1], h->buf);
 
        print_otag(h, TAG_I, 1, tag);
        print_otag(h, TAG_A, 2, tag);
@@ -1161,7 +1176,7 @@ static int
 mdoc_bd_pre(MDOC_ARGS)
 {
        struct htmlpair          tag[2];
-       int                      comp;
+       int                      comp, sv;
        const struct mdoc_node  *nn;
        struct roffsu            su;
 
@@ -1186,7 +1201,8 @@ mdoc_bd_pre(MDOC_ARGS)
        SCALE_HS_INIT(&su, 0);
        if (n->norm->Bd.offs)
                a2offs(n->norm->Bd.offs, &su);
-
+       
+       bufinit(h);
        bufcat_su(h, "margin-left", &su);
        PAIR_STYLE_INIT(&tag[0], h);
 
@@ -1200,8 +1216,13 @@ mdoc_bd_pre(MDOC_ARGS)
        PAIR_CLASS_INIT(&tag[1], "lit display");
        print_otag(h, TAG_PRE, 2, tag);
 
+       /* This can be recursive: save & set our literal state. */
+
+       sv = h->flags & HTML_LITERAL;
+       h->flags |= HTML_LITERAL;
+
        for (nn = n->child; nn; nn = nn->next) {
-               print_mdoc_node(m, nn, h);
+               print_mdoc_node(meta, nn, h);
                /*
                 * If the printed node flushes its own line, then we
                 * needn't do it here as well.  This is hacky, but the
@@ -1236,6 +1257,9 @@ mdoc_bd_pre(MDOC_ARGS)
                h->flags |= HTML_NOSPACE;
        }
 
+       if (0 == sv)
+               h->flags &= ~HTML_LITERAL;
+
        return(0);
 }
 
@@ -1345,12 +1369,16 @@ mdoc_fa_pre(MDOC_ARGS)
                t = print_otag(h, TAG_I, 1, &tag);
                print_text(h, nn->string);
                print_tagq(h, t);
-               if (nn->next)
+               if (nn->next) {
+                       h->flags |= HTML_NOSPACE;
                        print_text(h, ",");
+               }
        }
 
-       if (n->child && n->next && n->next->tok == MDOC_Fa)
+       if (n->child && n->next && n->next->tok == MDOC_Fa) {
+               h->flags |= HTML_NOSPACE;
                print_text(h, ",");
+       }
 
        return(0);
 }
@@ -1360,13 +1388,60 @@ mdoc_fa_pre(MDOC_ARGS)
 static int
 mdoc_fd_pre(MDOC_ARGS)
 {
-       struct htmlpair  tag;
+       struct htmlpair  tag[2];
+       char             buf[BUFSIZ];
+       size_t           sz;
+       int              i;
+       struct tag      *t;
 
        synopsis_pre(h, n);
 
-       PAIR_CLASS_INIT(&tag, "macro");
-       print_otag(h, TAG_B, 1, &tag);
-       return(1);
+       if (NULL == (n = n->child))
+               return(0);
+
+       assert(MDOC_TEXT == n->type);
+
+       if (strcmp(n->string, "#include")) {
+               PAIR_CLASS_INIT(&tag[0], "macro");
+               print_otag(h, TAG_B, 1, tag);
+               return(1);
+       }
+
+       PAIR_CLASS_INIT(&tag[0], "includes");
+       print_otag(h, TAG_B, 1, tag);
+       print_text(h, n->string);
+
+       if (NULL != (n = n->next)) {
+               assert(MDOC_TEXT == n->type);
+               strlcpy(buf, '<' == *n->string || '"' == *n->string ? 
+                               n->string + 1 : n->string, BUFSIZ);
+
+               sz = strlen(buf);
+               if (sz && ('>' == buf[sz - 1] || '"' == buf[sz - 1]))
+                       buf[sz - 1] = '\0';
+
+               PAIR_CLASS_INIT(&tag[0], "link-includes");
+               
+               i = 1;
+               if (h->base_includes) {
+                       buffmt_includes(h, buf);
+                       PAIR_HREF_INIT(&tag[i], h->buf);
+                       i++;
+               } 
+
+               t = print_otag(h, TAG_A, i, tag);
+               print_text(h, n->string);
+               print_tagq(h, t);
+
+               n = n->next;
+       }
+
+       for ( ; n; n = n->next) {
+               assert(MDOC_TEXT == n->type);
+               print_text(h, n->string);
+       }
+
+       return(0);
 }
 
 
@@ -1407,13 +1482,13 @@ mdoc_ft_pre(MDOC_ARGS)
 static int
 mdoc_fn_pre(MDOC_ARGS)
 {
-       struct tag              *t;
-       struct htmlpair          tag[2];
-       const struct mdoc_node  *nn;
-       char                     nbuf[BUFSIZ];
-       const char              *sp, *ep;
-       int                      sz, i;
+       struct tag      *t;
+       struct htmlpair  tag[2];
+       char             nbuf[BUFSIZ];
+       const char      *sp, *ep;
+       int              sz, i, pretty;
 
+       pretty = MDOC_SYNPRETTY & n->flags;
        synopsis_pre(h, n);
 
        /* Split apart into type and name. */
@@ -1465,26 +1540,33 @@ mdoc_fn_pre(MDOC_ARGS)
 
        h->flags |= HTML_NOSPACE;
        print_text(h, "(");
+       h->flags |= HTML_NOSPACE;
 
-       bufinit(h);
        PAIR_CLASS_INIT(&tag[0], "farg");
+       bufinit(h);
        bufcat_style(h, "white-space", "nowrap");
        PAIR_STYLE_INIT(&tag[1], h);
 
-       for (nn = n->child->next; nn; nn = nn->next) {
+       for (n = n->child->next; n; n = n->next) {
                i = 1;
                if (MDOC_SYNPRETTY & n->flags)
                        i = 2;
                t = print_otag(h, TAG_I, i, tag);
-               print_text(h, nn->string);
+               print_text(h, n->string);
                print_tagq(h, t);
-               if (nn->next)
+               if (n->next) {
+                       h->flags |= HTML_NOSPACE;
                        print_text(h, ",");
+               }
        }
 
+       h->flags |= HTML_NOSPACE;
        print_text(h, ")");
-       if (MDOC_SYNPRETTY & n->flags)
+
+       if (pretty) {
+               h->flags |= HTML_NOSPACE;
                print_text(h, ";");
+       }
 
        return(0);
 }
@@ -1535,11 +1617,13 @@ mdoc_sp_pre(MDOC_ARGS)
        SCALE_VS_INIT(&su, 1);
 
        if (MDOC_sp == n->tok) {
-               if (n->child)
-                       a2roffsu(n->child->string, &su, SCALE_VS);
+               if (NULL != (n = n->child))
+                       if ( ! a2roffsu(n->string, &su, SCALE_VS))
+                               SCALE_VS_INIT(&su, atoi(n->string));
        } else
                su.scale = 0;
 
+       bufinit(h);
        bufcat_su(h, "height", &su);
        PAIR_STYLE_INIT(&tag, h);
        print_otag(h, TAG_DIV, 1, &tag);
@@ -1555,20 +1639,23 @@ mdoc_sp_pre(MDOC_ARGS)
 static int
 mdoc_lk_pre(MDOC_ARGS)
 {
-       const struct mdoc_node  *nn;
-       struct htmlpair          tag[2];
+       struct htmlpair  tag[2];
 
-       nn = n->child;
+       if (NULL == (n = n->child))
+               return(0);
+
+       assert(MDOC_TEXT == n->type);
 
        PAIR_CLASS_INIT(&tag[0], "link-ext");
-       PAIR_HREF_INIT(&tag[1], nn->string);
+       PAIR_HREF_INIT(&tag[1], n->string);
+
        print_otag(h, TAG_A, 2, tag);
 
-       if (NULL == nn || NULL == nn->next) 
-               return(1);
+       if (NULL == n->next)
+               print_text(h, n->string);
 
-       for (nn = nn->next; nn; nn = nn->next) 
-               print_text(h, nn->string);
+       for (n = n->next; n; n = n->next)
+               print_text(h, n->string);
 
        return(0);
 }
@@ -1578,19 +1665,21 @@ mdoc_lk_pre(MDOC_ARGS)
 static int
 mdoc_mt_pre(MDOC_ARGS)
 {
-       struct htmlpair          tag[2];
-       struct tag              *t;
-       const struct mdoc_node  *nn;
+       struct htmlpair  tag[2];
+       struct tag      *t;
 
        PAIR_CLASS_INIT(&tag[0], "link-mail");
 
-       for (nn = n->child; nn; nn = nn->next) {
+       for (n = n->child; n; n = n->next) {
+               assert(MDOC_TEXT == n->type);
+
                bufinit(h);
                bufcat(h, "mailto:");
-               bufcat(h, nn->string);
+               bufcat(h, n->string);
+
                PAIR_HREF_INIT(&tag[1], h->buf);
                t = print_otag(h, TAG_A, 2, tag);
-               print_text(h, nn->string);
+               print_text(h, n->string);
                print_tagq(h, t);
        }
        
@@ -1635,7 +1724,9 @@ mdoc_fo_post(MDOC_ARGS)
 
        if (MDOC_BODY != n->type)
                return;
+       h->flags |= HTML_NOSPACE;
        print_text(h, ")");
+       h->flags |= HTML_NOSPACE;
        print_text(h, ";");
 }
 
@@ -1644,39 +1735,55 @@ mdoc_fo_post(MDOC_ARGS)
 static int
 mdoc_in_pre(MDOC_ARGS)
 {
-       const struct mdoc_node  *nn;
-       struct tag              *t;
-       struct htmlpair          tag[2];
-       int                      i;
+       struct tag      *t;
+       struct htmlpair  tag[2];
+       int              i;
 
        synopsis_pre(h, n);
 
        PAIR_CLASS_INIT(&tag[0], "includes");
        print_otag(h, TAG_B, 1, tag);
 
+       /*
+        * The first argument of the `In' gets special treatment as
+        * being a linked value.  Subsequent values are printed
+        * afterward.  groff does similarly.  This also handles the case
+        * of no children.
+        */
+
        if (MDOC_SYNPRETTY & n->flags && MDOC_LINE & n->flags)
                print_text(h, "#include");
 
        print_text(h, "<");
        h->flags |= HTML_NOSPACE;
 
-       for (nn = n->child; nn; nn = nn->next) {
+       if (NULL != (n = n->child)) {
+               assert(MDOC_TEXT == n->type);
+
                PAIR_CLASS_INIT(&tag[0], "link-includes");
+
                i = 1;
-               bufinit(h);
                if (h->base_includes) {
-                       buffmt_includes(h, nn->string);
+                       buffmt_includes(h, n->string);
                        PAIR_HREF_INIT(&tag[i], h->buf);
                        i++;
-               }
+               } 
+
                t = print_otag(h, TAG_A, i, tag);
-               print_mdoc_node(m, nn, h);
+               print_text(h, n->string);
                print_tagq(h, t);
+
+               n = n->next;
        }
 
        h->flags |= HTML_NOSPACE;
        print_text(h, ">");
 
+       for ( ; n; n = n->next) {
+               assert(MDOC_TEXT == n->type);
+               print_text(h, n->string);
+       }
+
        return(0);
 }
 
@@ -1697,31 +1804,38 @@ mdoc_ic_pre(MDOC_ARGS)
 static int
 mdoc_rv_pre(MDOC_ARGS)
 {
-       const struct mdoc_node  *nn;
-       struct htmlpair          tag;
-       struct tag              *t;
+       struct htmlpair  tag;
+       struct tag      *t;
+       int              nchild;
 
        if (n->prev)
                print_otag(h, TAG_BR, 0, NULL);
 
+       PAIR_CLASS_INIT(&tag, "fname");
+
        print_text(h, "The");
 
-       for (nn = n->child; nn; nn = nn->next) {
-               PAIR_CLASS_INIT(&tag, "fname");
+       nchild = n->nchild;
+       for (n = n->child; n; n = n->next) {
+               assert(MDOC_TEXT == n->type);
+
                t = print_otag(h, TAG_B, 1, &tag);
-               print_text(h, nn->string);
+               print_text(h, n->string);
                print_tagq(h, t);
 
                h->flags |= HTML_NOSPACE;
-               if (nn->next && NULL == nn->next->next)
-                       print_text(h, "(), and");
-               else if (nn->next)
-                       print_text(h, "(),");
-               else
-                       print_text(h, "()");
+               print_text(h, "()");
+
+               if (nchild > 2 && n->next) {
+                       h->flags |= HTML_NOSPACE;
+                       print_text(h, ",");
+               }
+
+               if (n->next && NULL == n->next->next)
+                       print_text(h, "and");
        }
 
-       if (n->child && n->child->next)
+       if (nchild > 1)
                print_text(h, "functions return");
        else
                print_text(h, "function returns");
@@ -1787,6 +1901,7 @@ mdoc_bf_pre(MDOC_ARGS)
         * We want this to be inline-formatted, but needs to be div to
         * accept block children. 
         */
+       bufinit(h);
        bufcat_style(h, "display", "inline");
        SCALE_HS_INIT(&su, 1);
        /* Needs a left-margin for spacing. */
@@ -1854,7 +1969,7 @@ mdoc_li_pre(MDOC_ARGS)
        struct htmlpair tag;
 
        PAIR_CLASS_INIT(&tag, "lit");
-       print_otag(h, TAG_SPAN, 1, &tag);
+       print_otag(h, TAG_CODE, 1, &tag);
        return(1);
 }
 
@@ -1996,6 +2111,7 @@ mdoc__x_post(MDOC_ARGS)
        if (NULL == n->parent || MDOC_Rs != n->parent->tok)
                return;
 
+       h->flags |= HTML_NOSPACE;
        print_text(h, n->next ? "," : ".");
 }
 
@@ -2066,6 +2182,8 @@ mdoc_quote_pre(MDOC_ARGS)
                PAIR_CLASS_INIT(&tag, "opt");
                print_otag(h, TAG_SPAN, 1, &tag);
                break;
+       case (MDOC_Eo):
+               break;
        case (MDOC_Do):
                /* FALLTHROUGH */
        case (MDOC_Dq):
@@ -2081,7 +2199,11 @@ mdoc_quote_pre(MDOC_ARGS)
                print_text(h, "(");
                break;
        case (MDOC_Ql):
-               /* FALLTHROUGH */
+               print_text(h, "\\(oq");
+               h->flags |= HTML_NOSPACE;
+               PAIR_CLASS_INIT(&tag, "lit");
+               print_otag(h, TAG_CODE, 1, &tag);
+               break;
        case (MDOC_So):
                /* FALLTHROUGH */
        case (MDOC_Sq):
@@ -2127,6 +2249,8 @@ mdoc_quote_post(MDOC_ARGS)
        case (MDOC_Bq):
                print_text(h, "\\(rB");
                break;
+       case (MDOC_Eo):
+               break;
        case (MDOC_Qo):
                /* FALLTHROUGH */
        case (MDOC_Qq):
@@ -2146,7 +2270,7 @@ mdoc_quote_post(MDOC_ARGS)
        case (MDOC_So):
                /* FALLTHROUGH */
        case (MDOC_Sq):
-               print_text(h, "\\(aq");
+               print_text(h, "\\(cq");
                break;
        default:
                abort();