]> git.cameronkatri.com Git - mandoc.git/blobdiff - tbl_term.c
Place mandoc.css into the public domain.
[mandoc.git] / tbl_term.c
index 52c41a21dd56d2e53bfb3ee6141ddd3ba1d58993..a552c5e84dd7896737994cd6331de49e1f7d56e6 100644 (file)
@@ -1,7 +1,7 @@
-/*     $Id: tbl_term.c,v 1.46 2017/06/08 18:11:22 schwarze Exp $ */
+/*     $Id: tbl_term.c,v 1.61 2018/11/25 19:24:20 schwarze Exp $ */
 /*
  * Copyright (c) 2009, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
- * Copyright (c) 2011,2012,2014,2015,2017 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2011-2018 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
@@ -20,6 +20,7 @@
 #include <sys/types.h>
 
 #include <assert.h>
+#include <ctype.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include "out.h"
 #include "term.h"
 
+#define        IS_HORIZ(cp)    ((cp)->pos == TBL_CELL_HORIZ || \
+                        (cp)->pos == TBL_CELL_DHORIZ)
+
 static size_t  term_tbl_len(size_t, void *);
 static size_t  term_tbl_strlen(const char *, void *);
 static size_t  term_tbl_sulen(const struct roffsu *, void *);
 static void    tbl_char(struct termp *, char, size_t);
 static void    tbl_data(struct termp *, const struct tbl_opts *,
+                       const struct tbl_cell *,
                        const struct tbl_dat *,
                        const struct roffcol *);
 static void    tbl_literal(struct termp *, const struct tbl_dat *,
@@ -47,7 +52,10 @@ static       void    tbl_word(struct termp *, const struct tbl_dat *);
 static size_t
 term_tbl_sulen(const struct roffsu *su, void *arg)
 {
-       return term_hspan((const struct termp *)arg, su) / 24;
+       int      i;
+
+       i = term_hen((const struct termp *)arg, su);
+       return i > 0 ? i : 0;
 }
 
 static size_t
@@ -65,15 +73,16 @@ term_tbl_len(size_t sz, void *arg)
 void
 term_tbl(struct termp *tp, const struct tbl_span *sp)
 {
-       const struct tbl_cell   *cp;
+       const struct tbl_cell   *cp, *cpn, *cpp;
        const struct tbl_dat    *dp;
        static size_t            offset;
-       size_t                   tsz;
-       int                      ic, horiz, spans, vert;
+       size_t                   coloff, tsz;
+       int                      ic, horiz, hspans, vert, more;
+       char                     fc;
 
        /* Inhibit printing of spaces: we do padding ourselves. */
 
-       tp->flags |= TERMP_NOSPACE | TERMP_NONOSPACE | TERMP_BRNEVER;
+       tp->flags |= TERMP_NOSPACE | TERMP_NONOSPACE;
 
        /*
         * The first time we're invoked for a given table block,
@@ -86,7 +95,18 @@ term_tbl(struct termp *tp, const struct tbl_span *sp)
                tp->tbl.sulen = term_tbl_sulen;
                tp->tbl.arg = tp;
 
-               tblcalc(&tp->tbl, sp, tp->tcol->rmargin - tp->tcol->offset);
+               tblcalc(&tp->tbl, sp, tp->tcol->offset, tp->tcol->rmargin);
+
+               /* Tables leak .ta settings to subsequent text. */
+
+               term_tab_set(tp, NULL);
+               coloff = sp->opts->opts & (TBL_OPT_BOX | TBL_OPT_DBOX) ||
+                   sp->opts->lvert;
+               for (ic = 0; ic < sp->opts->cols; ic++) {
+                       coloff += tp->tbl.cols[ic].width;
+                       term_tab_iset(coloff);
+                       coloff += tp->tbl.cols[ic].spacing;
+               }
 
                /* Center the table as a whole. */
 
@@ -94,9 +114,11 @@ term_tbl(struct termp *tp, const struct tbl_span *sp)
                if (sp->opts->opts & TBL_OPT_CENTRE) {
                        tsz = sp->opts->opts & (TBL_OPT_BOX | TBL_OPT_DBOX)
                            ? 2 : !!sp->opts->lvert + !!sp->opts->rvert;
-                       for (ic = 0; ic < sp->opts->cols; ic++)
-                               tsz += tp->tbl.cols[ic].width + 3;
-                       tsz -= 3;
+                       for (ic = 0; ic + 1 < sp->opts->cols; ic++)
+                               tsz += tp->tbl.cols[ic].width +
+                                   tp->tbl.cols[ic].spacing;
+                       if (sp->opts->cols)
+                               tsz += tp->tbl.cols[sp->opts->cols - 1].width;
                        if (offset + tsz > tp->tcol->rmargin)
                                tsz -= 1;
                        tp->tcol->offset = offset + tp->tcol->rmargin > tsz ?
@@ -106,161 +128,419 @@ term_tbl(struct termp *tp, const struct tbl_span *sp)
                /* Horizontal frame at the start of boxed tables. */
 
                if (sp->opts->opts & TBL_OPT_DBOX)
-                       tbl_hrule(tp, sp, 2);
+                       tbl_hrule(tp, sp, 3);
                if (sp->opts->opts & (TBL_OPT_DBOX | TBL_OPT_BOX))
-                       tbl_hrule(tp, sp, 1);
+                       tbl_hrule(tp, sp, 2);
        }
 
-       /* Vertical frame at the start of each row. */
+       /* Set up the columns. */
 
-       horiz = sp->pos == TBL_SPAN_HORIZ || sp->pos == TBL_SPAN_DHORIZ;
+       tp->flags |= TERMP_MULTICOL;
+       horiz = 0;
+       switch (sp->pos) {
+       case TBL_SPAN_HORIZ:
+       case TBL_SPAN_DHORIZ:
+               horiz = 1;
+               term_setcol(tp, 1);
+               break;
+       case TBL_SPAN_DATA:
+               term_setcol(tp, sp->opts->cols + 2);
+               coloff = tp->tcol->offset;
 
-       if (sp->layout->vert ||
-           (sp->prev != NULL && sp->prev->layout->vert) ||
-           sp->opts->opts & (TBL_OPT_BOX | TBL_OPT_DBOX))
-               term_word(tp, horiz ? "+" : "|");
-       else if (sp->opts->lvert)
-               tbl_char(tp, horiz ? '-' : ASCII_NBRSP, 1);
+               /* Set up a column for a left vertical frame. */
 
-       /*
-        * Now print the actual data itself depending on the span type.
-        * Match data cells to column numbers.
-        */
+               if (sp->opts->opts & (TBL_OPT_BOX | TBL_OPT_DBOX) ||
+                   sp->opts->lvert)
+                       coloff++;
+               tp->tcol->rmargin = coloff;
+
+               /* Set up the data columns. */
 
-       if (sp->pos == TBL_SPAN_DATA) {
-               cp = sp->layout->first;
                dp = sp->first;
-               spans = 0;
+               hspans = 0;
                for (ic = 0; ic < sp->opts->cols; ic++) {
+                       if (hspans == 0) {
+                               tp->tcol++;
+                               tp->tcol->offset = coloff;
+                       }
+                       coloff += tp->tbl.cols[ic].width;
+                       tp->tcol->rmargin = coloff;
+                       if (ic + 1 < sp->opts->cols)
+                               coloff += tp->tbl.cols[ic].spacing;
+                       if (hspans) {
+                               hspans--;
+                               continue;
+                       }
+                       if (dp == NULL)
+                               continue;
+                       hspans = dp->hspans;
+                       if (ic || sp->layout->first->pos != TBL_CELL_SPAN)
+                               dp = dp->next;
+               }
 
-                       /*
-                        * Remeber whether we need a vertical bar
-                        * after this cell.
-                        */
+               /* Set up a column for a right vertical frame. */
 
-                       vert = cp == NULL ? 0 : cp->vert;
+               tp->tcol++;
+               tp->tcol->offset = coloff + 1;
+               tp->tcol->rmargin = tp->maxrmargin;
 
-                       /*
-                        * Print the data and advance to the next cell.
-                        */
+               /* Spans may have reduced the number of columns. */
 
-                       if (spans == 0) {
-                               tbl_data(tp, sp->opts, dp, tp->tbl.cols + ic);
-                               if (dp != NULL) {
-                                       spans = dp->spans;
-                                       dp = dp->next;
-                               }
-                       } else
-                               spans--;
-                       if (cp != NULL)
-                               cp = cp->next;
+               tp->lasttcol = tp->tcol - tp->tcols;
 
-                       /*
-                        * Separate columns, except in the middle
-                        * of spans and after the last cell.
-                        */
+               /* Fill the buffers for all data columns. */
 
-                       if (ic + 1 == sp->opts->cols || spans)
+               tp->tcol = tp->tcols;
+               cp = cpn = sp->layout->first;
+               dp = sp->first;
+               hspans = 0;
+               for (ic = 0; ic < sp->opts->cols; ic++) {
+                       if (cpn != NULL) {
+                               cp = cpn;
+                               cpn = cpn->next;
+                       }
+                       if (hspans) {
+                               hspans--;
                                continue;
+                       }
+                       tp->tcol++;
+                       tp->col = 0;
+                       tbl_data(tp, sp->opts, cp, dp, tp->tbl.cols + ic);
+                       if (dp == NULL)
+                               continue;
+                       hspans = dp->hspans;
+                       if (cp->pos != TBL_CELL_SPAN)
+                               dp = dp->next;
+               }
+               break;
+       }
 
-                       tbl_char(tp, ASCII_NBRSP, 1);
-                       if (vert > 0)
-                               tbl_char(tp, '|', vert);
-                       if (vert < 2)
-                               tbl_char(tp, ASCII_NBRSP, 2 - vert);
+       do {
+               /* Print the vertical frame at the start of each row. */
+
+               tp->tcol = tp->tcols;
+               fc = '\0';
+               if (sp->layout->vert ||
+                   (sp->next != NULL && sp->next->layout->vert &&
+                    sp->next->pos == TBL_SPAN_DATA) ||
+                   (sp->prev != NULL && sp->prev->layout->vert &&
+                    (horiz || (IS_HORIZ(sp->layout->first) &&
+                      !IS_HORIZ(sp->prev->layout->first)))) ||
+                   sp->opts->opts & (TBL_OPT_BOX | TBL_OPT_DBOX))
+                       fc = horiz || IS_HORIZ(sp->layout->first) ? '+' : '|';
+               else if (horiz && sp->opts->lvert)
+                       fc = '-';
+               if (fc != '\0') {
+                       (*tp->advance)(tp, tp->tcols->offset);
+                       (*tp->letter)(tp, fc);
+                       tp->viscol = tp->tcol->offset + 1;
                }
-       } else if (horiz)
-               tbl_hrule(tp, sp, 0);
 
-       /* Vertical frame at the end of each row. */
+               /* Print the data cells. */
+
+               more = 0;
+               if (horiz) {
+                       tbl_hrule(tp, sp, 0);
+                       term_flushln(tp);
+               } else {
+                       cp = sp->layout->first;
+                       cpn = sp->next == NULL ? NULL :
+                           sp->next->layout->first;
+                       cpp = sp->prev == NULL ? NULL :
+                           sp->prev->layout->first;
+                       dp = sp->first;
+                       hspans = 0;
+                       for (ic = 0; ic < sp->opts->cols; ic++) {
+
+                               /*
+                                * Figure out whether to print a
+                                * vertical line after this cell
+                                * and advance to next layout cell.
+                                */
+
+                               if (cp != NULL) {
+                                       vert = cp->vert;
+                                       switch (cp->pos) {
+                                       case TBL_CELL_HORIZ:
+                                               fc = '-';
+                                               break;
+                                       case TBL_CELL_DHORIZ:
+                                               fc = '=';
+                                               break;
+                                       default:
+                                               fc = ' ';
+                                               break;
+                                       }
+                               } else {
+                                       vert = 0;
+                                       fc = ' ';
+                               }
+                               if (cpp != NULL) {
+                                       if (vert == 0 &&
+                                           cp != NULL &&
+                                           ((IS_HORIZ(cp) &&
+                                             !IS_HORIZ(cpp)) ||
+                                            (cp->next != NULL &&
+                                             cpp->next != NULL &&
+                                             IS_HORIZ(cp->next) &&
+                                             !IS_HORIZ(cpp->next))))
+                                               vert = cpp->vert;
+                                       cpp = cpp->next;
+                               }
+                               if (vert == 0 &&
+                                   sp->opts->opts & TBL_OPT_ALLBOX)
+                                       vert = 1;
+                               if (cpn != NULL) {
+                                       if (vert == 0)
+                                               vert = cpn->vert;
+                                       cpn = cpn->next;
+                               }
+                               if (cp != NULL)
+                                       cp = cp->next;
+
+                               /*
+                                * Skip later cells in a span,
+                                * figure out whether to start a span,
+                                * and advance to next data cell.
+                                */
+
+                               if (hspans) {
+                                       hspans--;
+                                       continue;
+                               }
+                               if (dp != NULL) {
+                                       hspans = dp->hspans;
+                                       if (ic || sp->layout->first->pos
+                                           != TBL_CELL_SPAN)
+                                               dp = dp->next;
+                               }
 
-       if (sp->layout->last->vert ||
-           (sp->prev != NULL && sp->prev->layout->last->vert) ||
-           (sp->opts->opts & (TBL_OPT_BOX | TBL_OPT_DBOX)))
-               term_word(tp, horiz ? "+" : " |");
-       else if (sp->opts->rvert)
-               tbl_char(tp, horiz ? '-' : ASCII_NBRSP, 1);
-       term_flushln(tp);
+                               /*
+                                * Print one line of text in the cell
+                                * and remember whether there is more.
+                                */
+
+                               tp->tcol++;
+                               if (tp->tcol->col < tp->tcol->lastcol)
+                                       term_flushln(tp);
+                               if (tp->tcol->col < tp->tcol->lastcol)
+                                       more = 1;
+
+                               /*
+                                * Vertical frames between data cells,
+                                * but not after the last column.
+                                */
+
+                               if (fc == ' ' && ((vert == 0 &&
+                                    (cp == NULL || !IS_HORIZ(cp))) ||
+                                   tp->tcol + 1 == tp->tcols + tp->lasttcol))
+                                       continue;
+
+                               if (tp->viscol < tp->tcol->rmargin) {
+                                       (*tp->advance)(tp, tp->tcol->rmargin
+                                          - tp->viscol);
+                                       tp->viscol = tp->tcol->rmargin;
+                               }
+                               while (tp->viscol < tp->tcol->rmargin +
+                                   tp->tbl.cols[ic].spacing / 2) {
+                                       (*tp->letter)(tp, fc);
+                                       tp->viscol++;
+                               }
+
+                               if (tp->tcol + 1 == tp->tcols + tp->lasttcol)
+                                       continue;
+
+                               if (fc == ' ' && cp != NULL) {
+                                       switch (cp->pos) {
+                                       case TBL_CELL_HORIZ:
+                                               fc = '-';
+                                               break;
+                                       case TBL_CELL_DHORIZ:
+                                               fc = '=';
+                                               break;
+                                       default:
+                                               break;
+                                       }
+                               }
+                               if (tp->tbl.cols[ic].spacing) {
+                                       (*tp->letter)(tp, fc == ' ' ? '|' :
+                                           vert ? '+' : fc);
+                                       tp->viscol++;
+                               }
+
+                               if (fc != ' ') {
+                                       if (cp != NULL &&
+                                           cp->pos == TBL_CELL_HORIZ)
+                                               fc = '-';
+                                       else if (cp != NULL &&
+                                           cp->pos == TBL_CELL_DHORIZ)
+                                               fc = '=';
+                                       else
+                                               fc = ' ';
+                               }
+                               if (tp->tbl.cols[ic].spacing > 2 &&
+                                   (vert > 1 || fc != ' ')) {
+                                       (*tp->letter)(tp, fc == ' ' ? '|' :
+                                           vert > 1 ? '+' : fc);
+                                       tp->viscol++;
+                               }
+                       }
+               }
+
+               /* Print the vertical frame at the end of each row. */
+
+               fc = '\0';
+               if ((sp->layout->last->vert &&
+                    sp->layout->last->col + 1 == sp->opts->cols) ||
+                   (sp->next != NULL &&
+                    sp->next->layout->last->vert &&
+                    sp->next->layout->last->col + 1 == sp->opts->cols) ||
+                   (sp->prev != NULL &&
+                    sp->prev->layout->last->vert &&
+                    sp->prev->layout->last->col + 1 == sp->opts->cols &&
+                    (horiz || (IS_HORIZ(sp->layout->last) &&
+                     !IS_HORIZ(sp->prev->layout->last)))) ||
+                   (sp->opts->opts & (TBL_OPT_BOX | TBL_OPT_DBOX)))
+                       fc = horiz || IS_HORIZ(sp->layout->last) ? '+' : '|';
+               else if (horiz && sp->opts->rvert)
+                       fc = '-';
+               if (fc != '\0') {
+                       if (horiz == 0 && (IS_HORIZ(sp->layout->last) == 0 ||
+                           sp->layout->last->col + 1 < sp->opts->cols)) {
+                               tp->tcol++;
+                               (*tp->advance)(tp,
+                                   tp->tcol->offset > tp->viscol ?
+                                   tp->tcol->offset - tp->viscol : 1);
+                       }
+                       (*tp->letter)(tp, fc);
+               }
+               (*tp->endline)(tp);
+               tp->viscol = 0;
+       } while (more);
 
        /*
-        * If we're the last row, clean up after ourselves: clear the
-        * existing table configuration and set it to NULL.
+        * Clean up after this row.  If it is the last line
+        * of the table, print the box line and clean up
+        * column data; otherwise, print the allbox line.
         */
 
+       term_setcol(tp, 1);
+       tp->flags &= ~TERMP_MULTICOL;
+       tp->tcol->rmargin = tp->maxrmargin;
        if (sp->next == NULL) {
                if (sp->opts->opts & (TBL_OPT_DBOX | TBL_OPT_BOX)) {
-                       tbl_hrule(tp, sp, 1);
+                       tbl_hrule(tp, sp, 2);
                        tp->skipvsp = 1;
                }
                if (sp->opts->opts & TBL_OPT_DBOX) {
-                       tbl_hrule(tp, sp, 2);
+                       tbl_hrule(tp, sp, 3);
                        tp->skipvsp = 2;
                }
                assert(tp->tbl.cols);
                free(tp->tbl.cols);
                tp->tbl.cols = NULL;
                tp->tcol->offset = offset;
-       }
-       tp->flags &= ~(TERMP_NONOSPACE | TERMP_BRNEVER);
+       } else if (horiz == 0 && sp->opts->opts & TBL_OPT_ALLBOX &&
+           (sp->next == NULL || sp->next->pos == TBL_SPAN_DATA ||
+            sp->next->next != NULL))
+               tbl_hrule(tp, sp, 1);
+
+       tp->flags &= ~TERMP_NONOSPACE;
 }
 
 /*
  * Kinds of horizontal rulers:
  * 0: inside the table (single or double line with crossings)
- * 1: inner frame (single line with crossings and ends)
- * 2: outer frame (single line without crossings with ends)
+ * 1: inside the table (single or double line with crossings and ends)
+ * 2: inner frame (single line with crossings and ends)
+ * 3: outer frame (single line without crossings with ends)
  */
 static void
 tbl_hrule(struct termp *tp, const struct tbl_span *sp, int kind)
 {
-       const struct tbl_cell *c1, *c2;
+       const struct tbl_cell *cp, *cpn, *cpp;
+       const struct roffcol *col;
        int      vert;
-       char     line, cross;
-
-       line = (kind == 0 && TBL_SPAN_DHORIZ == sp->pos) ? '=' : '-';
-       cross = (kind < 2) ? '+' : '-';
-
+       char     cross, line, stdcross, stdline;
+
+       stdline = (kind < 2 && TBL_SPAN_DHORIZ == sp->pos) ? '=' : '-';
+       stdcross = (kind < 3) ? '+' : '-';
+
+       cp = sp->layout->first;
+       cpp = kind || sp->prev == NULL ? NULL : sp->prev->layout->first;
+       if (cpp == cp)
+               cpp = NULL;
+       cpn = kind > 1 || sp->next == NULL ? NULL : sp->next->layout->first;
+       if (cpn == cp)
+               cpn = NULL;
        if (kind)
-               term_word(tp, "+");
-       c1 = sp->layout->first;
-       c2 = sp->prev == NULL ? NULL : sp->prev->layout->first;
-       if (c2 == c1)
-               c2 = NULL;
+               term_word(tp,
+                   cpn == NULL || cpn->pos != TBL_CELL_DOWN ? "+" : "|");
        for (;;) {
-               tbl_char(tp, line, tp->tbl.cols[c1->col].width + 1);
-               vert = c1->vert;
-               if ((c1 = c1->next) == NULL)
+               col = tp->tbl.cols + cp->col;
+               if (cpn == NULL || cpn->pos != TBL_CELL_DOWN) {
+                       line = stdline;
+                       cross = stdcross;
+               } else {
+                       line = ' ';
+                       cross = (kind < 3) ? '|' : ' ';
+               }
+               tbl_char(tp, line, col->width + col->spacing / 2);
+               vert = cp->vert;
+               if ((cp = cp->next) == NULL)
                         break;
-               if (c2 != NULL) {
-                       if (vert < c2->vert)
-                               vert = c2->vert;
-                       c2 = c2->next;
+               if (cpp != NULL) {
+                       if (vert < cpp->vert)
+                               vert = cpp->vert;
+                       cpp = cpp->next;
+               }
+               if (cpn != NULL) {
+                       if (vert < cpn->vert)
+                               vert = cpn->vert;
+                       cpn = cpn->next;
                }
-               if (vert)
-                       tbl_char(tp, cross, vert);
-               if (vert < 2)
-                       tbl_char(tp, line, 2 - vert);
+               if (cpn == NULL || cpn->pos != TBL_CELL_DOWN) {
+                       line = stdline;
+                       cross = stdcross;
+               } else
+                       line = ' ';
+               if (sp->opts->opts & TBL_OPT_ALLBOX && !vert)
+                       vert = 1;
+               if (col->spacing)
+                       tbl_char(tp, vert ? cross : line, 1);
+               if (col->spacing > 2)
+                       tbl_char(tp, vert > 1 ? cross : line, 1);
+               if (col->spacing > 4)
+                       tbl_char(tp, line, (col->spacing - 3) / 2);
        }
        if (kind) {
-               term_word(tp, "+");
+               term_word(tp,
+                   cpn == NULL || cpn->pos != TBL_CELL_DOWN ? "+" : "|");
                term_flushln(tp);
        }
 }
 
 static void
 tbl_data(struct termp *tp, const struct tbl_opts *opts,
-       const struct tbl_dat *dp,
-       const struct roffcol *col)
+    const struct tbl_cell *cp, const struct tbl_dat *dp,
+    const struct roffcol *col)
 {
-
-       if (dp == NULL) {
-               tbl_char(tp, ASCII_NBRSP, col->width);
+       switch (cp->pos) {
+       case TBL_CELL_HORIZ:
+               tbl_char(tp, '-', col->width);
                return;
+       case TBL_CELL_DHORIZ:
+               tbl_char(tp, '=', col->width);
+               return;
+       default:
+               break;
        }
 
+       if (dp == NULL)
+               return;
+
        switch (dp->pos) {
        case TBL_DATA_NONE:
-               tbl_char(tp, ASCII_NBRSP, col->width);
                return;
        case TBL_DATA_HORIZ:
        case TBL_DATA_NHORIZ:
@@ -274,13 +554,7 @@ tbl_data(struct termp *tp, const struct tbl_opts *opts,
                break;
        }
 
-       switch (dp->layout->pos) {
-       case TBL_CELL_HORIZ:
-               tbl_char(tp, '-', col->width);
-               break;
-       case TBL_CELL_DHORIZ:
-               tbl_char(tp, '=', col->width);
-               break;
+       switch (cp->pos) {
        case TBL_CELL_LONG:
        case TBL_CELL_CENTRE:
        case TBL_CELL_LEFT:
@@ -291,7 +565,7 @@ tbl_data(struct termp *tp, const struct tbl_opts *opts,
                tbl_number(tp, opts, dp, col);
                break;
        case TBL_CELL_DOWN:
-               tbl_char(tp, ASCII_NBRSP, col->width);
+       case TBL_CELL_SPAN:
                break;
        default:
                abort();
@@ -318,14 +592,14 @@ tbl_literal(struct termp *tp, const struct tbl_dat *dp,
                const struct roffcol *col)
 {
        size_t           len, padl, padr, width;
-       int              ic, spans;
+       int              ic, hspans;
 
        assert(dp->string);
        len = term_strlen(tp, dp->string);
        width = col->width;
        ic = dp->layout->col;
-       spans = dp->spans;
-       while (spans--)
+       hspans = dp->hspans;
+       while (hspans--)
                width += tp->tbl.cols[++ic].width + 3;
 
        padr = width > len ? width - len : 0;
@@ -360,44 +634,66 @@ tbl_number(struct termp *tp, const struct tbl_opts *opts,
                const struct tbl_dat *dp,
                const struct roffcol *col)
 {
-       char            *cp;
+       const char      *cp, *lastdigit, *lastpoint;
+       size_t           intsz, padl, totsz;
        char             buf[2];
-       size_t           sz, psz, ssz, d, padl;
-       int              i;
 
        /*
-        * See calc_data_number().  Left-pad by taking the offset of our
-        * and the maximum decimal; right-pad by the remaining amount.
+        * Almost the same code as in tblcalc_number():
+        * First find the position of the decimal point.
         */
 
        assert(dp->string);
+       lastdigit = lastpoint = NULL;
+       for (cp = dp->string; cp[0] != '\0'; cp++) {
+               if (cp[0] == '\\' && cp[1] == '&') {
+                       lastdigit = lastpoint = cp;
+                       break;
+               } else if (cp[0] == opts->decimal &&
+                   (isdigit((unsigned char)cp[1]) ||
+                    (cp > dp->string && isdigit((unsigned char)cp[-1]))))
+                       lastpoint = cp;
+               else if (isdigit((unsigned char)cp[0]))
+                       lastdigit = cp;
+       }
 
-       sz = term_strlen(tp, dp->string);
+       /* Then measure both widths. */
 
-       buf[0] = opts->decimal;
-       buf[1] = '\0';
+       padl = 0;
+       totsz = term_strlen(tp, dp->string);
+       if (lastdigit != NULL) {
+               if (lastpoint == NULL)
+                       lastpoint = lastdigit + 1;
+               intsz = 0;
+               buf[1] = '\0';
+               for (cp = dp->string; cp < lastpoint; cp++) {
+                       buf[0] = cp[0];
+                       intsz += term_strlen(tp, buf);
+               }
 
-       psz = term_strlen(tp, buf);
+               /*
+                * Pad left to match the decimal position,
+                * but avoid exceeding the total column width.
+                */
 
-       if ((cp = strrchr(dp->string, opts->decimal)) != NULL) {
-               for (ssz = 0, i = 0; cp != &dp->string[i]; i++) {
-                       buf[0] = dp->string[i];
-                       ssz += term_strlen(tp, buf);
+               if (col->decimal > intsz && col->width > totsz) {
+                       padl = col->decimal - intsz;
+                       if (padl + totsz > col->width)
+                               padl = col->width - totsz;
                }
-               d = ssz + psz;
-       } else
-               d = sz + psz;
-
-       if (col->decimal > d && col->width > sz) {
-               padl = col->decimal - d;
-               if (padl + sz > col->width)
-                       padl = col->width - sz;
-               tbl_char(tp, ASCII_NBRSP, padl);
-       } else
-               padl = 0;
+
+       /* If it is not a number, simply center the string. */
+
+       } else if (col->width > totsz)
+               padl = (col->width - totsz) / 2;
+
+       tbl_char(tp, ASCII_NBRSP, padl);
        tbl_word(tp, dp);
-       if (col->width > sz + padl)
-               tbl_char(tp, ASCII_NBRSP, col->width - sz - padl);
+
+       /* Pad right to fill the column.  */
+
+       if (col->width > padl + totsz)
+               tbl_char(tp, ASCII_NBRSP, col->width - padl - totsz);
 }
 
 static void