]> git.cameronkatri.com Git - mandoc.git/blobdiff - man_term.c
git.ckatri.com / mandoc.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
Simplify: write HTTP 303 redirects with relative locations.
[mandoc.git] / man_term.c
diff --git a/man_term.c b/man_term.c
index 5a659503e3876c9813e7e0e2f28d205b7893e63c..b2732d45589137a6d10468ce8fdc0ff327cb7476 100644 (file)
--- a/man_term.c
+++ b/man_term.c
@@ -1,35 +1,36 @@
-/*     $Id: man_term.c,v 1.145 2014/04/08 07:13:11 schwarze Exp $ */
+/*     $Id: man_term.c,v 1.191 2017/02/15 14:10:08 schwarze Exp $ */
 /*
  * Copyright (c) 2008-2012 Kristaps Dzonsons <kristaps@bsd.lv>
 /*
  * Copyright (c) 2008-2012 Kristaps Dzonsons <kristaps@bsd.lv>
- * Copyright (c) 2010-2014 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2010-2015, 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
  * copyright notice and this permission notice appear in all copies.
  *
  *
  * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
  * copyright notice and this permission notice appear in all copies.
  *
- * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+ * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHORS DISCLAIM ALL WARRANTIES
  * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
  * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
- * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+ * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR
  * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
  * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
  * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
  * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
-#ifdef HAVE_CONFIG_H
 #include "config.h"
 #include "config.h"
-#endif
 
 #include <sys/types.h>
 
 #include <assert.h>
 #include <ctype.h>
 
 #include <sys/types.h>
 
 #include <assert.h>
 #include <ctype.h>
+#include <limits.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 
+#include "mandoc_aux.h"
 #include "mandoc.h"
 #include "mandoc.h"
-#include "out.h"
+#include "roff.h"
 #include "man.h"
 #include "man.h"
+#include "out.h"
 #include "term.h"
 #include "main.h"
 
 #include "term.h"
 #include "main.h"
 
@@ -38,17 +39,17 @@
 struct mtermp {
        int               fl;
 #define        MANT_LITERAL     (1 << 0)
 struct mtermp {
        int               fl;
 #define        MANT_LITERAL     (1 << 0)
-       size_t            lmargin[MAXMARGINS]; /* margins (incl. visible page) */
+       int               lmargin[MAXMARGINS]; /* margins (incl. vis. page) */
        int               lmargincur; /* index of current margin */
        int               lmarginsz; /* actual number of nested margins */
        size_t            offset; /* default offset to visible page */
        int               pardist; /* vert. space before par., unit: [v] */
 };
 
        int               lmargincur; /* index of current margin */
        int               lmarginsz; /* actual number of nested margins */
        size_t            offset; /* default offset to visible page */
        int               pardist; /* vert. space before par., unit: [v] */
 };
 
-#define        DECL_ARGS         struct termp *p, \
+#define        DECL_ARGS         struct termp *p, \
                          struct mtermp *mt, \
                          struct mtermp *mt, \
-                         const struct man_node *n, \
-                         const struct man_meta *meta
+                         struct roff_node *n, \
+                         const struct roff_meta *meta
 
 struct termact {
        int             (*pre)(DECL_ARGS);
 
 struct termact {
        int             (*pre)(DECL_ARGS);
@@ -57,15 +58,14 @@ struct      termact {
 #define        MAN_NOTEXT       (1 << 0) /* Never has text children. */
 };
 
 #define        MAN_NOTEXT       (1 << 0) /* Never has text children. */
 };
 
-static int               a2width(const struct termp *, const char *);
-static size_t            a2height(const struct termp *, const char *);
-
 static void              print_man_nodelist(DECL_ARGS);
 static void              print_man_node(DECL_ARGS);
 static void              print_man_nodelist(DECL_ARGS);
 static void              print_man_node(DECL_ARGS);
-static void              print_man_head(struct termp *, const void *);
-static void              print_man_foot(struct termp *, const void *);
-static void              print_bvspace(struct termp *, 
-                               const struct man_node *, int);
+static void              print_man_head(struct termp *,
+                               const struct roff_meta *);
+static void              print_man_foot(struct termp *,
+                               const struct roff_meta *);
+static void              print_bvspace(struct termp *,
+                               const struct roff_node *, int);
 
 static int               pre_B(DECL_ARGS);
 static int               pre_HP(DECL_ARGS);
 
 static int               pre_B(DECL_ARGS);
 static int               pre_HP(DECL_ARGS);
@@ -105,7 +105,7 @@ static      const struct termact termacts[MAN_MAX] = {
        { pre_PP, NULL, 0 }, /* PP */
        { pre_PP, NULL, 0 }, /* P */
        { pre_IP, post_IP, 0 }, /* IP */
        { pre_PP, NULL, 0 }, /* PP */
        { pre_PP, NULL, 0 }, /* P */
        { pre_IP, post_IP, 0 }, /* IP */
-       { pre_HP, post_HP, 0 }, /* HP */ 
+       { pre_HP, post_HP, 0 }, /* HP */
        { NULL, NULL, 0 }, /* SM */
        { pre_B, NULL, 0 }, /* SB */
        { pre_alternate, NULL, 0 }, /* BI */
        { NULL, NULL, 0 }, /* SM */
        { pre_B, NULL, 0 }, /* SB */
        { pre_alternate, NULL, 0 }, /* BI */
@@ -117,14 +117,13 @@ static    const struct termact termacts[MAN_MAX] = {
        { pre_I, NULL, 0 }, /* I */
        { pre_alternate, NULL, 0 }, /* IR */
        { pre_alternate, NULL, 0 }, /* RI */
        { pre_I, NULL, 0 }, /* I */
        { pre_alternate, NULL, 0 }, /* IR */
        { pre_alternate, NULL, 0 }, /* RI */
-       { pre_ign, NULL, MAN_NOTEXT }, /* na */
        { pre_sp, NULL, MAN_NOTEXT }, /* sp */
        { pre_literal, NULL, 0 }, /* nf */
        { pre_literal, NULL, 0 }, /* fi */
        { NULL, NULL, 0 }, /* RE */
        { pre_RS, post_RS, 0 }, /* RS */
        { pre_ign, NULL, 0 }, /* DT */
        { pre_sp, NULL, MAN_NOTEXT }, /* sp */
        { pre_literal, NULL, 0 }, /* nf */
        { pre_literal, NULL, 0 }, /* fi */
        { NULL, NULL, 0 }, /* RE */
        { pre_RS, post_RS, 0 }, /* RS */
        { pre_ign, NULL, 0 }, /* DT */
-       { pre_ign, NULL, 0 }, /* UC */
+       { pre_ign, NULL, MAN_NOTEXT }, /* UC */
        { pre_PD, NULL, MAN_NOTEXT }, /* PD */
        { pre_ign, NULL, 0 }, /* AT */
        { pre_in, NULL, MAN_NOTEXT }, /* in */
        { pre_PD, NULL, MAN_NOTEXT }, /* PD */
        { pre_ign, NULL, 0 }, /* AT */
        { pre_in, NULL, MAN_NOTEXT }, /* in */
@@ -138,67 +137,50 @@ static    const struct termact termacts[MAN_MAX] = {
 };
 
 
 };
 
 
-
 void
 void
-terminal_man(void *arg, const struct man *man)
+terminal_man(void *arg, const struct roff_man *man)
 {
        struct termp            *p;
 {
        struct termp            *p;
-       const struct man_node   *n;
-       const struct man_meta   *meta;
+       struct roff_node        *n;
        struct mtermp            mt;
        struct mtermp            mt;
+       size_t                   save_defindent;
 
        p = (struct termp *)arg;
 
        p = (struct termp *)arg;
-
-       if (0 == p->defindent)
-               p->defindent = 7;
-
        p->overstep = 0;
        p->overstep = 0;
-       p->maxrmargin = p->defrmargin;
+       p->rmargin = p->maxrmargin = p->defrmargin;
        p->tabwidth = term_len(p, 5);
 
        p->tabwidth = term_len(p, 5);
 
-       if (NULL == p->symtab)
-               p->symtab = mchars_alloc();
-
-       n = man_node(man);
-       meta = man_meta(man);
-
-       term_begin(p, print_man_head, print_man_foot, meta);
-       p->flags |= TERMP_NOSPACE;
-
        memset(&mt, 0, sizeof(struct mtermp));
        memset(&mt, 0, sizeof(struct mtermp));
-
        mt.lmargin[mt.lmargincur] = term_len(p, p->defindent);
        mt.offset = term_len(p, p->defindent);
        mt.pardist = 1;
 
        mt.lmargin[mt.lmargincur] = term_len(p, p->defindent);
        mt.offset = term_len(p, p->defindent);
        mt.pardist = 1;
 
-       if (n->child)
-               print_man_nodelist(p, &mt, n->child, meta);
-
-       term_end(p);
-}
-
-
-static size_t
-a2height(const struct termp *p, const char *cp)
-{
-       struct roffsu    su;
-
-       if ( ! a2roffsu(cp, &su, SCALE_VS))
-               SCALE_VS_INIT(&su, atoi(cp));
-
-       return(term_vspan(p, &su));
-}
-
-
-static int
-a2width(const struct termp *p, const char *cp)
-{
-       struct roffsu    su;
-
-       if ( ! a2roffsu(cp, &su, SCALE_BU))
-               return(-1);
-
-       return((int)term_hspan(p, &su));
+       n = man->first->child;
+       if (p->synopsisonly) {
+               while (n != NULL) {
+                       if (n->tok == MAN_SH &&
+                           n->child->child->type == ROFFT_TEXT &&
+                           !strcmp(n->child->child->string, "SYNOPSIS")) {
+                               if (n->child->next->child != NULL)
+                                       print_man_nodelist(p, &mt,
+                                           n->child->next->child,
+                                           &man->meta);
+                               term_newln(p);
+                               break;
+                       }
+                       n = n->next;
+               }
+       } else {
+               save_defindent = p->defindent;
+               if (p->defindent == 0)
+                       p->defindent = 7;
+               term_begin(p, print_man_head, print_man_foot, &man->meta);
+               p->flags |= TERMP_NOSPACE;
+               if (n != NULL)
+                       print_man_nodelist(p, &mt, n, &man->meta);
+               term_end(p);
+               p->defindent = save_defindent;
+       }
 }
 
 /*
 }
 
 /*
@@ -210,17 +192,17 @@ a2width(const struct termp *p, const char *cp)
  * first, print it.
  */
 static void
  * first, print it.
  */
 static void
-print_bvspace(struct termp *p, const struct man_node *n, int pardist)
+print_bvspace(struct termp *p, const struct roff_node *n, int pardist)
 {
        int      i;
 
        term_newln(p);
 
        if (n->body && n->body->child)
 {
        int      i;
 
        term_newln(p);
 
        if (n->body && n->body->child)
-               if (MAN_TBL == n->body->child->type)
+               if (n->body->child->type == ROFFT_TBL)
                        return;
 
                        return;
 
-       if (MAN_ROOT == n->parent->type || MAN_RS != n->parent->tok)
+       if (n->parent->type == ROFFT_ROOT || n->parent->tok != MAN_RS)
                if (NULL == n->prev)
                        return;
 
                if (NULL == n->prev)
                        return;
 
@@ -228,36 +210,30 @@ print_bvspace(struct termp *p, const struct man_node *n, int pardist)
                term_vspace(p);
 }
 
                term_vspace(p);
 }
 
-/* ARGSUSED */
+
 static int
 pre_ign(DECL_ARGS)
 {
 
 static int
 pre_ign(DECL_ARGS)
 {
 
-       return(0);
+       return 0;
 }
 
 }
 
-
-/* ARGSUSED */
 static int
 pre_ll(DECL_ARGS)
 {
 
 static int
 pre_ll(DECL_ARGS)
 {
 
-       term_setwidth(p, n->nchild ? n->child->string : NULL);
-       return(0);
+       term_setwidth(p, n->child != NULL ? n->child->string : NULL);
+       return 0;
 }
 
 }
 
-
-/* ARGSUSED */
 static int
 pre_I(DECL_ARGS)
 {
 
        term_fontrepl(p, TERMFONT_UNDER);
 static int
 pre_I(DECL_ARGS)
 {
 
        term_fontrepl(p, TERMFONT_UNDER);
-       return(1);
+       return 1;
 }
 
 }
 
-
-/* ARGSUSED */
 static int
 pre_literal(DECL_ARGS)
 {
 static int
 pre_literal(DECL_ARGS)
 {
@@ -282,54 +258,54 @@ pre_literal(DECL_ARGS)
                p->flags |= TERMP_NOSPACE;
        }
 
                p->flags |= TERMP_NOSPACE;
        }
 
-       return(0);
+       return 0;
 }
 
 }
 
-/* ARGSUSED */
 static int
 pre_PD(DECL_ARGS)
 {
 static int
 pre_PD(DECL_ARGS)
 {
+       struct roffsu    su;
 
        n = n->child;
 
        n = n->child;
-       if (0 == n) {
+       if (n == NULL) {
                mt->pardist = 1;
                mt->pardist = 1;
-               return(0);
+               return 0;
        }
        }
-       assert(MAN_TEXT == n->type);
-       mt->pardist = atoi(n->string);
-       return(0);
+       assert(n->type == ROFFT_TEXT);
+       if (a2roffsu(n->string, &su, SCALE_VS))
+               mt->pardist = term_vspan(p, &su);
+       return 0;
 }
 
 }
 
-/* ARGSUSED */
 static int
 pre_alternate(DECL_ARGS)
 {
        enum termfont            font[2];
 static int
 pre_alternate(DECL_ARGS)
 {
        enum termfont            font[2];
-       const struct man_node   *nn;
+       struct roff_node        *nn;
        int                      savelit, i;
 
        switch (n->tok) {
        int                      savelit, i;
 
        switch (n->tok) {
-       case (MAN_RB):
+       case MAN_RB:
                font[0] = TERMFONT_NONE;
                font[1] = TERMFONT_BOLD;
                break;
                font[0] = TERMFONT_NONE;
                font[1] = TERMFONT_BOLD;
                break;
-       case (MAN_RI):
+       case MAN_RI:
                font[0] = TERMFONT_NONE;
                font[1] = TERMFONT_UNDER;
                break;
                font[0] = TERMFONT_NONE;
                font[1] = TERMFONT_UNDER;
                break;
-       case (MAN_BR):
+       case MAN_BR:
                font[0] = TERMFONT_BOLD;
                font[1] = TERMFONT_NONE;
                break;
                font[0] = TERMFONT_BOLD;
                font[1] = TERMFONT_NONE;
                break;
-       case (MAN_BI):
+       case MAN_BI:
                font[0] = TERMFONT_BOLD;
                font[1] = TERMFONT_UNDER;
                break;
                font[0] = TERMFONT_BOLD;
                font[1] = TERMFONT_UNDER;
                break;
-       case (MAN_IR):
+       case MAN_IR:
                font[0] = TERMFONT_UNDER;
                font[1] = TERMFONT_NONE;
                break;
                font[0] = TERMFONT_UNDER;
                font[1] = TERMFONT_NONE;
                break;
-       case (MAN_IB):
+       case MAN_IB:
                font[0] = TERMFONT_UNDER;
                font[1] = TERMFONT_BOLD;
                break;
                font[0] = TERMFONT_UNDER;
                font[1] = TERMFONT_BOLD;
                break;
@@ -344,24 +320,25 @@ pre_alternate(DECL_ARGS)
                term_fontrepl(p, font[i]);
                if (savelit && NULL == nn->next)
                        mt->fl |= MANT_LITERAL;
                term_fontrepl(p, font[i]);
                if (savelit && NULL == nn->next)
                        mt->fl |= MANT_LITERAL;
-               print_man_node(p, mt, nn, meta);
+               assert(nn->type == ROFFT_TEXT);
+               term_word(p, nn->string);
+               if (nn->flags & NODE_EOS)
+                       p->flags |= TERMP_SENTENCE;
                if (nn->next)
                        p->flags |= TERMP_NOSPACE;
        }
 
                if (nn->next)
                        p->flags |= TERMP_NOSPACE;
        }
 
-       return(0);
+       return 0;
 }
 
 }
 
-/* ARGSUSED */
 static int
 pre_B(DECL_ARGS)
 {
 
        term_fontrepl(p, TERMFONT_BOLD);
 static int
 pre_B(DECL_ARGS)
 {
 
        term_fontrepl(p, TERMFONT_BOLD);
-       return(1);
+       return 1;
 }
 
 }
 
-/* ARGSUSED */
 static int
 pre_OP(DECL_ARGS)
 {
 static int
 pre_OP(DECL_ARGS)
 {
@@ -381,10 +358,9 @@ pre_OP(DECL_ARGS)
        term_fontrepl(p, TERMFONT_NONE);
        p->flags |= TERMP_NOSPACE;
        term_word(p, "]");
        term_fontrepl(p, TERMFONT_NONE);
        p->flags |= TERMP_NOSPACE;
        term_word(p, "]");
-       return(0);
+       return 0;
 }
 
 }
 
-/* ARGSUSED */
 static int
 pre_ft(DECL_ARGS)
 {
 static int
 pre_ft(DECL_ARGS)
 {
@@ -392,52 +368,47 @@ pre_ft(DECL_ARGS)
 
        if (NULL == n->child) {
                term_fontlast(p);
 
        if (NULL == n->child) {
                term_fontlast(p);
-               return(0);
+               return 0;
        }
 
        cp = n->child->string;
        switch (*cp) {
        }
 
        cp = n->child->string;
        switch (*cp) {
-       case ('4'):
-               /* FALLTHROUGH */
-       case ('3'):
-               /* FALLTHROUGH */
-       case ('B'):
+       case '4':
+       case '3':
+       case 'B':
                term_fontrepl(p, TERMFONT_BOLD);
                break;
                term_fontrepl(p, TERMFONT_BOLD);
                break;
-       case ('2'):
-               /* FALLTHROUGH */
-       case ('I'):
+       case '2':
+       case 'I':
                term_fontrepl(p, TERMFONT_UNDER);
                break;
                term_fontrepl(p, TERMFONT_UNDER);
                break;
-       case ('P'):
+       case 'P':
                term_fontlast(p);
                break;
                term_fontlast(p);
                break;
-       case ('1'):
-               /* FALLTHROUGH */
-       case ('C'):
-               /* FALLTHROUGH */
-       case ('R'):
+       case '1':
+       case 'C':
+       case 'R':
                term_fontrepl(p, TERMFONT_NONE);
                break;
        default:
                break;
        }
                term_fontrepl(p, TERMFONT_NONE);
                break;
        default:
                break;
        }
-       return(0);
+       return 0;
 }
 
 }
 
-/* ARGSUSED */
 static int
 pre_in(DECL_ARGS)
 {
 static int
 pre_in(DECL_ARGS)
 {
-       int              len, less;
-       size_t           v;
+       struct roffsu    su;
        const char      *cp;
        const char      *cp;
+       size_t           v;
+       int              less;
 
        term_newln(p);
 
        if (NULL == n->child) {
                p->offset = mt->offset;
 
        term_newln(p);
 
        if (NULL == n->child) {
                p->offset = mt->offset;
-               return(0);
+               return 0;
        }
 
        cp = n->child->string;
        }
 
        cp = n->child->string;
@@ -450,100 +421,89 @@ pre_in(DECL_ARGS)
        else
                cp--;
 
        else
                cp--;
 
-       if ((len = a2width(p, ++cp)) < 0)
-               return(0);
+       if ( ! a2roffsu(++cp, &su, SCALE_EN))
+               return 0;
 
 
-       v = (size_t)len;
+       v = (term_hspan(p, &su) + 11) / 24;
 
        if (less < 0)
                p->offset -= p->offset > v ? v : p->offset;
        else if (less > 0)
                p->offset += v;
 
        if (less < 0)
                p->offset -= p->offset > v ? v : p->offset;
        else if (less > 0)
                p->offset += v;
-       else 
+       else
                p->offset = v;
                p->offset = v;
+       if (p->offset > SHRT_MAX)
+               p->offset = term_len(p, p->defindent);
 
 
-       /* Don't let this creep beyond the right margin. */
-
-       if (p->offset > p->rmargin)
-               p->offset = p->rmargin;
-
-       return(0);
+       return 0;
 }
 
 }
 
-
-/* ARGSUSED */
 static int
 pre_sp(DECL_ARGS)
 {
 static int
 pre_sp(DECL_ARGS)
 {
-       char            *s;
-       size_t           i, len;
-       int              neg;
+       struct roffsu    su;
+       int              i, len;
 
        if ((NULL == n->prev && n->parent)) {
                switch (n->parent->tok) {
 
        if ((NULL == n->prev && n->parent)) {
                switch (n->parent->tok) {
-               case (MAN_SH):
-                       /* FALLTHROUGH */
-               case (MAN_SS):
-                       /* FALLTHROUGH */
-               case (MAN_PP):
-                       /* FALLTHROUGH */
-               case (MAN_LP):
-                       /* FALLTHROUGH */
-               case (MAN_P):
-                       /* FALLTHROUGH */
-                       return(0);
+               case MAN_SH:
+               case MAN_SS:
+               case MAN_PP:
+               case MAN_LP:
+               case MAN_P:
+                       return 0;
                default:
                        break;
                }
        }
 
                default:
                        break;
                }
        }
 
-       neg = 0;
-       switch (n->tok) {
-       case (MAN_br):
+       if (n->tok == MAN_br)
                len = 0;
                len = 0;
-               break;
-       default:
-               if (NULL == n->child) {
-                       len = 1;
-                       break;
-               }
-               s = n->child->string;
-               if ('-' == *s) {
-                       neg = 1;
-                       s++;
-               }
-               len = a2height(p, s);
-               break;
+       else if (n->child == NULL)
+               len = 1;
+       else {
+               if ( ! a2roffsu(n->child->string, &su, SCALE_VS))
+                       su.scale = 1.0;
+               len = term_vspan(p, &su);
        }
 
        }
 
-       if (0 == len)
+       if (len == 0)
                term_newln(p);
                term_newln(p);
-       else if (neg)
-               p->skipvsp += len;
+       else if (len < 0)
+               p->skipvsp -= len;
        else
                for (i = 0; i < len; i++)
                        term_vspace(p);
 
        else
                for (i = 0; i < len; i++)
                        term_vspace(p);
 
-       return(0);
-}
+       /*
+        * Handle an explicit break request in the same way
+        * as an overflowing line.
+        */
+
+       if (p->flags & TERMP_BRIND) {
+               p->offset = p->rmargin;
+               p->rmargin = p->maxrmargin;
+               p->flags &= ~(TERMP_NOBREAK | TERMP_BRIND);
+       }
 
 
+       return 0;
+}
 
 
-/* ARGSUSED */
 static int
 pre_HP(DECL_ARGS)
 {
 static int
 pre_HP(DECL_ARGS)
 {
-       size_t                   len, one;
-       int                      ival;
-       const struct man_node   *nn;
+       struct roffsu            su;
+       const struct roff_node  *nn;
+       int                      len;
 
        switch (n->type) {
 
        switch (n->type) {
-       case (MAN_BLOCK):
+       case ROFFT_BLOCK:
                print_bvspace(p, n, mt->pardist);
                print_bvspace(p, n, mt->pardist);
-               return(1);
-       case (MAN_BODY):
+               return 1;
+       case ROFFT_BODY:
                break;
        default:
                break;
        default:
-               return(0);
+               return 0;
        }
 
        if ( ! (MANT_LITERAL & mt->fl)) {
        }
 
        if ( ! (MANT_LITERAL & mt->fl)) {
@@ -551,37 +511,42 @@ pre_HP(DECL_ARGS)
                p->trailspace = 2;
        }
 
                p->trailspace = 2;
        }
 
-       len = mt->lmargin[mt->lmargincur];
-       ival = -1;
-
        /* Calculate offset. */
 
        /* Calculate offset. */
 
-       if (NULL != (nn = n->parent->head->child))
-               if ((ival = a2width(p, nn->string)) >= 0)
-                       len = (size_t)ival;
-
-       one = term_len(p, 1);
-       if (len < one)
-               len = one;
+       if ((nn = n->parent->head->child) != NULL &&
+           a2roffsu(nn->string, &su, SCALE_EN)) {
+               len = term_hspan(p, &su) / 24;
+               if (len < 0 && (size_t)(-len) > mt->offset)
+                       len = -mt->offset;
+               else if (len > SHRT_MAX)
+                       len = term_len(p, p->defindent);
+               mt->lmargin[mt->lmargincur] = len;
+       } else
+               len = mt->lmargin[mt->lmargincur];
 
        p->offset = mt->offset;
        p->rmargin = mt->offset + len;
 
        p->offset = mt->offset;
        p->rmargin = mt->offset + len;
-
-       if (ival >= 0)
-               mt->lmargin[mt->lmargincur] = (size_t)ival;
-
-       return(1);
+       return 1;
 }
 
 }
 
-
-/* ARGSUSED */
 static void
 post_HP(DECL_ARGS)
 {
 
        switch (n->type) {
 static void
 post_HP(DECL_ARGS)
 {
 
        switch (n->type) {
-       case (MAN_BODY):
+       case ROFFT_BODY:
                term_newln(p);
                term_newln(p);
+
+               /*
+                * Compatibility with a groff bug.
+                * The .HP macro uses the undocumented .tag request
+                * which causes a line break and cancels no-space
+                * mode even if there isn't any output.
+                */
+
+               if (n->child == NULL)
+                       term_vspace(p);
+
                p->flags &= ~(TERMP_NOBREAK | TERMP_BRIND);
                p->trailspace = 0;
                p->offset = mt->offset;
                p->flags &= ~(TERMP_NOBREAK | TERMP_BRIND);
                p->trailspace = 0;
                p->offset = mt->offset;
@@ -592,14 +557,12 @@ post_HP(DECL_ARGS)
        }
 }
 
        }
 }
 
-
-/* ARGSUSED */
 static int
 pre_PP(DECL_ARGS)
 {
 
        switch (n->type) {
 static int
 pre_PP(DECL_ARGS)
 {
 
        switch (n->type) {
-       case (MAN_BLOCK):
+       case ROFFT_BLOCK:
                mt->lmargin[mt->lmargincur] = term_len(p, p->defindent);
                print_bvspace(p, n, mt->pardist);
                break;
                mt->lmargin[mt->lmargincur] = term_len(p, p->defindent);
                print_bvspace(p, n, mt->pardist);
                break;
@@ -608,55 +571,48 @@ pre_PP(DECL_ARGS)
                break;
        }
 
                break;
        }
 
-       return(MAN_HEAD != n->type);
+       return n->type != ROFFT_HEAD;
 }
 
 }
 
-
-/* ARGSUSED */
 static int
 pre_IP(DECL_ARGS)
 {
 static int
 pre_IP(DECL_ARGS)
 {
-       const struct man_node   *nn;
-       size_t                   len;
-       int                      savelit, ival;
+       struct roffsu            su;
+       const struct roff_node  *nn;
+       int                      len, savelit;
 
        switch (n->type) {
 
        switch (n->type) {
-       case (MAN_BODY):
+       case ROFFT_BODY:
                p->flags |= TERMP_NOSPACE;
                break;
                p->flags |= TERMP_NOSPACE;
                break;
-       case (MAN_HEAD):
+       case ROFFT_HEAD:
                p->flags |= TERMP_NOBREAK;
                p->trailspace = 1;
                break;
                p->flags |= TERMP_NOBREAK;
                p->trailspace = 1;
                break;
-       case (MAN_BLOCK):
+       case ROFFT_BLOCK:
                print_bvspace(p, n, mt->pardist);
                /* FALLTHROUGH */
        default:
                print_bvspace(p, n, mt->pardist);
                /* FALLTHROUGH */
        default:
-               return(1);
+               return 1;
        }
 
        }
 
-       len = mt->lmargin[mt->lmargincur];
-       ival = -1;
-
        /* Calculate the offset from the optional second argument. */
        /* Calculate the offset from the optional second argument. */
-       if (NULL != (nn = n->parent->head->child))
-               if (NULL != (nn = nn->next))
-                       if ((ival = a2width(p, nn->string)) >= 0)
-                               len = (size_t)ival;
+       if ((nn = n->parent->head->child) != NULL &&
+           (nn = nn->next) != NULL &&
+           a2roffsu(nn->string, &su, SCALE_EN)) {
+               len = term_hspan(p, &su) / 24;
+               if (len < 0 && (size_t)(-len) > mt->offset)
+                       len = -mt->offset;
+               else if (len > SHRT_MAX)
+                       len = term_len(p, p->defindent);
+               mt->lmargin[mt->lmargincur] = len;
+       } else
+               len = mt->lmargin[mt->lmargincur];
 
        switch (n->type) {
 
        switch (n->type) {
-       case (MAN_HEAD):
-               /* Handle zero-width lengths. */
-               if (0 == len)
-                       len = term_len(p, 1);
-
+       case ROFFT_HEAD:
                p->offset = mt->offset;
                p->rmargin = mt->offset + len;
                p->offset = mt->offset;
                p->rmargin = mt->offset + len;
-               if (ival < 0)
-                       break;
-
-               /* Set the saved left-margin. */
-               mt->lmargin[mt->lmargincur] = (size_t)ival;
 
                savelit = MANT_LITERAL & mt->fl;
                mt->fl &= ~MANT_LITERAL;
 
                savelit = MANT_LITERAL & mt->fl;
                mt->fl &= ~MANT_LITERAL;
@@ -667,33 +623,30 @@ pre_IP(DECL_ARGS)
                if (savelit)
                        mt->fl |= MANT_LITERAL;
 
                if (savelit)
                        mt->fl |= MANT_LITERAL;
 
-               return(0);
-       case (MAN_BODY):
+               return 0;
+       case ROFFT_BODY:
                p->offset = mt->offset + len;
                p->offset = mt->offset + len;
-               p->rmargin = p->maxrmargin > p->offset ?
-                               p->maxrmargin : p->offset;
+               p->rmargin = p->maxrmargin;
                break;
        default:
                break;
        }
 
                break;
        default:
                break;
        }
 
-       return(1);
+       return 1;
 }
 
 }
 
-
-/* ARGSUSED */
 static void
 post_IP(DECL_ARGS)
 {
 
        switch (n->type) {
 static void
 post_IP(DECL_ARGS)
 {
 
        switch (n->type) {
-       case (MAN_HEAD):
+       case ROFFT_HEAD:
                term_flushln(p);
                p->flags &= ~TERMP_NOBREAK;
                p->trailspace = 0;
                p->rmargin = p->maxrmargin;
                break;
                term_flushln(p);
                p->flags &= ~TERMP_NOBREAK;
                p->trailspace = 0;
                p->rmargin = p->maxrmargin;
                break;
-       case (MAN_BODY):
+       case ROFFT_BODY:
                term_newln(p);
                p->offset = mt->offset;
                break;
                term_newln(p);
                p->offset = mt->offset;
                break;
@@ -702,46 +655,44 @@ post_IP(DECL_ARGS)
        }
 }
 
        }
 }
 
-
-/* ARGSUSED */
 static int
 pre_TP(DECL_ARGS)
 {
 static int
 pre_TP(DECL_ARGS)
 {
-       const struct man_node   *nn;
-       size_t                   len;
-       int                      savelit, ival;
+       struct roffsu            su;
+       struct roff_node        *nn;
+       int                      len, savelit;
 
        switch (n->type) {
 
        switch (n->type) {
-       case (MAN_HEAD):
-               p->flags |= TERMP_NOBREAK;
+       case ROFFT_HEAD:
+               p->flags |= TERMP_NOBREAK | TERMP_BRTRSP;
                p->trailspace = 1;
                break;
                p->trailspace = 1;
                break;
-       case (MAN_BODY):
+       case ROFFT_BODY:
                p->flags |= TERMP_NOSPACE;
                break;
                p->flags |= TERMP_NOSPACE;
                break;
-       case (MAN_BLOCK):
+       case ROFFT_BLOCK:
                print_bvspace(p, n, mt->pardist);
                /* FALLTHROUGH */
        default:
                print_bvspace(p, n, mt->pardist);
                /* FALLTHROUGH */
        default:
-               return(1);
+               return 1;
        }
 
        }
 
-       len = (size_t)mt->lmargin[mt->lmargincur];
-       ival = -1;
-
        /* Calculate offset. */
 
        /* Calculate offset. */
 
-       if (NULL != (nn = n->parent->head->child))
-               if (nn->string && 0 == (MAN_LINE & nn->flags))
-                       if ((ival = a2width(p, nn->string)) >= 0)
-                               len = (size_t)ival;
+       if ((nn = n->parent->head->child) != NULL &&
+           nn->string != NULL && ! (NODE_LINE & nn->flags) &&
+           a2roffsu(nn->string, &su, SCALE_EN)) {
+               len = term_hspan(p, &su) / 24;
+               if (len < 0 && (size_t)(-len) > mt->offset)
+                       len = -mt->offset;
+               else if (len > SHRT_MAX)
+                       len = term_len(p, p->defindent);
+               mt->lmargin[mt->lmargincur] = len;
+       } else
+               len = mt->lmargin[mt->lmargincur];
 
        switch (n->type) {
 
        switch (n->type) {
-       case (MAN_HEAD):
-               /* Handle zero-length properly. */
-               if (0 == len)
-                       len = term_len(p, 1);
-
+       case ROFFT_HEAD:
                p->offset = mt->offset;
                p->rmargin = mt->offset + len;
 
                p->offset = mt->offset;
                p->rmargin = mt->offset + len;
 
@@ -750,7 +701,7 @@ pre_TP(DECL_ARGS)
 
                /* Don't print same-line elements. */
                nn = n->child;
 
                /* Don't print same-line elements. */
                nn = n->child;
-               while (NULL != nn && 0 == (MAN_LINE & nn->flags))
+               while (NULL != nn && 0 == (NODE_LINE & nn->flags))
                        nn = nn->next;
 
                while (NULL != nn) {
                        nn = nn->next;
 
                while (NULL != nn) {
@@ -760,35 +711,29 @@ pre_TP(DECL_ARGS)
 
                if (savelit)
                        mt->fl |= MANT_LITERAL;
 
                if (savelit)
                        mt->fl |= MANT_LITERAL;
-               if (ival >= 0)
-                       mt->lmargin[mt->lmargincur] = (size_t)ival;
-
-               return(0);
-       case (MAN_BODY):
+               return 0;
+       case ROFFT_BODY:
                p->offset = mt->offset + len;
                p->offset = mt->offset + len;
-               p->rmargin = p->maxrmargin > p->offset ?
-                               p->maxrmargin : p->offset;
+               p->rmargin = p->maxrmargin;
                p->trailspace = 0;
                p->trailspace = 0;
-               p->flags &= ~TERMP_NOBREAK;
+               p->flags &= ~(TERMP_NOBREAK | TERMP_BRTRSP);
                break;
        default:
                break;
        }
 
                break;
        default:
                break;
        }
 
-       return(1);
+       return 1;
 }
 
 }
 
-
-/* ARGSUSED */
 static void
 post_TP(DECL_ARGS)
 {
 
        switch (n->type) {
 static void
 post_TP(DECL_ARGS)
 {
 
        switch (n->type) {
-       case (MAN_HEAD):
+       case ROFFT_HEAD:
                term_flushln(p);
                break;
                term_flushln(p);
                break;
-       case (MAN_BODY):
+       case ROFFT_BODY:
                term_newln(p);
                p->offset = mt->offset;
                break;
                term_newln(p);
                p->offset = mt->offset;
                break;
@@ -797,52 +742,61 @@ post_TP(DECL_ARGS)
        }
 }
 
        }
 }
 
-
-/* ARGSUSED */
 static int
 pre_SS(DECL_ARGS)
 {
        int      i;
 
        switch (n->type) {
 static int
 pre_SS(DECL_ARGS)
 {
        int      i;
 
        switch (n->type) {
-       case (MAN_BLOCK):
+       case ROFFT_BLOCK:
                mt->fl &= ~MANT_LITERAL;
                mt->lmargin[mt->lmargincur] = term_len(p, p->defindent);
                mt->offset = term_len(p, p->defindent);
                mt->fl &= ~MANT_LITERAL;
                mt->lmargin[mt->lmargincur] = term_len(p, p->defindent);
                mt->offset = term_len(p, p->defindent);
-               /* If following a prior empty `SS', no vspace. */
-               if (n->prev && MAN_SS == n->prev->tok)
-                       if (NULL == n->prev->body->child)
-                               break;
-               if (NULL == n->prev)
+
+               /*
+                * No vertical space before the first subsection
+                * and after an empty subsection.
+                */
+
+               do {
+                       n = n->prev;
+               } while (n != NULL && n->tok != TOKEN_NONE &&
+                   termacts[n->tok].flags & MAN_NOTEXT);
+               if (n == NULL || (n->tok == MAN_SS && n->body->child == NULL))
                        break;
                        break;
+
                for (i = 0; i < mt->pardist; i++)
                        term_vspace(p);
                break;
                for (i = 0; i < mt->pardist; i++)
                        term_vspace(p);
                break;
-       case (MAN_HEAD):
+       case ROFFT_HEAD:
                term_fontrepl(p, TERMFONT_BOLD);
                p->offset = term_len(p, 3);
                term_fontrepl(p, TERMFONT_BOLD);
                p->offset = term_len(p, 3);
+               p->rmargin = mt->offset;
+               p->trailspace = mt->offset;
+               p->flags |= TERMP_NOBREAK | TERMP_BRIND;
                break;
                break;
-       case (MAN_BODY):
+       case ROFFT_BODY:
                p->offset = mt->offset;
                p->offset = mt->offset;
+               p->rmargin = p->maxrmargin;
+               p->trailspace = 0;
+               p->flags &= ~(TERMP_NOBREAK | TERMP_BRIND);
                break;
        default:
                break;
        }
 
                break;
        default:
                break;
        }
 
-       return(1);
+       return 1;
 }
 
 }
 
-
-/* ARGSUSED */
 static void
 post_SS(DECL_ARGS)
 {
 static void
 post_SS(DECL_ARGS)
 {
-       
+
        switch (n->type) {
        switch (n->type) {
-       case (MAN_HEAD):
+       case ROFFT_HEAD:
                term_newln(p);
                break;
                term_newln(p);
                break;
-       case (MAN_BODY):
+       case ROFFT_BODY:
                term_newln(p);
                break;
        default:
                term_newln(p);
                break;
        default:
@@ -850,53 +804,61 @@ post_SS(DECL_ARGS)
        }
 }
 
        }
 }
 
-
-/* ARGSUSED */
 static int
 pre_SH(DECL_ARGS)
 {
        int      i;
 
        switch (n->type) {
 static int
 pre_SH(DECL_ARGS)
 {
        int      i;
 
        switch (n->type) {
-       case (MAN_BLOCK):
+       case ROFFT_BLOCK:
                mt->fl &= ~MANT_LITERAL;
                mt->lmargin[mt->lmargincur] = term_len(p, p->defindent);
                mt->offset = term_len(p, p->defindent);
                mt->fl &= ~MANT_LITERAL;
                mt->lmargin[mt->lmargincur] = term_len(p, p->defindent);
                mt->offset = term_len(p, p->defindent);
-               /* If following a prior empty `SH', no vspace. */
-               if (n->prev && MAN_SH == n->prev->tok)
-                       if (NULL == n->prev->body->child)
-                               break;
-               /* If the first macro, no vspae. */
-               if (NULL == n->prev)
+
+               /*
+                * No vertical space before the first section
+                * and after an empty section.
+                */
+
+               do {
+                       n = n->prev;
+               } while (n != NULL && n->tok != TOKEN_NONE &&
+                   termacts[n->tok].flags & MAN_NOTEXT);
+               if (n == NULL || (n->tok == MAN_SH && n->body->child == NULL))
                        break;
                        break;
+
                for (i = 0; i < mt->pardist; i++)
                        term_vspace(p);
                break;
                for (i = 0; i < mt->pardist; i++)
                        term_vspace(p);
                break;
-       case (MAN_HEAD):
+       case ROFFT_HEAD:
                term_fontrepl(p, TERMFONT_BOLD);
                p->offset = 0;
                term_fontrepl(p, TERMFONT_BOLD);
                p->offset = 0;
+               p->rmargin = mt->offset;
+               p->trailspace = mt->offset;
+               p->flags |= TERMP_NOBREAK | TERMP_BRIND;
                break;
                break;
-       case (MAN_BODY):
+       case ROFFT_BODY:
                p->offset = mt->offset;
                p->offset = mt->offset;
+               p->rmargin = p->maxrmargin;
+               p->trailspace = 0;
+               p->flags &= ~(TERMP_NOBREAK | TERMP_BRIND);
                break;
        default:
                break;
        }
 
                break;
        default:
                break;
        }
 
-       return(1);
+       return 1;
 }
 
 }
 
-
-/* ARGSUSED */
 static void
 post_SH(DECL_ARGS)
 {
 static void
 post_SH(DECL_ARGS)
 {
-       
+
        switch (n->type) {
        switch (n->type) {
-       case (MAN_HEAD):
+       case ROFFT_HEAD:
                term_newln(p);
                break;
                term_newln(p);
                break;
-       case (MAN_BODY):
+       case ROFFT_BODY:
                term_newln(p);
                break;
        default:
                term_newln(p);
                break;
        default:
@@ -904,85 +866,76 @@ post_SH(DECL_ARGS)
        }
 }
 
        }
 }
 
-/* ARGSUSED */
 static int
 pre_RS(DECL_ARGS)
 {
 static int
 pre_RS(DECL_ARGS)
 {
-       int              ival;
-       size_t           sz;
+       struct roffsu    su;
 
        switch (n->type) {
 
        switch (n->type) {
-       case (MAN_BLOCK):
+       case ROFFT_BLOCK:
                term_newln(p);
                term_newln(p);
-               return(1);
-       case (MAN_HEAD):
-               return(0);
+               return 1;
+       case ROFFT_HEAD:
+               return 0;
        default:
                break;
        }
 
        default:
                break;
        }
 
-       sz = term_len(p, p->defindent);
-
-       if (NULL != (n = n->parent->head->child))
-               if ((ival = a2width(p, n->string)) >= 0) 
-                       sz = (size_t)ival;
-
-       mt->offset += sz;
+       n = n->parent->head;
+       n->aux = SHRT_MAX + 1;
+       if (n->child == NULL)
+               n->aux = mt->lmargin[mt->lmargincur];
+       else if (a2roffsu(n->child->string, &su, SCALE_EN))
+               n->aux = term_hspan(p, &su) / 24;
+       if (n->aux < 0 && (size_t)(-n->aux) > mt->offset)
+               n->aux = -mt->offset;
+       else if (n->aux > SHRT_MAX)
+               n->aux = term_len(p, p->defindent);
+
+       mt->offset += n->aux;
        p->offset = mt->offset;
        p->offset = mt->offset;
-       p->rmargin = p->maxrmargin > p->offset ?
-                       p->maxrmargin : p->offset;
+       p->rmargin = p->maxrmargin;
 
        if (++mt->lmarginsz < MAXMARGINS)
                mt->lmargincur = mt->lmarginsz;
 
 
        if (++mt->lmarginsz < MAXMARGINS)
                mt->lmargincur = mt->lmarginsz;
 
-       mt->lmargin[mt->lmargincur] = mt->lmargin[mt->lmargincur - 1];
-       return(1);
+       mt->lmargin[mt->lmargincur] = term_len(p, p->defindent);
+       return 1;
 }
 
 }
 
-/* ARGSUSED */
 static void
 post_RS(DECL_ARGS)
 {
 static void
 post_RS(DECL_ARGS)
 {
-       int              ival;
-       size_t           sz;
 
        switch (n->type) {
 
        switch (n->type) {
-       case (MAN_BLOCK):
+       case ROFFT_BLOCK:
                return;
                return;
-       case (MAN_HEAD):
+       case ROFFT_HEAD:
                return;
        default:
                term_newln(p);
                break;
        }
 
                return;
        default:
                term_newln(p);
                break;
        }
 
-       sz = term_len(p, p->defindent);
-
-       if (NULL != (n = n->parent->head->child)) 
-               if ((ival = a2width(p, n->string)) >= 0) 
-                       sz = (size_t)ival;
-
-       mt->offset = mt->offset < sz ?  0 : mt->offset - sz;
+       mt->offset -= n->parent->head->aux;
        p->offset = mt->offset;
 
        if (--mt->lmarginsz < MAXMARGINS)
                mt->lmargincur = mt->lmarginsz;
 }
 
        p->offset = mt->offset;
 
        if (--mt->lmarginsz < MAXMARGINS)
                mt->lmargincur = mt->lmarginsz;
 }
 
-/* ARGSUSED */
 static int
 pre_UR(DECL_ARGS)
 {
 
 static int
 pre_UR(DECL_ARGS)
 {
 
-       return (MAN_HEAD != n->type);
+       return n->type != ROFFT_HEAD;
 }
 
 }
 
-/* ARGSUSED */
 static void
 post_UR(DECL_ARGS)
 {
 
 static void
 post_UR(DECL_ARGS)
 {
 
-       if (MAN_BLOCK != n->type)
+       if (n->type != ROFFT_BLOCK)
                return;
 
        term_word(p, "<");
                return;
 
        term_word(p, "<");
@@ -1002,7 +955,7 @@ print_man_node(DECL_ARGS)
        int              c;
 
        switch (n->type) {
        int              c;
 
        switch (n->type) {
-       case(MAN_TEXT):
+       case ROFFT_TEXT:
                /*
                 * If we have a blank line, output a vertical space.
                 * If we have a space as the first character, break
                /*
                 * If we have a blank line, output a vertical space.
                 * If we have a space as the first character, break
@@ -1011,22 +964,22 @@ print_man_node(DECL_ARGS)
                if ('\0' == *n->string) {
                        term_vspace(p);
                        return;
                if ('\0' == *n->string) {
                        term_vspace(p);
                        return;
-               } else if (' ' == *n->string && MAN_LINE & n->flags)
+               } else if (' ' == *n->string && NODE_LINE & n->flags)
                        term_newln(p);
 
                term_word(p, n->string);
                goto out;
 
                        term_newln(p);
 
                term_word(p, n->string);
                goto out;
 
-       case (MAN_EQN):
+       case ROFFT_EQN:
+               if ( ! (n->flags & NODE_LINE))
+                       p->flags |= TERMP_NOSPACE;
                term_eqn(p, n->eqn);
                term_eqn(p, n->eqn);
+               if (n->next != NULL && ! (n->next->flags & NODE_LINE))
+                       p->flags |= TERMP_NOSPACE;
                return;
                return;
-       case (MAN_TBL):
-               /*
-                * Tables are preceded by a newline.  Then process a
-                * table line, which will cause line termination,
-                */
-               if (TBL_SPAN_FIRST & n->span->flags) 
-                       term_newln(p);
+       case ROFFT_TBL:
+               if (p->tbl.cols == NULL)
+                       term_vspace(p);
                term_tbl(p, n->span);
                return;
        default:
                term_tbl(p, n->span);
                return;
        default:
@@ -1056,13 +1009,14 @@ out:
         * -man doesn't have nested macros, we don't need to be
         * more specific than this.
         */
         * -man doesn't have nested macros, we don't need to be
         * more specific than this.
         */
-       if (MANT_LITERAL & mt->fl && ! (TERMP_NOBREAK & p->flags) &&
-           (NULL == n->next || MAN_LINE & n->next->flags)) {
+       if (mt->fl & MANT_LITERAL &&
+           ! (p->flags & (TERMP_NOBREAK | TERMP_NONEWLINE)) &&
+           (n->next == NULL || n->next->flags & NODE_LINE)) {
                rm = p->rmargin;
                rmax = p->maxrmargin;
                p->rmargin = p->maxrmargin = TERM_MAXMARGIN;
                p->flags |= TERMP_NOSPACE;
                rm = p->rmargin;
                rmax = p->maxrmargin;
                p->rmargin = p->maxrmargin = TERM_MAXMARGIN;
                p->flags |= TERMP_NOSPACE;
-               if (NULL != n->string && '\0' != *n->string)
+               if (n->string != NULL && *n->string != '\0')
                        term_flushln(p);
                else
                        term_newln(p);
                        term_flushln(p);
                else
                        term_newln(p);
@@ -1073,7 +1027,7 @@ out:
                        p->rmargin = rm;
                p->maxrmargin = rmax;
        }
                        p->rmargin = rm;
                p->maxrmargin = rmax;
        }
-       if (MAN_EOS & n->flags)
+       if (NODE_EOS & n->flags)
                p->flags |= TERMP_SENTENCE;
 }
 
                p->flags |= TERMP_SENTENCE;
 }
 
@@ -1082,64 +1036,65 @@ static void
 print_man_nodelist(DECL_ARGS)
 {
 
 print_man_nodelist(DECL_ARGS)
 {
 
-       print_man_node(p, mt, n, meta);
-       if ( ! n->next)
-               return;
-       print_man_nodelist(p, mt, n->next, meta);
+       while (n != NULL) {
+               print_man_node(p, mt, n, meta);
+               n = n->next;
+       }
 }
 
 }
 
-
 static void
 static void
-print_man_foot(struct termp *p, const void *arg)
+print_man_foot(struct termp *p, const struct roff_meta *meta)
 {
 {
-       char            title[BUFSIZ];
-       size_t          datelen;
-       const struct man_meta *meta;
+       char                    *title;
+       size_t                   datelen, titlen;
 
 
-       meta = (const struct man_meta *)arg;
        assert(meta->title);
        assert(meta->msec);
        assert(meta->date);
 
        term_fontrepl(p, TERMFONT_NONE);
 
        assert(meta->title);
        assert(meta->msec);
        assert(meta->date);
 
        term_fontrepl(p, TERMFONT_NONE);
 
-       term_vspace(p);
+       if (meta->hasbody)
+               term_vspace(p);
 
        /*
         * Temporary, undocumented option to imitate mdoc(7) output.
 
        /*
         * Temporary, undocumented option to imitate mdoc(7) output.
-        * In the bottom right corner, use the source instead of
-        * the title.
+        * In the bottom right corner, use the operating system
+        * instead of the title.
         */
 
        if ( ! p->mdocstyle) {
         */
 
        if ( ! p->mdocstyle) {
-               term_vspace(p);
-               term_vspace(p);
-               snprintf(title, BUFSIZ, "%s(%s)", meta->title, meta->msec);
-       } else if (meta->source) {
-               strlcpy(title, meta->source, BUFSIZ);
+               if (meta->hasbody) {
+                       term_vspace(p);
+                       term_vspace(p);
+               }
+               mandoc_asprintf(&title, "%s(%s)",
+                   meta->title, meta->msec);
+       } else if (meta->os) {
+               title = mandoc_strdup(meta->os);
        } else {
        } else {
-               title[0] = '\0';
+               title = mandoc_strdup("");
        }
        datelen = term_strlen(p, meta->date);
 
        }
        datelen = term_strlen(p, meta->date);
 
-       /* Bottom left corner: manual source. */
+       /* Bottom left corner: operating system. */
 
        p->flags |= TERMP_NOSPACE | TERMP_NOBREAK;
        p->trailspace = 1;
        p->offset = 0;
 
        p->flags |= TERMP_NOSPACE | TERMP_NOBREAK;
        p->trailspace = 1;
        p->offset = 0;
-       p->rmargin = (p->maxrmargin - datelen + term_len(p, 1)) / 2;
+       p->rmargin = p->maxrmargin > datelen ?
+           (p->maxrmargin + term_len(p, 1) - datelen) / 2 : 0;
 
 
-       if (meta->source)
-               term_word(p, meta->source);
+       if (meta->os)
+               term_word(p, meta->os);
        term_flushln(p);
 
        /* At the bottom in the middle: manual date. */
 
        term_flushln(p);
 
        /* At the bottom in the middle: manual date. */
 
-       p->flags |= TERMP_NOSPACE;
        p->offset = p->rmargin;
        p->offset = p->rmargin;
-       p->rmargin = p->maxrmargin - term_strlen(p, title);
-       if (p->offset + datelen >= p->rmargin)
-               p->rmargin = p->offset + datelen;
+       titlen = term_strlen(p, title);
+       p->rmargin = p->maxrmargin > titlen ? p->maxrmargin - titlen : 0;
+       p->flags |= TERMP_NOSPACE;
 
        term_word(p, meta->date);
        term_flushln(p);
 
        term_word(p, meta->date);
        term_flushln(p);
@@ -1154,38 +1109,33 @@ print_man_foot(struct termp *p, const void *arg)
 
        term_word(p, title);
        term_flushln(p);
 
        term_word(p, title);
        term_flushln(p);
+       free(title);
 }
 
 }
 
-
 static void
 static void
-print_man_head(struct termp *p, const void *arg)
+print_man_head(struct termp *p, const struct roff_meta *meta)
 {
 {
-       char            buf[BUFSIZ], title[BUFSIZ];
-       size_t          buflen, titlen;
-       const struct man_meta *meta;
+       const char              *volume;
+       char                    *title;
+       size_t                   vollen, titlen;
 
 
-       meta = (const struct man_meta *)arg;
        assert(meta->title);
        assert(meta->msec);
 
        assert(meta->title);
        assert(meta->msec);
 
-       if (meta->vol)
-               strlcpy(buf, meta->vol, BUFSIZ);
-       else
-               buf[0] = '\0';
-       buflen = term_strlen(p, buf);
+       volume = NULL == meta->vol ? "" : meta->vol;
+       vollen = term_strlen(p, volume);
 
        /* Top left corner: manual title and section. */
 
 
        /* Top left corner: manual title and section. */
 
-       snprintf(title, BUFSIZ, "%s(%s)", meta->title, meta->msec);
+       mandoc_asprintf(&title, "%s(%s)", meta->title, meta->msec);
        titlen = term_strlen(p, title);
 
        p->flags |= TERMP_NOBREAK | TERMP_NOSPACE;
        p->trailspace = 1;
        p->offset = 0;
        titlen = term_strlen(p, title);
 
        p->flags |= TERMP_NOBREAK | TERMP_NOSPACE;
        p->trailspace = 1;
        p->offset = 0;
-       p->rmargin = 2 * (titlen+1) + buflen < p->maxrmargin ?
-           (p->maxrmargin - 
-            term_strlen(p, buf) + term_len(p, 1)) / 2 :
-           p->maxrmargin - buflen;
+       p->rmargin = 2 * (titlen+1) + vollen < p->maxrmargin ?
+           (p->maxrmargin - vollen + term_len(p, 1)) / 2 :
+           vollen < p->maxrmargin ? p->maxrmargin - vollen : 0;
 
        term_word(p, title);
        term_flushln(p);
 
        term_word(p, title);
        term_flushln(p);
@@ -1194,10 +1144,10 @@ print_man_head(struct termp *p, const void *arg)
 
        p->flags |= TERMP_NOSPACE;
        p->offset = p->rmargin;
 
        p->flags |= TERMP_NOSPACE;
        p->offset = p->rmargin;
-       p->rmargin = p->offset + buflen + titlen < p->maxrmargin ?
+       p->rmargin = p->offset + vollen + titlen < p->maxrmargin ?
            p->maxrmargin - titlen : p->maxrmargin;
 
            p->maxrmargin - titlen : p->maxrmargin;
 
-       term_word(p, buf);
+       term_word(p, volume);
        term_flushln(p);
 
        /* Top right corner: title and section, again. */
        term_flushln(p);
 
        /* Top right corner: title and section, again. */
@@ -1216,7 +1166,7 @@ print_man_head(struct termp *p, const void *arg)
        p->offset = 0;
        p->rmargin = p->maxrmargin;
 
        p->offset = 0;
        p->rmargin = p->maxrmargin;
 
-       /* 
+       /*
         * Groff prints three blank lines before the content.
         * Do the same, except in the temporary, undocumented
         * mode imitating mdoc(7) output.
         * Groff prints three blank lines before the content.
         * Do the same, except in the temporary, undocumented
         * mode imitating mdoc(7) output.
@@ -1227,4 +1177,5 @@ print_man_head(struct termp *p, const void *arg)
                term_vspace(p);
                term_vspace(p);
        }
                term_vspace(p);
                term_vspace(p);
        }
+       free(title);
 }
 }
A git conversion mirror of mandoc.bsd.lv