bugGNU roff - Bugs: bug #62181, [me] setting of hyphenation mode...

 
 

bug #62181: [me] setting of hyphenation mode not robust to meddling

Submitter:  G. Branden Robinson <gbranden>
Submitted:  Mon 14 Mar 2022 06:12:39 AM UTC
   
 
Category:  Macro me 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
   

Fri 18 Mar 2022 04:43:46 AM UTC, comment #1: 


commit a7853c5b5bb975a6e058238405bb0080a9a4a19a
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
Date:   Tue Mar 15 18:17:42 2022 +1100

    [me]: Fix Savannah #62181.

    * tmac/e.tmac: Test register for existence before using it to set
      hyphenation mode.  A site troffrc or other local customization might
      remove the register (which is guaranteed to be set under the stock
      configuration) and, perversely, Heirloom Doctools troff interprets a
      `do` request, instead of ignoring it as a historical troff
      implementation would, so if they borrow our me(7) changes, the
      register would be unset, interpolate zero, and hyphenation would wind
      up disabled.

    Fixes <https://savannah.gnu.org/bugs/?62181>.  Thanks to Dave Kemper for
    the report.


G. Branden Robinson <gbranden>
Group administrator
Mon 14 Mar 2022 06:12:39 AM UTC, original submission:  

Dave pointed out in bug #61734 that some new code in our me(7) is not robust to meddling with the hyphenation mode registers.


.\" Set package default hyphenation mode, but override it with groff's
.\" localized value if available.
.hy 6
.do hy \\n[\\*[locale]*hyphenation-mode-trap]


It's possible that the register named in the `do` request won't be defined, in which case the hyphenation mode will be set to zero--disabled.

That would be a surprise to experienced me(7) users.

Test the register for existence before using it.

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
  • -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 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-03-18 gbranden StatusIn Progress Fixed
        Open/ClosedOpen Closed
        Planned ReleaseNone 1.23.0
    2022-03-14 barx Carbon-Copy- Added barx

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code