bugGNU roff - Bugs: bug #64029, [man,mdoc]: page break handling...

 
 

bug #64029: [man,mdoc]: page break handling probably introduced another problem

Submitter:  None
Submitted:  Tue 11 Apr 2023 09:03:01 PM UTC
   
 
Category:  Macro package - others/general Severity:  3 - Normal
Item Group:  Incorrect behaviour Status:  Duplicate
Privacy:  Public Assigned to:  gbranden
Open/Closed:  Closed Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 12 Apr 2023 07:48:56 PM UTC, comment #4: 

Thanks, Dirk.  Closing as duplicate of bug #63960.

G. Branden Robinson <gbranden>
Group administrator
Wed 12 Apr 2023 07:12:47 PM UTC, comment #3: 

I applied the patch and the first tests look good -- even with a more complex example than the attached one.

So, this indeed seems to be a duplicate of bug #63960.

I will continue to work with the patch and report if I notice problems -- either here or in bug #63960.

Dirk Gouders <dgouders>
Wed 12 Apr 2023 05:37:34 AM UTC, comment #2: 

Thank you Branden,

will test it and report.

(And sorry for submitting this anonymously -- I did not notice I am not logged in.)

Dirk Gouders <dgouders>
Wed 12 Apr 2023 02:12:02 AM UTC, comment #1: 

Hi there,

This sounds like it might be the same issue as bug #63960.

The following patch seems to fix it.

Unfortunately, we're frozen for everything except documentation changes for groff 1.23.0 at this point; you might want to consider a local patch.

A workaround is to not use vertical rules in tables in man(7) or mdoc(7), but I'm sure that feels even more lame to hear as it does for me to say it.


diff --git a/tmac/an.tmac b/tmac/an.tmac
index 9e38ba569..ef1eab241 100644
--- a/tmac/an.tmac
+++ b/tmac/an.tmac
@@ -1203,6 +1203,10 @@ contains unsupported escape sequence
 .  \" If continuous rendering, tell tbl not to use keeps.
 .  ie \\n[cR] .nr 3usekeeps 0
 .  el         .nr 3usekeeps 1
+.  if \\n[cR] \{\
+.    nr an*saved-trap-distance \\n[.t]
+.    pl +1000i
+.  \}
 .  if \\n[an*is-output-html] \{\
 .    nr an-TS-ll \\n[.l]
 .    ll 1000n
@@ -1218,6 +1222,10 @@ contains unsupported escape sequence
 .de1 TE
 .  HTML-IMAGE-END
 .  if \\n[an*is-output-html] .ll \\n[an-TS-ll]u
+.  if \\n[cR] \{\
+.    pl (u;\\n[nl] + \\n[an*saved-trap-distance])
+.    rr an*saved-trap-distance
+.  \}
 .  if !r TW .if !\\n[an-was-tbl-failure-reported] \{\
 .    ds an-msg tbl preprocessor failed, or it or soelim was not run;\"
 .    as an-msg " table(s) likely not rendered\"


Please confirm if this fixes the problem for you.

G. Branden Robinson <gbranden>
Group administrator
Tue 11 Apr 2023 09:03:01 PM UTC, original submission:  

groff-1.22.4 had a problem with tbl inserting empty lines at
page breaks even for unlimited page sizes.

groff-1.23.0_rc3 fixed that problem but maybe introduced another one:  in large tables the column separator seems to be vertically
displaced, enlarging the whole document.

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #54605:  sample.1 added by None (395B - application/x-troff-man - Example manual page plus output that I get.)
file #54606:  sample.out added by None (3KiB - application/octet-stream - Example manual page plus output that I get.)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by dgouders (Posted a comment)
  • -email is unavailable- added by gbranden (Posted a comment)
  •  

    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 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2023-04-12 gbranden StatusNeed Info Duplicate
        Open/ClosedOpen Closed
    2023-04-12 gbranden CategoryPreprocessor tbl Macro package - others/general
        StatusNone Need Info
        Assigned toNone gbranden
        Summarytbl: page break handling probably introduced another problem [man,mdoc]: page break handling probably introduced another problem
    2023-04-11 None Attached File- Added sample.1, #54605
        Attached File- Added sample.out, #54606

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code