bugGNU roff - Bugs: bug #61020, [me] centers pic(1) pictures,...

 
 

bug #61020: [me] centers pic(1) pictures, Heirloom doesn't

Submitter:  G. Branden Robinson <gbranden>
Submitted:  Sat 07 Aug 2021 04:38:41 PM UTC
   
 
Category:  Macro me Severity:  3 - Normal
Item Group:  Incorrect behaviour Status:  Invalid
Privacy:  Public Assigned to:  gbranden
Open/Closed:  Closed Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 12 Oct 2022 09:30:26 PM UTC, comment #4: 

comment #3:

> It may be best at this point to regard this ticket as
> irretrievable "invalid" without dwelling too much on details.


I'll do that as long as you share those mushrooms!

Dave <barx>
Group Member
Wed 12 Oct 2022 05:29:00 PM UTC, comment #3: 


comment #2:

> The Summary cites a difference in groff vs Heirloom pic handling, but comment #1 justifies the bug's closing based on Heirloom's lack of grn, a different preprocessor.  Since both roffs include pic, if there's a difference in how they handle it, that stands regardless of anything regarding grn, yes?  Or am I missing something?


I think I confused myself into wild delusion regarding this issue.

Also see the following pending commit of mine.


commit 1bc919d5e7ba613dbe48b7e0750737cdd4fa0b97 (HEAD -> master)
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
Date:   Wed Oct 12 12:21:23 2022 -0500

    doc/meref.me.in: Fix momentary lapse of reason.

    Partially revert commit 1728f9229a, 5 October.  I must have eaten the
    wrong mushrooms; the correct argument order was "h w" all along.  That
    is what pic(1) produces and what me(7) must accept.

    DWB 3.3 pic:
    pltroff.c: printf(".PS %.3fi %.3fi %s", yconv(ymin), xconv(xmax), s);

    GNU pic:
    src/preproc/pic/troff.cpp: printf(".PS %.3fi %.3fi", height, width);

    4.4BSD e.tmac:
    .de PS \" *** start picture: $1=height, $2=width in units or inches
    [...]
    .nr g7 \\$2
    .in (\\n(.lu-\\n(g7u)/2u
    .ne \\$1u

    GNU e.tmac:
    .de PS  \" *** start picture: $1=height, $2=width in units or inches
    [...]
    .nr g7 \\$2
    .in (u;\\n(.l-\\n(g7>?0/2)
    .ne \\$1u


It may be best at this point to regard this ticket as irretrievable "invalid" without dwelling too much on details.

😅

G. Branden Robinson <gbranden>
Group administrator
Sat 08 Oct 2022 12:25:04 AM UTC, comment #2: 

The Summary cites a difference in groff vs Heirloom pic handling, but comment #1 justifies the bug's closing based on Heirloom's lack of grn, a different preprocessor.  Since both roffs include pic, if there's a difference in how they handle it, that stands regardless of anything regarding grn, yes?  Or am I missing something?

Dave <barx>
Group Member
Tue 28 Dec 2021 11:32:21 AM UTC, comment #1: 

Heirloom's me(7) appears to be based on SCCS version 2.14 of the
macros from Berkeley.  Inspecting the source, there is no
evidence of grn(1) support at all.

There is therefore nothing to correct or document here.

G. Branden Robinson <gbranden>
Group administrator
Sat 07 Aug 2021 04:38:41 PM UTC, original submission:  

Should me(7) center pictures by default?

This is either a documentation issue or a correctness issue.

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 barx (Posted a comment)
  • -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 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2021-12-28 gbranden StatusNeed Info Invalid
        Assigned toNone gbranden
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code