bugGNU roff - Bugs: bug #60624, [man] unnecessarily breaks after...

 
 

bug #60624: [man] unnecessarily breaks after short TP tag at page boundary

Submitter:  G. Branden Robinson <gbranden>
Submitted:  Mon 17 May 2021 10:48:22 PM UTC
   
 
Category:  Macro package man Severity:  3 - Normal
Item Group:  Incorrect behaviour Status:  Fixed
Privacy:  Public Assigned to:  gbranden
Open/Closed:  Closed Planned Release:  1.23.0
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 14 Jun 2021 06:34:20 PM UTC, comment #1: 


commit 4e1dbb55dc805d8eb5f5bcfa8ab8cbe1059cb7fd
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
Date:   Tue Jun 15 04:20:30 2021 +1000

    [man]: Fix Savannah #60624.

    [man]: Don't spuriously break a page after a paragraph tag when the tag
    is short enough to fit within the paragraph's indentation.

    * tmac/an-old.tmac (an-write-paragraph-tag): Reserve ("need") 2 vees
      plus 1 basic unit of vertical space regardless of the width of the
      tag.  In the short tag case, we reverse space by one vee after
      outputting the diversion in which the tag is stored, but it will
      always contain a break, and by the time we reverse, we might have
      already hit a page location trap (an-break-body-text), and by then
      it's too late to back up.

    Fixes <https://savannah.gnu.org/bugs/index.php?60624>.

commit 42ff1127bc4c1ea19482b7147f28238156e1269e
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
Date:   Tue Jun 15 04:00:38 2021 +1000

    Add regression test for Savannah #60624.

    * tmac/tests/an-old_no-break-after-short-paragraph-tags.sh: Test it.
    * tmac/tmac.am (tmac_TESTS): Run test.

    Test fails at this commit.


G. Branden Robinson <gbranden>
Group administrator
Mon 17 May 2021 10:48:22 PM UTC, original submission:  

Present in groff 1.22.4 and Git HEAD.

Input:

.\" groff -man -rcR=0
.TH foo 1 2021-05-18 "groff test suite"
.SH Name
foo \- frobnicate a thingamajig
.rs \" force spacing on
.sp 50
.TP
3
7th edition


Output:

$ groff -Tutf8 -man -rcR=0 EXPERIMENTS/bad-TP-break.man | cat -s

foo(1)                      General Commands Manual                     foo(1)

Name
       foo - frobnicate a thingamajig

       3

groff test suite                  2021‐05‐18                                 1

foo(1)                      General Commands Manual                     foo(1)

              7th edition

groff test suite                  2021‐05‐18                                 2


G. Branden Robinson <gbranden>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by gbranden (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2021-06-14 gbranden StatusNone Fixed
        Assigned toNone gbranden
        Open/ClosedOpen Closed
        Planned ReleaseNone 1.23.0
    2021-05-19 gbranden Summary[man] unnecessarily breaks short TP tag at page boundary [man] unnecessarily breaks after short TP tag at page boundary

    Back to the top

    Powered by Savane 3.13-bb6a.
    Corresponding source code