improve rounding rules for scaling units
authorschwarze <schwarze@openbsd.org>
Wed, 14 Jun 2017 17:50:43 +0000 (17:50 +0000)
committerschwarze <schwarze@openbsd.org>
Wed, 14 Jun 2017 17:50:43 +0000 (17:50 +0000)
in horizontal orientation in the terminal formatter

23 files changed:
regress/usr.bin/mandoc/man/HP/spacing.in
regress/usr.bin/mandoc/man/IP/width.in
regress/usr.bin/mandoc/man/RS/width.in
regress/usr.bin/mandoc/man/TP/width.in
regress/usr.bin/mandoc/mdoc/Bd/offset-neg.in
regress/usr.bin/mandoc/mdoc/Bl/offset.in
regress/usr.bin/mandoc/mdoc/Bl/offset.out_ascii
regress/usr.bin/mandoc/mdoc/Bl/offset.out_markdown
regress/usr.bin/mandoc/mdoc/Bl/tag.in
regress/usr.bin/mandoc/roff/esc/h.in
regress/usr.bin/mandoc/roff/esc/h.out_ascii
regress/usr.bin/mandoc/roff/esc/h.out_lint
regress/usr.bin/mandoc/roff/ta/basic-man.in
regress/usr.bin/mandoc/roff/ta/basic-man.out_ascii
regress/usr.bin/mandoc/roff/ti/basic-man.in
regress/usr.bin/mandoc/tbl/mod/width.in
usr.bin/mandoc/man_term.c
usr.bin/mandoc/mdoc_term.c
usr.bin/mandoc/roff_term.c
usr.bin/mandoc/tbl_term.c
usr.bin/mandoc/term.c
usr.bin/mandoc/term.h
usr.bin/mandoc/term_tab.c

index 0faec1b..be51168 100644 (file)
@@ -17,7 +17,7 @@ Each hanged paragraph gets a sufficient amount of text
 to wrap to the next line.
 .br
 And a second line.
-.HP -4n
+.HP -0.36i
 Each hanged paragraph gets a sufficient amount of text
 to wrap to the next line.
 .br
@@ -42,7 +42,7 @@ Each hanged paragraph gets a sufficient amount of text
 to wrap to the next line.
 .br
 And a second line.
-.HP 8n
+.HP .76i
 Each hanged paragraph gets a sufficient amount of text
 to wrap to the next line.
 .br
index 1c919a3..7437200 100644 (file)
@@ -7,7 +7,7 @@ Regular mode:
 indented
 .br
 text
-.IP tag -4n
+.IP tag -0.36i
 indented
 .br
 text
@@ -61,7 +61,7 @@ text
 .IP tag 2n
 indented
 text
-.IP tag 3n
+.IP tag 0.26i
 indented
 text
 .IP tag 4n
index e723b58..bba4abc 100644 (file)
@@ -10,13 +10,13 @@ text
 .RE
 regular
 text
-.RS -4n
+.RS -0.36i
 indented
 text
 .RE
 regular
 text
-.RS 4n
+.RS 0.36i
 indented
 text
 .RE
index 1ba99e4..2f4cb1f 100644 (file)
@@ -8,7 +8,7 @@ tag
 indented
 .br
 text
-.TP -4n
+.TP -0.36i
 tag
 indented
 .br
@@ -23,7 +23,7 @@ tag
 indented
 .br
 text
-.TP 2n
+.TP 0.16i
 tag
 indented
 .br
@@ -51,7 +51,7 @@ text
 .PP
 Literal mode:
 .nf
-.TP -10n
+.TP -0.96i
 tag
 indented
 text
@@ -71,7 +71,7 @@ text
 tag
 indented
 text
-.TP 3n
+.TP 0.26i
 tag
 indented
 text
index 30f0fe7..f1d0a2a 100644 (file)
@@ -6,9 +6,9 @@
 .Nd indenting display blocks
 .Sh DESCRIPTION
 regular text
-.Bd -ragged -offset 8n
+.Bd -ragged -offset 0.76i
 outer display
-.Bd -ragged -offset -5n
+.Bd -ragged -offset -0.46i
 inner display
 .Ed
 outer display
index 9564b10..923dd68 100644 (file)
@@ -57,9 +57,9 @@ Bl tag width Ds offset indent
 .It tag
 Bl tag width Ds offset indent-two
 .El
-.Bl -tag -width Ds -offset 4n
+.Bl -tag -width Ds -offset 0.36i
 .It tag
-Bl tag width Ds offset 4n
+Bl tag width Ds offset 0.36i
 .El
 .Bl -tag -width Ds -offset mystring
 .It tag
@@ -73,7 +73,7 @@ Bl tag width Ds offset Ds
 .It tag
 Bl tag width Ds offset 78n
 .El
-.Bl -tag -width Ds -offset -3n
+.Bl -tag -width Ds -offset -0.26i
 .It tag
-Bl tag width Ds offset -3n
+Bl tag width Ds offset -0.26i
 .El
index 8c871d4..22474aa 100644 (file)
@@ -33,7 +33,7 @@ D\bDE\bES\bSC\bCR\bRI\bIP\bPT\bTI\bIO\bON\bN
 
                tag     Bl tag width Ds offset indent-two
 
-         tag     Bl tag width Ds offset 4n
+         tag     Bl tag width Ds offset 0.36i
 
              tag     Bl tag width Ds offset mystring
 
@@ -46,6 +46,6 @@ D\bDE\bES\bSC\bCR\bRI\bIP\bPT\bTI\bIO\bON\bN
                                                                                            offset
                                                                                            78n
 
-  tag     Bl tag width Ds offset -3n
+  tag     Bl tag width Ds offset -0.26i
 
 OpenBSD                        December 25, 2014                       OpenBSD
index 807950b..a58ce6b 100644 (file)
@@ -42,7 +42,7 @@ tag
 
 tag
 
-> Bl tag width Ds offset 4n
+> Bl tag width Ds offset 0.36i
 
 tag
 
@@ -58,6 +58,6 @@ tag
 
 tag
 
-> Bl tag width Ds offset -3n
+> Bl tag width Ds offset -0.26i
 
 OpenBSD - December 25, 2014
index 49cc903..c7b9982 100644 (file)
@@ -9,7 +9,7 @@
 .It tag
 x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x
 .El
-.Bl -tag -width -3n
+.Bl -tag -width -0.26i
 .It tag
 x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x
 .El
@@ -53,7 +53,7 @@ x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x
 .It quint
 x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x
 .El
-.Bl -tag -width 5n
+.Bl -tag -width 0.46i
 .It indent
 x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x x
 .El
index 4bc1953..56e4275 100644 (file)
@@ -7,6 +7,8 @@
 .Sh DESCRIPTION
 simple: >\h'0'<
 .br
+rounding: >\h'0.16i'<
+.br
 escape only: >\h'\w'\&'M'<
 .br
 escape at the end: >\h'0+\w'\&''<
index 858dcf3..ae378a4 100644 (file)
@@ -5,6 +5,7 @@ N\bNA\bAM\bME\bE
 
 D\bDE\bES\bSC\bCR\bRI\bIP\bPT\bTI\bIO\bON\bN
      simple: ><
+     rounding: >  <
      escape only: ><
      escape at the end: ><
      escape at the beginning: ><
index 6e2eb34..b7facfb 100644 (file)
@@ -1,2 +1,2 @@
-mandoc: h.in:18:21: WARNING: invalid escape sequence: \h-
+mandoc: h.in:20:21: WARNING: invalid escape sequence: \h-
 mandoc: h.in:1:5: STYLE: Mdocdate missing: Dd May
index f2f4641..db0644a 100644 (file)
@@ -17,3 +17,6 @@ none:
 default unit:
 .ta 3 +4 12
 1      2       3       4
+rounding:
+.ta 0.26i T 1c
+1      2       3       4       5       6       7       8       9
index 3706224..659422f 100644 (file)
@@ -16,6 +16,8 @@ D\bDE\bES\bSC\bCR\bRI\bIP\bPT\bTI\bIO\bON\bN
        1  2     3   4 5   6 7   8 9
        default unit:
        1  2   3    4
+       rounding:
+       1  2   3   4   5   6   7   8   9
 
 
 
index deb1542..d4f00a0 100644 (file)
@@ -3,14 +3,14 @@
 ti-basic-man \- temporary indentation in man(7)
 .SH DESCRIPTION
 initial text
-.ti 10n
+.ti 0.96i
 This text is indented.
 However, the indent is temporary,
 so as soon as the line wraps, it is gone.
 .ti +10n
 The temporary indent can be relative.
 All the same, it will expire in the usual way.
-.ti -3n
+.ti -0.26i
 It is also possible to temporarily reduce the indentation,
 but that will also last until the next line break only.
 .ti -10n
index 702dd19..4630608 100644 (file)
@@ -5,7 +5,7 @@ tbl-mod-width \- width modifier in table layout
 normal text
 .TS
 box tab(:);
-lw2 | lw(2n) | lw(0.2i) | lw2 .
+lw2 | lw(2n) | lw(0.16i) | lw2 .
 a:abcd:T{
 a
 T}:T{
index f43d1bd..9b76fdc 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: man_term.c,v 1.157 2017/06/08 12:54:40 schwarze Exp $ */
+/*     $OpenBSD: man_term.c,v 1.158 2017/06/14 17:50:43 schwarze Exp $ */
 /*
  * Copyright (c) 2008-2012 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010-2015, 2017 Ingo Schwarze <schwarze@openbsd.org>
@@ -375,7 +375,7 @@ pre_in(DECL_ARGS)
        if (a2roffsu(++cp, &su, SCALE_EN) == NULL)
                return 0;
 
-       v = (term_hspan(p, &su) + 11) / 24;
+       v = term_hen(p, &su);
 
        if (less < 0)
                p->tcol->offset -= p->tcol->offset > v ? v : p->tcol->offset;
@@ -424,7 +424,7 @@ pre_HP(DECL_ARGS)
 
        if ((nn = n->parent->head->child) != NULL &&
            a2roffsu(nn->string, &su, SCALE_EN) != NULL) {
-               len = term_hspan(p, &su) / 24;
+               len = term_hen(p, &su);
                if (len < 0 && (size_t)(-len) > mt->offset)
                        len = -mt->offset;
                else if (len > SHRT_MAX)
@@ -509,7 +509,7 @@ pre_IP(DECL_ARGS)
        if ((nn = n->parent->head->child) != NULL &&
            (nn = nn->next) != NULL &&
            a2roffsu(nn->string, &su, SCALE_EN) != NULL) {
-               len = term_hspan(p, &su) / 24;
+               len = term_hen(p, &su);
                if (len < 0 && (size_t)(-len) > mt->offset)
                        len = -mt->offset;
                else if (len > SHRT_MAX)
@@ -591,7 +591,7 @@ pre_TP(DECL_ARGS)
        if ((nn = n->parent->head->child) != NULL &&
            nn->string != NULL && ! (NODE_LINE & nn->flags) &&
            a2roffsu(nn->string, &su, SCALE_EN) != NULL) {
-               len = term_hspan(p, &su) / 24;
+               len = term_hen(p, &su);
                if (len < 0 && (size_t)(-len) > mt->offset)
                        len = -mt->offset;
                else if (len > SHRT_MAX)
@@ -795,7 +795,7 @@ pre_RS(DECL_ARGS)
        if (n->child == NULL)
                n->aux = mt->lmargin[mt->lmargincur];
        else if (a2roffsu(n->child->string, &su, SCALE_EN) != NULL)
-               n->aux = term_hspan(p, &su) / 24;
+               n->aux = term_hen(p, &su);
        if (n->aux < 0 && (size_t)(-n->aux) > mt->offset)
                n->aux = -mt->offset;
        else if (n->aux > SHRT_MAX)
index d91ad6e..3adb397 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: mdoc_term.c,v 1.262 2017/06/08 12:54:40 schwarze Exp $ */
+/*     $OpenBSD: mdoc_term.c,v 1.263 2017/06/14 17:50:43 schwarze Exp $ */
 /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010, 2012-2017 Ingo Schwarze <schwarze@openbsd.org>
@@ -538,7 +538,7 @@ a2width(const struct termp *p, const char *v)
                SCALE_HS_INIT(&su, term_strlen(p, v));
                su.scale /= term_strlen(p, "0");
        }
-       return term_hspan(p, &su) / 24;
+       return term_hen(p, &su);
 }
 
 /*
@@ -684,7 +684,7 @@ termp_it_pre(DECL_ARGS)
                        SCALE_HS_INIT(&su,
                            term_strlen(p, bl->norm->Bl.cols[i]));
                        su.scale /= term_strlen(p, "0");
-                       offset += term_hspan(p, &su) / 24 + dcol;
+                       offset += term_hen(p, &su) + dcol;
                }
 
                /*
@@ -702,7 +702,7 @@ termp_it_pre(DECL_ARGS)
                 */
                SCALE_HS_INIT(&su, term_strlen(p, bl->norm->Bl.cols[i]));
                su.scale /= term_strlen(p, "0");
-               width = term_hspan(p, &su) / 24 + dcol;
+               width = term_hen(p, &su) + dcol;
                break;
        default:
                if (NULL == bl->norm->Bl.width)
index 608a9c5..e561b43 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: roff_term.c,v 1.11 2017/06/14 13:00:13 schwarze Exp $ */
+/*     $OpenBSD: roff_term.c,v 1.12 2017/06/14 17:50:43 schwarze Exp $ */
 /*
  * Copyright (c) 2010, 2014, 2015, 2017 Ingo Schwarze <schwarze@openbsd.org>
  *
@@ -205,7 +205,7 @@ roff_term_pre_ti(ROFF_TERM_ARGS)
 
        if (a2roffsu(cp, &su, SCALE_EM) == NULL)
                return;
-       len = term_hspan(p, &su) / 24;
+       len = term_hen(p, &su);
 
        if (sign == 0) {
                p->ti = len - p->tcol->offset;
index 8011095..1c98af0 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: tbl_term.c,v 1.39 2017/06/13 14:38:38 schwarze Exp $ */
+/*     $OpenBSD: tbl_term.c,v 1.40 2017/06/14 17:50:43 schwarze Exp $ */
 /*
  * Copyright (c) 2009, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2011,2012,2014,2015,2017 Ingo Schwarze <schwarze@openbsd.org>
@@ -45,7 +45,7 @@ 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;
+       return term_hen((const struct termp *)arg, su);
 }
 
 static size_t
index e6aebdc..9c93a3a 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: term.c,v 1.130 2017/06/14 01:31:19 schwarze Exp $ */
+/*     $OpenBSD: term.c,v 1.131 2017/06/14 17:50:43 schwarze Exp $ */
 /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010-2017 Ingo Schwarze <schwarze@openbsd.org>
@@ -526,7 +526,7 @@ term_word(struct termp *p, const char *word)
                case ESCAPE_HORIZ:
                        if (a2roffsu(seq, &su, SCALE_EM) == NULL)
                                continue;
-                       uc = term_hspan(p, &su) / 24;
+                       uc = term_hen(p, &su);
                        if (uc > 0)
                                while (uc-- > 0)
                                        bufferc(p, ASCII_NBRSP);
@@ -547,7 +547,7 @@ term_word(struct termp *p, const char *word)
                case ESCAPE_HLINE:
                        if ((seq = a2roffsu(seq, &su, SCALE_EM)) == NULL)
                                continue;
-                       uc = term_hspan(p, &su) / 24;
+                       uc = term_hen(p, &su);
                        if (uc <= 0) {
                                if (p->tcol->rmargin <= p->tcol->offset)
                                        continue;
@@ -964,7 +964,7 @@ term_vspan(const struct termp *p, const struct roffsu *su)
 }
 
 /*
- * Convert a scaling width to basic units, rounding down.
+ * Convert a scaling width to basic units, rounding towards 0.
  */
 int
 term_hspan(const struct termp *p, const struct roffsu *su)
@@ -972,3 +972,17 @@ term_hspan(const struct termp *p, const struct roffsu *su)
 
        return (*p->hspan)(p, su);
 }
+
+/*
+ * Convert a scaling width to basic units, rounding to closest.
+ */
+int
+term_hen(const struct termp *p, const struct roffsu *su)
+{
+       int bu;
+
+       if ((bu = (*p->hspan)(p, su)) >= 0)
+               return (bu + 11) / 24;
+       else
+               return -((-bu + 11) / 24);
+}
index ffbeb24..0042627 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: term.h,v 1.71 2017/06/12 18:55:42 schwarze Exp $ */
+/*     $OpenBSD: term.h,v 1.72 2017/06/14 17:50:43 schwarze Exp $ */
 /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2011-2015, 2017 Ingo Schwarze <schwarze@openbsd.org>
@@ -140,6 +140,7 @@ void                  term_end(struct termp *);
 
 void             term_setwidth(struct termp *, const char *);
 int              term_hspan(const struct termp *, const struct roffsu *);
+int              term_hen(const struct termp *, const struct roffsu *);
 int              term_vspan(const struct termp *, const struct roffsu *);
 size_t           term_strlen(const struct termp *, const char *);
 size_t           term_len(const struct termp *, size_t);
index 326a112..3164840 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: term_tab.c,v 1.2 2017/06/08 12:54:40 schwarze Exp $ */
+/*     $OpenBSD: term_tab.c,v 1.3 2017/06/14 17:50:43 schwarze Exp $ */
 /*
  * Copyright (c) 2017 Ingo Schwarze <schwarze@openbsd.org>
  *
@@ -52,7 +52,7 @@ term_tab_set(const struct termp *p, const char *arg)
                recording_period = 0;
                if (tabs.d == 0) {
                        a2roffsu(".8i", &su, SCALE_IN);
-                       tabs.d = term_hspan(p, &su) / 24;
+                       tabs.d = term_hen(p, &su);
                }
                return;
        }
@@ -81,7 +81,7 @@ term_tab_set(const struct termp *p, const char *arg)
 
        /* Append the new position. */
 
-       pos = term_hspan(p, &su);
+       pos = term_hen(p, &su);
        tl->t[tl->n] = pos;
        if (add && tl->n)
                tl->t[tl->n] += tl->t[tl->n - 1];
@@ -97,10 +97,6 @@ term_tab_next(size_t prev)
                if (i == tabs.a.n) {
                        if (tabs.p.n == 0)
                                return prev;
-/*
-                               return i ? prev :
-                                   (prev / tabs.d + 1) * tabs.d;
- */
                        tabs.a.n += tabs.p.n;
                        if (tabs.a.s < tabs.a.n) {
                                tabs.a.s = tabs.a.n;
@@ -111,7 +107,7 @@ term_tab_next(size_t prev)
                                tabs.a.t[i + j] = tabs.p.t[j] +
                                    (i ? tabs.a.t[i - 1] : 0);
                }
-               if (prev < tabs.a.t[i] / 24)
-                       return tabs.a.t[i] / 24;
+               if (prev < tabs.a.t[i])
+                       return tabs.a.t[i];
        }
 }