]> git.cameronkatri.com Git - mandoc.git/blobdiff - man_term.c
Fill mode changes don't break next-line scope in all cases,
[mandoc.git] / man_term.c
index a710a1e61ef16236cc3a883f2ca0d13745e8f290..a79e5e27b3c7e07f30e8c0e5f56afaa0ac7be22d 100644 (file)
@@ -1,15 +1,15 @@
-/*     $Id: man_term.c,v 1.163 2014/12/23 13:48:57 schwarze Exp $ */
+/*     $Id: man_term.c,v 1.183 2015/04/19 19:44:21 schwarze Exp $ */
 /*
  * Copyright (c) 2008-2012 Kristaps Dzonsons <kristaps@bsd.lv>
- * Copyright (c) 2010-2014 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2010-2015 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.
  *
- * 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
- * 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
 
 #include <assert.h>
 #include <ctype.h>
+#include <limits.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 
-#include "mandoc.h"
 #include "mandoc_aux.h"
-#include "out.h"
+#include "mandoc.h"
+#include "roff.h"
 #include "man.h"
+#include "out.h"
 #include "term.h"
 #include "main.h"
 
@@ -46,8 +48,8 @@ struct        mtermp {
 
 #define        DECL_ARGS         struct termp *p, \
                          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);
@@ -58,10 +60,12 @@ struct      termact {
 
 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_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 man_node *, int);
+                               const struct roff_node *, int);
 
 static int               pre_B(DECL_ARGS);
 static int               pre_HP(DECL_ARGS);
@@ -113,7 +117,6 @@ static      const struct termact termacts[MAN_MAX] = {
        { 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 */
@@ -135,36 +138,32 @@ static    const struct termact termacts[MAN_MAX] = {
 
 
 void
-terminal_man(void *arg, const struct man *man)
+terminal_man(void *arg, const struct roff_man *man)
 {
        struct termp            *p;
-       const struct man_meta   *meta;
-       struct man_node         *n;
+       struct roff_node        *n;
        struct mtermp            mt;
 
        p = (struct termp *)arg;
-
        p->overstep = 0;
        p->rmargin = p->maxrmargin = p->defrmargin;
        p->tabwidth = term_len(p, 5);
 
-       n = man_node(man)->child;
-       meta = man_meta(man);
-
        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;
 
+       n = man->first->child;
        if (p->synopsisonly) {
                while (n != NULL) {
                        if (n->tok == MAN_SH &&
-                           n->child->child->type == MAN_TEXT &&
+                           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, meta);
+                                           n->child->next->child,
+                                           &man->meta);
                                term_newln(p);
                                break;
                        }
@@ -173,10 +172,10 @@ terminal_man(void *arg, const struct man *man)
        } else {
                if (p->defindent == 0)
                        p->defindent = 7;
-               term_begin(p, print_man_head, print_man_foot, meta);
+               term_begin(p, print_man_head, print_man_foot, &man->meta);
                p->flags |= TERMP_NOSPACE;
                if (n != NULL)
-                       print_man_nodelist(p, &mt, n, meta);
+                       print_man_nodelist(p, &mt, n, &man->meta);
                term_end(p);
        }
 }
@@ -190,17 +189,17 @@ terminal_man(void *arg, const struct man *man)
  * 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)
-               if (MAN_TBL == n->body->child->type)
+               if (n->body->child->type == ROFFT_TBL)
                        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;
 
@@ -269,7 +268,7 @@ pre_PD(DECL_ARGS)
                mt->pardist = 1;
                return(0);
        }
-       assert(MAN_TEXT == n->type);
+       assert(n->type == ROFFT_TEXT);
        if (a2roffsu(n->string, &su, SCALE_VS))
                mt->pardist = term_vspan(p, &su);
        return(0);
@@ -279,7 +278,7 @@ 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) {
@@ -318,7 +317,10 @@ pre_alternate(DECL_ARGS)
                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 & MAN_EOS)
+                       p->flags |= TERMP_SENTENCE;
                if (nn->next)
                        p->flags |= TERMP_NOSPACE;
        }
@@ -424,7 +426,7 @@ pre_in(DECL_ARGS)
        if ( ! a2roffsu(++cp, &su, SCALE_EN))
                return(0);
 
-       v = term_hspan(p, &su);
+       v = (term_hspan(p, &su) + 11) / 24;
 
        if (less < 0)
                p->offset -= p->offset > v ? v : p->offset;
@@ -432,6 +434,8 @@ pre_in(DECL_ARGS)
                p->offset += v;
        else
                p->offset = v;
+       if (p->offset > SHRT_MAX)
+               p->offset = term_len(p, p->defindent);
 
        return(0);
 }
@@ -478,6 +482,17 @@ pre_sp(DECL_ARGS)
                for (i = 0; i < len; i++)
                        term_vspace(p);
 
+       /*
+        * 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);
 }
 
@@ -485,14 +500,14 @@ static int
 pre_HP(DECL_ARGS)
 {
        struct roffsu            su;
-       const struct man_node   *nn;
+       const struct roff_node  *nn;
        int                      len;
 
        switch (n->type) {
-       case MAN_BLOCK:
+       case ROFFT_BLOCK:
                print_bvspace(p, n, mt->pardist);
                return(1);
-       case MAN_BODY:
+       case ROFFT_BODY:
                break;
        default:
                return(0);
@@ -507,17 +522,17 @@ pre_HP(DECL_ARGS)
 
        if ((nn = n->parent->head->child) != NULL &&
            a2roffsu(nn->string, &su, SCALE_EN)) {
-               len = term_hspan(p, &su);
+               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;
-       if (len > 0 || (size_t)(-len) < mt->offset)
-               p->rmargin = mt->offset + len;
-       else
-               p->rmargin = 0;
-
+       p->rmargin = mt->offset + len;
        return(1);
 }
 
@@ -526,8 +541,19 @@ post_HP(DECL_ARGS)
 {
 
        switch (n->type) {
-       case MAN_BODY:
+       case ROFFT_BODY:
                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;
@@ -543,7 +569,7 @@ 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;
@@ -552,25 +578,25 @@ pre_PP(DECL_ARGS)
                break;
        }
 
-       return(MAN_HEAD != n->type);
+       return(n->type != ROFFT_HEAD);
 }
 
 static int
 pre_IP(DECL_ARGS)
 {
        struct roffsu            su;
-       const struct man_node   *nn;
+       const struct roff_node  *nn;
        int                      len, savelit;
 
        switch (n->type) {
-       case MAN_BODY:
+       case ROFFT_BODY:
                p->flags |= TERMP_NOSPACE;
                break;
-       case MAN_HEAD:
+       case ROFFT_HEAD:
                p->flags |= TERMP_NOBREAK;
                p->trailspace = 1;
                break;
-       case MAN_BLOCK:
+       case ROFFT_BLOCK:
                print_bvspace(p, n, mt->pardist);
                /* FALLTHROUGH */
        default:
@@ -581,15 +607,17 @@ pre_IP(DECL_ARGS)
        if ((nn = n->parent->head->child) != NULL &&
            (nn = nn->next) != NULL &&
            a2roffsu(nn->string, &su, SCALE_EN)) {
-               len = term_hspan(p, &su);
-               mt->lmargin[mt->lmargincur] = len;
+               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) {
-       case MAN_HEAD:
+       case ROFFT_HEAD:
                p->offset = mt->offset;
                p->rmargin = mt->offset + len;
 
@@ -603,7 +631,7 @@ pre_IP(DECL_ARGS)
                        mt->fl |= MANT_LITERAL;
 
                return(0);
-       case MAN_BODY:
+       case ROFFT_BODY:
                p->offset = mt->offset + len;
                p->rmargin = p->maxrmargin;
                break;
@@ -619,13 +647,13 @@ 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;
-       case MAN_BODY:
+       case ROFFT_BODY:
                term_newln(p);
                p->offset = mt->offset;
                break;
@@ -638,18 +666,18 @@ static int
 pre_TP(DECL_ARGS)
 {
        struct roffsu            su;
-       const struct man_node   *nn;
+       struct roff_node        *nn;
        int                      len, savelit;
 
        switch (n->type) {
-       case MAN_HEAD:
+       case ROFFT_HEAD:
                p->flags |= TERMP_NOBREAK;
                p->trailspace = 1;
                break;
-       case MAN_BODY:
+       case ROFFT_BODY:
                p->flags |= TERMP_NOSPACE;
                break;
-       case MAN_BLOCK:
+       case ROFFT_BLOCK:
                print_bvspace(p, n, mt->pardist);
                /* FALLTHROUGH */
        default:
@@ -661,15 +689,17 @@ pre_TP(DECL_ARGS)
        if ((nn = n->parent->head->child) != NULL &&
            nn->string != NULL && ! (MAN_LINE & nn->flags) &&
            a2roffsu(nn->string, &su, SCALE_EN)) {
-               len = term_hspan(p, &su);
-               mt->lmargin[mt->lmargincur] = len;
+               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) {
-       case MAN_HEAD:
+       case ROFFT_HEAD:
                p->offset = mt->offset;
                p->rmargin = mt->offset + len;
 
@@ -689,7 +719,7 @@ pre_TP(DECL_ARGS)
                if (savelit)
                        mt->fl |= MANT_LITERAL;
                return(0);
-       case MAN_BODY:
+       case ROFFT_BODY:
                p->offset = mt->offset + len;
                p->rmargin = p->maxrmargin;
                p->trailspace = 0;
@@ -707,10 +737,10 @@ post_TP(DECL_ARGS)
 {
 
        switch (n->type) {
-       case MAN_HEAD:
+       case ROFFT_HEAD:
                term_flushln(p);
                break;
-       case MAN_BODY:
+       case ROFFT_BODY:
                term_newln(p);
                p->offset = mt->offset;
                break;
@@ -725,7 +755,7 @@ 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);
@@ -737,19 +767,26 @@ pre_SS(DECL_ARGS)
 
                do {
                        n = n->prev;
-               } while (n != NULL && termacts[n->tok].flags & MAN_NOTEXT);
+               } 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;
 
                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);
+               p->rmargin = mt->offset;
+               p->trailspace = mt->offset;
+               p->flags |= TERMP_NOBREAK | TERMP_BRIND;
                break;
-       case MAN_BODY:
+       case ROFFT_BODY:
                p->offset = mt->offset;
+               p->rmargin = p->maxrmargin;
+               p->trailspace = 0;
+               p->flags &= ~(TERMP_NOBREAK | TERMP_BRIND);
                break;
        default:
                break;
@@ -763,10 +800,10 @@ post_SS(DECL_ARGS)
 {
 
        switch (n->type) {
-       case MAN_HEAD:
+       case ROFFT_HEAD:
                term_newln(p);
                break;
-       case MAN_BODY:
+       case ROFFT_BODY:
                term_newln(p);
                break;
        default:
@@ -780,7 +817,7 @@ 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);
@@ -799,12 +836,18 @@ pre_SH(DECL_ARGS)
                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;
+               p->rmargin = mt->offset;
+               p->trailspace = mt->offset;
+               p->flags |= TERMP_NOBREAK | TERMP_BRIND;
                break;
-       case MAN_BODY:
+       case ROFFT_BODY:
                p->offset = mt->offset;
+               p->rmargin = p->maxrmargin;
+               p->trailspace = 0;
+               p->flags &= ~(TERMP_NOBREAK | TERMP_BRIND);
                break;
        default:
                break;
@@ -818,10 +861,10 @@ post_SH(DECL_ARGS)
 {
 
        switch (n->type) {
-       case MAN_HEAD:
+       case ROFFT_HEAD:
                term_newln(p);
                break;
-       case MAN_BODY:
+       case ROFFT_BODY:
                term_newln(p);
                break;
        default:
@@ -833,64 +876,54 @@ static int
 pre_RS(DECL_ARGS)
 {
        struct roffsu    su;
-       int              len;
 
        switch (n->type) {
-       case MAN_BLOCK:
+       case ROFFT_BLOCK:
                term_newln(p);
                return(1);
-       case MAN_HEAD:
+       case ROFFT_HEAD:
                return(0);
        default:
                break;
        }
 
-       if ((n = n->parent->head->child) != NULL &&
-           a2roffsu(n->string, &su, SCALE_EN))
-               len = term_hspan(p, &su);
-       else
-               len = term_len(p, p->defindent);
-
-       if (len > 0 || (size_t)(-len) < mt->offset)
-               mt->offset += len;
-       else
-               mt->offset = 0;
+       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->rmargin = p->maxrmargin;
 
        if (++mt->lmarginsz < MAXMARGINS)
                mt->lmargincur = mt->lmarginsz;
 
-       mt->lmargin[mt->lmargincur] = mt->lmargin[mt->lmargincur - 1];
+       mt->lmargin[mt->lmargincur] = term_len(p, p->defindent);
        return(1);
 }
 
 static void
 post_RS(DECL_ARGS)
 {
-       struct roffsu    su;
-       int              len;
 
        switch (n->type) {
-       case MAN_BLOCK:
+       case ROFFT_BLOCK:
                return;
-       case MAN_HEAD:
+       case ROFFT_HEAD:
                return;
        default:
                term_newln(p);
                break;
        }
 
-       if ((n = n->parent->head->child) != NULL &&
-           a2roffsu(n->string, &su, SCALE_EN))
-               len = term_hspan(p, &su);
-       else
-               len = term_len(p, p->defindent);
-
-       if (len < 0 || (size_t)len < mt->offset)
-               mt->offset -= len;
-       else
-               mt->offset = 0;
+       mt->offset -= n->parent->head->aux;
        p->offset = mt->offset;
 
        if (--mt->lmarginsz < MAXMARGINS)
@@ -901,14 +934,14 @@ static int
 pre_UR(DECL_ARGS)
 {
 
-       return (MAN_HEAD != n->type);
+       return (n->type != ROFFT_HEAD);
 }
 
 static void
 post_UR(DECL_ARGS)
 {
 
-       if (MAN_BLOCK != n->type)
+       if (n->type != ROFFT_BLOCK)
                return;
 
        term_word(p, "<");
@@ -928,7 +961,7 @@ print_man_node(DECL_ARGS)
        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
@@ -943,20 +976,16 @@ print_man_node(DECL_ARGS)
                term_word(p, n->string);
                goto out;
 
-       case MAN_EQN:
+       case ROFFT_EQN:
                if ( ! (n->flags & MAN_LINE))
                        p->flags |= TERMP_NOSPACE;
                term_eqn(p, n->eqn);
                if (n->next != NULL && ! (n->next->flags & MAN_LINE))
                        p->flags |= TERMP_NOSPACE;
                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:
@@ -1013,20 +1042,18 @@ static void
 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
-print_man_foot(struct termp *p, const void *arg)
+print_man_foot(struct termp *p, const struct roff_meta *meta)
 {
-       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);
@@ -1038,8 +1065,8 @@ print_man_foot(struct termp *p, const void *arg)
 
        /*
         * 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) {
@@ -1049,14 +1076,14 @@ print_man_foot(struct termp *p, const void *arg)
                }
                mandoc_asprintf(&title, "%s(%s)",
                    meta->title, meta->msec);
-       } else if (meta->source) {
-               title = mandoc_strdup(meta->source);
+       } else if (meta->os) {
+               title = mandoc_strdup(meta->os);
        } else {
                title = mandoc_strdup("");
        }
        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;
@@ -1064,8 +1091,8 @@ print_man_foot(struct termp *p, const void *arg)
        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. */
@@ -1092,14 +1119,12 @@ print_man_foot(struct termp *p, const void *arg)
 }
 
 static void
-print_man_head(struct termp *p, const void *arg)
+print_man_head(struct termp *p, const struct roff_meta *meta)
 {
-       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);