]> git.cameronkatri.com Git - mandoc.git/blobdiff - mdoc_man.c
1) Remove documentation of the groff-1.15 compatibility quirk
[mandoc.git] / mdoc_man.c
index b1044210b9cd2495b0c16a12bb39b1e2b99134c3..7def0df88ca008d8160ebeecda670293985fb381 100644 (file)
@@ -1,4 +1,4 @@
-/*     $Id: mdoc_man.c,v 1.26 2012/07/09 17:53:01 schwarze Exp $ */
+/*     $Id: mdoc_man.c,v 1.41 2012/07/16 10:45:41 schwarze Exp $ */
 /*
  * Copyright (c) 2011, 2012 Ingo Schwarze <schwarze@openbsd.org>
  *
@@ -43,16 +43,22 @@ static      int       cond_body(DECL_ARGS);
 static int       cond_head(DECL_ARGS);
 static  void     font_push(char);
 static void      font_pop(void);
+static void      post__t(DECL_ARGS);
 static void      post_bd(DECL_ARGS);
+static void      post_bf(DECL_ARGS);
 static void      post_bk(DECL_ARGS);
+static void      post_bl(DECL_ARGS);
 static void      post_dl(DECL_ARGS);
 static void      post_enc(DECL_ARGS);
+static void      post_eo(DECL_ARGS);
 static void      post_fa(DECL_ARGS);
+static void      post_fd(DECL_ARGS);
 static void      post_fl(DECL_ARGS);
 static void      post_fn(DECL_ARGS);
 static void      post_fo(DECL_ARGS);
 static void      post_font(DECL_ARGS);
 static void      post_in(DECL_ARGS);
+static void      post_it(DECL_ARGS);
 static void      post_lb(DECL_ARGS);
 static void      post_nm(DECL_ARGS);
 static void      post_percent(DECL_ARGS);
@@ -60,16 +66,20 @@ static      void      post_pf(DECL_ARGS);
 static void      post_sect(DECL_ARGS);
 static void      post_sp(DECL_ARGS);
 static void      post_vt(DECL_ARGS);
+static int       pre__t(DECL_ARGS);
 static int       pre_an(DECL_ARGS);
 static int       pre_ap(DECL_ARGS);
 static int       pre_bd(DECL_ARGS);
+static int       pre_bf(DECL_ARGS);
 static int       pre_bk(DECL_ARGS);
+static int       pre_bl(DECL_ARGS);
 static int       pre_br(DECL_ARGS);
 static int       pre_bx(DECL_ARGS);
 static int       pre_dl(DECL_ARGS);
 static int       pre_enc(DECL_ARGS);
 static int       pre_em(DECL_ARGS);
 static int       pre_fa(DECL_ARGS);
+static int       pre_fd(DECL_ARGS);
 static int       pre_fl(DECL_ARGS);
 static int       pre_fn(DECL_ARGS);
 static int       pre_fo(DECL_ARGS);
@@ -82,6 +92,7 @@ static        int       pre_nm(DECL_ARGS);
 static int       pre_no(DECL_ARGS);
 static int       pre_ns(DECL_ARGS);
 static int       pre_pp(DECL_ARGS);
+static int       pre_rs(DECL_ARGS);
 static int       pre_sm(DECL_ARGS);
 static int       pre_sp(DECL_ARGS);
 static int       pre_sect(DECL_ARGS);
@@ -91,7 +102,12 @@ static      int       pre_vt(DECL_ARGS);
 static int       pre_ux(DECL_ARGS);
 static int       pre_xr(DECL_ARGS);
 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 *);
+static void      print_width(const char *,
+                               const struct mdoc_node *, size_t);
+static void      print_count(int *);
 static void      print_node(DECL_ARGS);
 
 static const struct manact manacts[MDOC_MAX + 1] = {
@@ -106,9 +122,9 @@ static      const struct manact manacts[MDOC_MAX + 1] = {
        { cond_body, pre_dl, post_dl, NULL, NULL }, /* Dl */
        { cond_body, pre_bd, post_bd, NULL, NULL }, /* Bd */
        { NULL, NULL, NULL, NULL, NULL }, /* Ed */
-       { NULL, NULL, NULL, NULL, NULL }, /* Bl */
+       { cond_body, pre_bl, post_bl, NULL, NULL }, /* Bl */
        { NULL, NULL, NULL, NULL, NULL }, /* El */
-       { NULL, pre_it, NULL, NULL, NULL }, /* _It */
+       { NULL, pre_it, post_it, NULL, NULL }, /* It */
        { NULL, pre_em, post_font, NULL, NULL }, /* Ad */
        { NULL, pre_an, NULL, NULL, NULL }, /* An */
        { NULL, pre_em, post_font, NULL, NULL }, /* Ar */
@@ -121,7 +137,7 @@ static      const struct manact manacts[MDOC_MAX + 1] = {
            "\\fP\nutility exits 0 on success, and >0 if an error occurs."
            }, /* Ex */
        { NULL, pre_fa, post_fa, NULL, NULL }, /* Fa */
-       { NULL, NULL, NULL, NULL, NULL }, /* _Fd */
+       { NULL, pre_fd, post_fd, NULL, NULL }, /* Fd */
        { NULL, pre_fl, post_fl, NULL, NULL }, /* Fl */
        { NULL, pre_fn, post_fn, NULL, NULL }, /* Fn */
        { NULL, pre_ft, post_font, NULL, NULL }, /* Ft */
@@ -142,23 +158,23 @@ static    const struct manact manacts[MDOC_MAX + 1] = {
        { NULL, pre_em, post_font, NULL, NULL }, /* Va */
        { NULL, pre_vt, post_vt, NULL, NULL }, /* Vt */
        { NULL, pre_xr, NULL, NULL, NULL }, /* Xr */
-       { NULL, NULL, post_percent, NULL, NULL }, /* _%A */
-       { NULL, NULL, NULL, NULL, NULL }, /* _%B */
-       { NULL, NULL, post_percent, NULL, NULL }, /* _%D */
-       { NULL, NULL, NULL, NULL, NULL }, /* _%I */
-       { NULL, pre_enc, post_percent, "\\fI", "\\fP" }, /* %J */
-       { NULL, NULL, NULL, NULL, NULL }, /* _%N */
-       { NULL, NULL, NULL, NULL, NULL }, /* _%O */
-       { NULL, NULL, NULL, NULL, NULL }, /* _%P */
-       { NULL, NULL, NULL, NULL, NULL }, /* _%R */
-       { NULL, pre_enc, post_percent, "\"", "\"" }, /* %T */
-       { NULL, NULL, NULL, NULL, NULL }, /* _%V */
+       { NULL, NULL, post_percent, NULL, NULL }, /* %A */
+       { NULL, pre_em, post_percent, NULL, NULL }, /* %B */
+       { NULL, NULL, post_percent, NULL, NULL }, /* %D */
+       { NULL, pre_em, post_percent, NULL, NULL }, /* %I */
+       { NULL, pre_em, post_percent, NULL, NULL }, /* %J */
+       { NULL, NULL, post_percent, NULL, NULL }, /* %N */
+       { NULL, NULL, post_percent, NULL, NULL }, /* %O */
+       { NULL, NULL, post_percent, NULL, NULL }, /* %P */
+       { NULL, NULL, post_percent, NULL, NULL }, /* %R */
+       { NULL, pre__t, post__t, NULL, NULL }, /* %T */
+       { NULL, NULL, post_percent, NULL, NULL }, /* %V */
        { NULL, NULL, NULL, NULL, NULL }, /* Ac */
        { cond_body, pre_enc, post_enc, "<", ">" }, /* Ao */
        { cond_body, pre_enc, post_enc, "<", ">" }, /* Aq */
        { NULL, NULL, NULL, NULL, NULL }, /* At */
        { NULL, NULL, NULL, NULL, NULL }, /* Bc */
-       { NULL, NULL, NULL, NULL, NULL }, /* _Bf */
+       { NULL, pre_bf, post_bf, NULL, NULL }, /* Bf */
        { cond_body, pre_enc, post_enc, "[", "]" }, /* Bo */
        { cond_body, pre_enc, post_enc, "[", "]" }, /* Bq */
        { NULL, pre_ux, NULL, "BSD/OS", NULL }, /* Bsx */
@@ -167,10 +183,10 @@ static    const struct manact manacts[MDOC_MAX + 1] = {
        { NULL, NULL, NULL, NULL, NULL }, /* Dc */
        { cond_body, pre_enc, post_enc, "``", "''" }, /* Do */
        { cond_body, pre_enc, post_enc, "``", "''" }, /* Dq */
-       { NULL, NULL, NULL, NULL, NULL }, /* _Ec */
-       { NULL, NULL, NULL, NULL, NULL }, /* _Ef */
+       { NULL, NULL, NULL, NULL, NULL }, /* Ec */
+       { NULL, NULL, NULL, NULL, NULL }, /* Ef */
        { NULL, pre_em, post_font, NULL, NULL }, /* Em */
-       { NULL, NULL, NULL, NULL, NULL }, /* _Eo */
+       { NULL, NULL, post_eo, NULL, NULL }, /* Eo */
        { NULL, pre_ux, NULL, "FreeBSD", NULL }, /* Fx */
        { NULL, pre_sy, post_font, NULL, NULL }, /* Ms */
        { NULL, pre_no, NULL, NULL, NULL }, /* No */
@@ -186,7 +202,7 @@ static      const struct manact manacts[MDOC_MAX + 1] = {
        { cond_body, pre_enc, post_enc, "\"", "\"" }, /* Qo */
        { cond_body, pre_enc, post_enc, "\"", "\"" }, /* Qq */
        { NULL, NULL, NULL, NULL, NULL }, /* Re */
-       { cond_body, pre_pp, NULL, NULL, NULL }, /* Rs */
+       { cond_body, pre_rs, NULL, NULL, NULL }, /* Rs */
        { NULL, NULL, NULL, NULL, NULL }, /* Sc */
        { cond_body, pre_enc, post_enc, "`", "'" }, /* So */
        { cond_body, pre_enc, post_enc, "`", "'" }, /* Sq */
@@ -195,8 +211,8 @@ static      const struct manact manacts[MDOC_MAX + 1] = {
        { NULL, pre_sy, post_font, NULL, NULL }, /* Sy */
        { NULL, pre_li, post_font, NULL, NULL }, /* Tn */
        { NULL, pre_ux, NULL, "UNIX", NULL }, /* Ux */
-       { NULL, NULL, NULL, NULL, NULL }, /* _Xc */
-       { NULL, NULL, NULL, NULL, NULL }, /* _Xo */
+       { NULL, NULL, NULL, NULL, NULL }, /* Xc */
+       { NULL, NULL, NULL, NULL, NULL }, /* Xo */
        { NULL, pre_fo, post_fo, NULL, NULL }, /* Fo */
        { NULL, NULL, NULL, NULL, NULL }, /* Fc */
        { cond_body, pre_enc, post_enc, "[", "]" }, /* Oo */
@@ -214,28 +230,31 @@ static    const struct manact manacts[MDOC_MAX + 1] = {
        { cond_body, pre_enc, post_enc, "{", "}" }, /* Brq */
        { cond_body, pre_enc, post_enc, "{", "}" }, /* Bro */
        { NULL, NULL, NULL, NULL, NULL }, /* Brc */
-       { NULL, NULL, NULL, NULL, NULL }, /* _%C */
-       { NULL, NULL, NULL, NULL, NULL }, /* _Es */
-       { NULL, NULL, NULL, NULL, NULL }, /* _En */
+       { NULL, NULL, post_percent, NULL, NULL }, /* %C */
+       { NULL, NULL, NULL, NULL, NULL }, /* Es */
+       { NULL, NULL, NULL, NULL, NULL }, /* En */
        { NULL, pre_ux, NULL, "DragonFly", NULL }, /* Dx */
-       { NULL, NULL, NULL, NULL, NULL }, /* _%Q */
+       { NULL, NULL, post_percent, NULL, NULL }, /* %Q */
        { NULL, pre_br, NULL, NULL, NULL }, /* br */
        { NULL, pre_sp, post_sp, NULL, NULL }, /* sp */
-       { NULL, NULL, NULL, NULL, NULL }, /* _%U */
-       { NULL, NULL, NULL, NULL, NULL }, /* _Ta */
+       { NULL, NULL, post_percent, NULL, NULL }, /* %U */
+       { NULL, NULL, NULL, NULL, NULL }, /* Ta */
        { NULL, NULL, NULL, NULL, NULL }, /* ROOT */
 };
 
 static int             outflags;
-#define        MMAN_spc        (1 << 0)
-#define        MMAN_spc_force  (1 << 1)
-#define        MMAN_nl         (1 << 2)
-#define        MMAN_br         (1 << 3)
-#define        MMAN_sp         (1 << 4)
-#define        MMAN_Sm         (1 << 5)
-#define        MMAN_Bk         (1 << 6)
-#define        MMAN_An_split   (1 << 7)
-#define        MMAN_An_nosplit (1 << 8)
+#define        MMAN_spc        (1 << 0)  /* blank character before next word */
+#define        MMAN_spc_force  (1 << 1)  /* even before trailing punctuation */
+#define        MMAN_nl         (1 << 2)  /* break man(7) code line */
+#define        MMAN_br         (1 << 3)  /* break output line */
+#define        MMAN_sp         (1 << 4)  /* insert a blank output line */
+#define        MMAN_PP         (1 << 5)  /* reset indentation etc. */
+#define        MMAN_Sm         (1 << 6)  /* horizontal spacing mode */
+#define        MMAN_Bk         (1 << 7)  /* word keep mode */
+#define        MMAN_An_split   (1 << 8)  /* author mode is "split" */
+#define        MMAN_An_nosplit (1 << 9)  /* author mode is "nosplit" */
+
+static int             TPremain;  /* characters before tag is full */
 
 static struct {
        char    *head;
@@ -253,7 +272,8 @@ font_push(char newfont)
                                fontqueue.size);
        }
        *fontqueue.tail = newfont;
-       print_word("\\f");
+       print_word("");
+       printf("\\f");
        putchar(newfont);
        outflags &= ~MMAN_spc;
 }
@@ -265,7 +285,8 @@ font_pop(void)
        if (fontqueue.tail > fontqueue.head)
                fontqueue.tail--;
        outflags &= ~MMAN_spc;
-       print_word("\\f");
+       print_word("");
+       printf("\\f");
        putchar(*fontqueue.tail);
 }
 
@@ -273,39 +294,49 @@ static void
 print_word(const char *s)
 {
 
-       if ((MMAN_sp | MMAN_br | MMAN_nl) & outflags) {
+       if ((MMAN_PP | MMAN_sp | MMAN_br | MMAN_nl) & outflags) {
                /* 
                 * If we need a newline, print it now and start afresh.
                 */
-               if (MMAN_sp & outflags)
+               if (MMAN_PP & outflags) {
+                       if ( ! (MMAN_sp & outflags))
+                               printf("\n.sp -1v");
+                       printf("\n.PP\n");
+               } else if (MMAN_sp & outflags)
                        printf("\n.sp\n");
                else if (MMAN_br & outflags)
                        printf("\n.br\n");
                else if (MMAN_nl & outflags)
                        putchar('\n');
-               outflags &= ~(MMAN_sp|MMAN_br|MMAN_nl|MMAN_spc);
-       } else if (MMAN_spc & outflags && '\0' != s[0])
+               outflags &= ~(MMAN_PP|MMAN_sp|MMAN_br|MMAN_nl|MMAN_spc);
+               if (1 == TPremain)
+                       printf(".br\n");
+               TPremain = 0;
+       } else if (MMAN_spc & outflags) {
                /*
                 * If we need a space, only print it if
                 * (1) it is forced by `No' or
                 * (2) what follows is not terminating punctuation or
                 * (3) what follows is longer than one character.
                 */
-               if (MMAN_spc_force & outflags ||
+               if (MMAN_spc_force & outflags || '\0' == s[0] ||
                    NULL == strchr(".,:;)]?!", s[0]) || '\0' != s[1]) {
                        if (MMAN_Bk & outflags) {
                                putchar('\\');
                                putchar('~');
                        } else 
                                putchar(' ');
+                       if (TPremain)
+                               TPremain--;
                }
+       }
 
        /*
         * Reassign needing space if we're not following opening
         * punctuation.
         */
-       if (MMAN_Sm & outflags &&
-           (('(' != s[0] && '[' != s[0]) || '\0' != s[1]))
+       if (MMAN_Sm & outflags && ('\0' == s[0] ||
+           (('(' != s[0] && '[' != s[0]) || '\0' != s[1])))
                outflags |= MMAN_spc;
        else
                outflags &= ~MMAN_spc;
@@ -323,9 +354,35 @@ print_word(const char *s)
                        putchar((unsigned char)*s);
                        break;
                }
+               if (TPremain)
+                       TPremain--;
        }
 }
 
+static void
+print_line(const char *s, int newflags)
+{
+
+       outflags &= ~MMAN_br;
+       outflags |= MMAN_nl;
+       print_word(s);
+       outflags |= newflags;
+}
+
+static void
+print_block(const char *s, int newflags)
+{
+
+       outflags &= ~MMAN_PP;
+       if (MMAN_sp & outflags)
+               outflags &= ~(MMAN_sp | MMAN_br);
+       else
+               print_line(".sp -1v", 0);
+       outflags |= MMAN_nl;
+       print_word(s);
+       outflags |= newflags;
+}
+
 static void
 print_offs(const char *v)
 {
@@ -349,6 +406,55 @@ print_offs(const char *v)
        print_word(buf);
 }
 
+void
+print_width(const char *v, const struct mdoc_node *child, size_t defsz)
+{
+       char              buf[24];
+       struct roffsu     su;
+       size_t            sz, chsz;
+       int               numeric, remain;
+
+       numeric = 1;
+       remain = 0;
+       if (NULL == v)
+               sz = defsz;
+       else if (a2roffsu(v, &su, SCALE_MAX)) {
+               if (SCALE_EN == su.unit)
+                       sz = su.scale;
+               else {
+                       sz = 0;
+                       numeric = 0;
+               }
+       } else
+               sz = strlen(v);
+
+       /* XXX Rough estimation, might have multiple parts. */
+       chsz = (NULL != child && MDOC_TEXT == child->type) ?
+                       strlen(child->string) : 0;
+
+       if (defsz && chsz > sz)
+               print_block(".HP", 0);
+       else {
+               print_block(".TP", 0);
+               remain = sz + 2;
+       }
+       if (numeric) {
+               snprintf(buf, sizeof(buf), "%ldn", sz + 2);
+               print_word(buf);
+       } else
+               print_word(v);
+       TPremain = remain;
+}
+
+void
+print_count(int *count)
+{
+       char              buf[12];
+
+       snprintf(buf, sizeof(buf), "%d.", ++*count);
+       print_word(buf);
+}
+
 void
 man_man(void *arg, const struct man *man)
 {
@@ -396,8 +502,7 @@ print_node(DECL_ARGS)
         * This makes the page structure be more consistent.
         */
        prev = n->prev ? n->prev : n->parent;
-       if (prev && prev->line < n->line &&
-           MDOC_Fo != prev->tok && MDOC_Ns != prev->tok)
+       if (MMAN_spc & outflags && prev && prev->line < n->line)
                outflags |= MMAN_nl;
 
        act = NULL;
@@ -411,7 +516,8 @@ print_node(DECL_ARGS)
                 */
                if (MMAN_nl & outflags && ('.' == *n->string || 
                                        '\'' == *n->string)) {
-                       print_word("\\&");
+                       print_word("");
+                       printf("\\&");
                        outflags &= ~MMAN_spc;
                }
                print_word(n->string);
@@ -488,24 +594,51 @@ post_font(DECL_ARGS)
        font_pop();
 }
 
-/*
- * Used in listings (percent = %A, e.g.).
- * FIXME: this is incomplete. 
- * It doesn't print a nice ", and" for lists.
- */
 static void
 post_percent(DECL_ARGS)
 {
 
-       post_enc(m, n);
-       if (n->next)
+       if (pre_em == manacts[n->tok].pre)
+               font_pop();
+       if (n->next) {
                print_word(",");
-       else {
+               if (n->prev &&  n->prev->tok == n->tok &&
+                               n->next->tok == n->tok)
+                       print_word("and");
+       } else {
                print_word(".");
                outflags |= MMAN_nl;
        }
 }
 
+static int
+pre__t(DECL_ARGS)
+{
+
+        if (n->parent && MDOC_Rs == n->parent->tok &&
+                        n->parent->norm->Rs.quote_T) {
+               print_word("");
+               putchar('\"');
+               outflags &= ~MMAN_spc;
+       } else
+               font_push('I');
+       return(1);
+}
+
+static void
+post__t(DECL_ARGS)
+{
+
+        if (n->parent && MDOC_Rs == n->parent->tok &&
+                        n->parent->norm->Rs.quote_T) {
+               outflags &= ~MMAN_spc;
+               print_word("");
+               putchar('\"');
+       } else
+               font_pop();
+       post_percent(m, n);
+}
+
 /*
  * Print before a section header.
  */
@@ -515,9 +648,10 @@ pre_sect(DECL_ARGS)
 
        if (MDOC_HEAD != n->type)
                return(1);
-       outflags |= MMAN_nl;
-       print_word(manacts[n->tok].prefix);
-       print_word("\"");
+       outflags |= MMAN_sp;
+       print_block(manacts[n->tok].prefix, 0);
+       print_word("");
+       putchar('\"');
        outflags &= ~MMAN_spc;
        return(1);
 }
@@ -532,7 +666,8 @@ post_sect(DECL_ARGS)
        if (MDOC_HEAD != n->type)
                return;
        outflags &= ~MMAN_spc;
-       print_word("\"");
+       print_word("");
+       putchar('\"');
        outflags |= MMAN_nl;
        if (MDOC_Sh == n->tok && SEC_AUTHORS == n->sec)
                outflags &= ~(MMAN_An_split | MMAN_An_nosplit);
@@ -615,15 +750,14 @@ static int
 pre_bd(DECL_ARGS)
 {
 
-       if (0 == n->norm->Bd.comp)
-               outflags |= MMAN_sp;
+       outflags &= ~(MMAN_PP | MMAN_sp | MMAN_br);
+
        if (DISP_unfilled == n->norm->Bd.type ||
-           DISP_literal  == n->norm->Bd.type) {
-               outflags |= MMAN_nl;
-               print_word(".nf");
-       }
-       outflags |= MMAN_nl;
-       print_word(".RS");
+           DISP_literal  == n->norm->Bd.type)
+               print_line(".nf", 0);
+       if (0 == n->norm->Bd.comp && NULL != n->parent->prev)
+               outflags |= MMAN_sp;
+       print_line(".RS", 0);
        print_offs(n->norm->Bd.offs);
        outflags |= MMAN_nl;
        return(1);
@@ -633,14 +767,44 @@ static void
 post_bd(DECL_ARGS)
 {
 
-       outflags |= MMAN_nl;
-       print_word(".RE");
+       print_line(".RE", MMAN_nl);
        if (DISP_unfilled == n->norm->Bd.type ||
-           DISP_literal  == n->norm->Bd.type) {
-               outflags |= MMAN_nl;
-               print_word(".fi");
+           DISP_literal  == n->norm->Bd.type)
+               print_line(".fi", MMAN_nl);
+}
+
+static int
+pre_bf(DECL_ARGS)
+{
+
+       switch (n->type) {
+       case (MDOC_BLOCK):
+               return(1);
+       case (MDOC_BODY):
+               break;
+       default:
+               return(0);
        }
-       outflags |= MMAN_nl;
+       switch (n->norm->Bf.font) {
+       case (FONT_Em):
+               font_push('I');
+               break;
+       case (FONT_Sy):
+               font_push('B');
+               break;
+       default:
+               font_push('R');
+               break;
+       }
+       return(1);
+}
+
+static void
+post_bf(DECL_ARGS)
+{
+
+       if (MDOC_BODY == n->type)
+               font_pop();
 }
 
 static int
@@ -666,6 +830,47 @@ post_bk(DECL_ARGS)
                outflags &= ~MMAN_Bk;
 }
 
+static int
+pre_bl(DECL_ARGS)
+{
+       size_t           icol;
+
+       switch (n->norm->Bl.type) {
+       case (LIST_enum):
+               n->norm->Bl.count = 0;
+               return(1);
+       case (LIST_column):
+               break;
+       default:
+               return(1);
+       }
+
+       print_line(".TS", MMAN_nl);
+       for (icol = 0; icol < n->norm->Bl.ncols; icol++)
+               print_word("l");
+       print_word(".");
+       outflags |= MMAN_nl;
+       return(1);
+}
+
+static void
+post_bl(DECL_ARGS)
+{
+
+       switch (n->norm->Bl.type) {
+       case (LIST_column):
+               print_line(".TE", 0);
+               break;
+       case (LIST_enum):
+               n->norm->Bl.count = 0;
+               break;
+       default:
+               break;
+       }
+       outflags |= MMAN_PP | MMAN_nl;
+       outflags &= ~(MMAN_sp | MMAN_br);
+}
+
 static int
 pre_br(DECL_ARGS)
 {
@@ -698,9 +903,7 @@ static int
 pre_dl(DECL_ARGS)
 {
 
-       outflags |= MMAN_nl;
-       print_word(".RS 6n");
-       outflags |= MMAN_nl;
+       print_line(".RS 6n", MMAN_nl);
        return(1);
 }
 
@@ -708,9 +911,7 @@ static void
 post_dl(DECL_ARGS)
 {
 
-       outflags |= MMAN_nl;
-       print_word(".RE");
-       outflags |= MMAN_nl;
+       print_line(".RE", MMAN_nl);
 }
 
 static int
@@ -721,6 +922,14 @@ pre_em(DECL_ARGS)
        return(1);
 }
 
+static void
+post_eo(DECL_ARGS)
+{
+
+       if (MDOC_HEAD == n->type || MDOC_BODY == n->type)
+               outflags &= ~MMAN_spc;
+}
+
 static int
 pre_fa(DECL_ARGS)
 {
@@ -746,6 +955,23 @@ post_fa(DECL_ARGS)
                print_word(",");
 }
 
+static int
+pre_fd(DECL_ARGS)
+{
+
+       pre_syn(n);
+       font_push('B');
+       return(1);
+}
+
+static void
+post_fd(DECL_ARGS)
+{
+
+       font_pop();
+       outflags |= MMAN_br;
+}
+
 static int
 pre_fl(DECL_ARGS)
 {
@@ -761,7 +987,8 @@ post_fl(DECL_ARGS)
 {
 
        font_pop();
-       if (0 == n->nchild)
+       if (0 == n->nchild && NULL != n->next &&
+                       n->next->line == n->line)
                outflags &= ~MMAN_spc;
 }
 
@@ -781,7 +1008,11 @@ pre_fn(DECL_ARGS)
        outflags &= ~MMAN_spc;
        print_word("(");
        outflags &= ~MMAN_spc;
-       return(pre_fa(m, n->next));
+
+       n = n->next;
+       if (NULL != n)
+               pre_fa(m, n);
+       return(0);
 }
 
 static void
@@ -880,26 +1111,102 @@ pre_it(DECL_ARGS)
 {
        const struct mdoc_node *bln;
 
-       if (MDOC_HEAD == n->type) {
-               outflags |= MMAN_nl;
-               print_word(".TP");
-               bln = n->parent->parent->prev;
+       switch (n->type) {
+       case (MDOC_HEAD):
+               outflags |= MMAN_PP | MMAN_nl;
+               bln = n->parent->parent;
+               if (0 == bln->norm->Bl.comp ||
+                   (NULL == n->parent->prev &&
+                    NULL == bln->parent->prev))
+                       outflags |= MMAN_sp;
+               outflags &= ~MMAN_br;
                switch (bln->norm->Bl.type) {
+               case (LIST_item):
+                       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);
                case (LIST_bullet):
-                       print_word("4n");
+                       /* FALLTHROUGH */
+               case (LIST_dash):
+                       /* FALLTHROUGH */
+               case (LIST_hyphen):
+                       print_width(bln->norm->Bl.width, NULL, 0);
+                       TPremain = 0;
                        outflags |= MMAN_nl;
-                       print_word("\\fBo\\fP");
+                       font_push('B');
+                       if (LIST_bullet == bln->norm->Bl.type)
+                               print_word("o");
+                       else
+                               print_word("-");
+                       font_pop();
                        break;
-               default:
-                       if (bln->norm->Bl.width)
-                               print_word(bln->norm->Bl.width);
+               case (LIST_enum):
+                       print_width(bln->norm->Bl.width, NULL, 0);
+                       TPremain = 0;
+                       outflags |= MMAN_nl;
+                       print_count(&bln->norm->Bl.count);
                        break;
+               case (LIST_hang):
+                       print_width(bln->norm->Bl.width, n->child, 6);
+                       TPremain = 0;
+                       break;
+               case (LIST_tag):
+                       print_width(bln->norm->Bl.width, n->child, 0);
+                       putchar('\n');
+                       outflags &= ~MMAN_spc;
+                       return(1);
+               default:
+                       return(1);
                }
                outflags |= MMAN_nl;
+       default:
+               break;
        }
        return(1);
 }
 
+static void
+post_it(DECL_ARGS)
+{
+       const struct mdoc_node *bln;
+
+       bln = n->parent->parent;
+
+       switch (n->type) {
+       case (MDOC_HEAD):
+               switch (bln->norm->Bl.type) {
+               case (LIST_diag):
+                       outflags &= ~MMAN_spc;
+                       print_word("\\ ");
+                       break;
+               case (LIST_ohang):
+                       outflags |= MMAN_br;
+                       break;
+               default:
+                       break;
+               }
+               break;
+       case (MDOC_BODY):
+               if (LIST_column == bln->norm->Bl.type &&
+                   NULL != n->next) {
+                       putchar('\t');
+                       outflags &= ~MMAN_spc;
+               }
+               break;
+       default:
+               break;
+       }
+}
+
 static void
 post_lb(DECL_ARGS)
 {
@@ -943,13 +1250,22 @@ pre_li(DECL_ARGS)
 static int
 pre_nm(DECL_ARGS)
 {
+       char    *name;
 
        if (MDOC_BLOCK == n->type)
                pre_syn(n);
        if (MDOC_ELEM != n->type && MDOC_HEAD != n->type)
                return(1);
-       if (NULL == n->child && NULL == m->name)
+       name = n->child ? n->child->string : m->name;
+       if (NULL == name)
                return(0);
+       if (MDOC_HEAD == n->type) {
+               if (NULL == n->parent->prev)
+                       outflags |= MMAN_sp;
+               print_block(".HP", 0);
+               printf(" %ldn", strlen(name) + 1);
+               outflags |= MMAN_nl;
+       }
        font_push('B');
        if (NULL == n->child)
                print_word(m->name);
@@ -992,13 +1308,22 @@ static int
 pre_pp(DECL_ARGS)
 {
 
-       outflags |= MMAN_nl;
-       if (MDOC_It == n->parent->tok)
-               print_word(".sp");
-       else
-               print_word(".PP");
-       outflags |= MMAN_nl;
-       return(MDOC_Rs == n->tok);
+       if (MDOC_It != n->parent->tok)
+               outflags |= MMAN_PP;
+       outflags |= MMAN_sp | MMAN_nl;
+       outflags &= ~MMAN_br;
+       return(0);
+}
+
+static int
+pre_rs(DECL_ARGS)
+{
+
+       if (SEC_SEE_ALSO == n->sec) {
+               outflags |= MMAN_PP | MMAN_sp | MMAN_nl;
+               outflags &= ~MMAN_br;
+       }
+       return(1);
 }
 
 static int
@@ -1007,7 +1332,7 @@ pre_sm(DECL_ARGS)
 
        assert(n->child && MDOC_TEXT == n->child->type);
        if (0 == strcmp("on", n->child->string))
-               outflags |= MMAN_Sm;
+               outflags |= MMAN_Sm | MMAN_spc;
        else
                outflags &= ~MMAN_Sm;
        return(0);
@@ -1017,8 +1342,11 @@ static int
 pre_sp(DECL_ARGS)
 {
 
-       outflags |= MMAN_nl;
-       print_word(".sp");
+       if (MMAN_PP & outflags) {
+               outflags &= ~MMAN_PP;
+               print_line(".PP", 0);
+       } else
+               print_line(".sp", 0);
        return(1);
 }