bugGNU roff - Bugs: bug #53773, protect spaces before comments...

 
 

bug #53773: protect spaces before comments from the stripper

Submitter:  Bjarni Ingi Gislason <bjarniig>
Submitted:  Sat 28 Apr 2018 12:09:42 AM UTC
   
 
Category:  Macro - others/general Severity:  1 - Wish
Item Group:  Lint Status:  Duplicate
Privacy:  Public Assigned to:  gbranden
Open/Closed:  Closed Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 30 Mar 2023 05:07:56 PM UTC, comment #2: 

Dropping patch annotation from rejected ticket.

Reclassifying as duplicate of bug #53760, of which it is a generalization.

G. Branden Robinson <gbranden>
Group administrator
Mon 05 Nov 2018 01:23:36 PM UTC, comment #1: 

I'm not crazy about this idea for two reasons:

1. Stripped macro files are write-only anyway.

2. People should be configuring their text editors to show them trailing space.  It will make their lives better and their work cleaner in multifarious ways.

For instance, from my .vimrc:


set list
if &encoding =~ "utf-8"
    set listchars=tab:»–,trail:·,extends:→,precedes:←,nbsp:␣
    if has("conceal")
        set listchars+=conceal:×
    endif
    " set listchars+=eol:¤
else
    set listchars=tab:>-,trail:.,extends:/,precedes:\\,nbsp:_
    if has("conceal")
        set listchars+=conceal:*
    endif
    " set listchars+=eol:$
endif


G. Branden Robinson <gbranden>
Group administrator
Sat 28 Apr 2018 12:09:42 AM UTC, original submission:  

From 750b9fefc49ea1362eac79a078c1a4c03f8cccd8 Mon Sep 17 00:00:00 2001
From: Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
Date: Sat, 28 Apr 2018 00:02:57 +0000
Subject: [PATCH] macro files: Protect space and its visibility

  Add a '\&' in front of '\"' to preserve the visibility of significant
space characters in the case the comment gets stripped off (with
"tmac/strip.sed").

  Add a missing comment ("Protect space and its visibility") to an
empty comment field.  Not all readers understand the function of '\"'
in these cases.

Signed-off-by: Bjarni Ingi Gislason <bjarniig@rhi.hi.is>
---
 contrib/mm/m.tmac         | 10 +++++-----
 contrib/mm/mse.tmac       |  2 +-
 contrib/pdfmark/spdf.tmac |  6 +++---
 tmac/cs.tmac              |  2 +-
 tmac/de.tmac              |  2 +-
 tmac/doc.tmac-u           |  6 +++---
 tmac/fr.tmac              |  2 +-
 tmac/pdfpic.tmac          |  8 ++++----
 tmac/ptx.tmac             |  6 +++---
 tmac/sv.tmac              |  2 +-
 tmac/trace.tmac           | 14 +++++++-------
 tmac/www.tmac.in          | 10 +++++-----
 12 files changed, 35 insertions(+), 35 deletions(-)

  The patch is in the attachment.

Bjarni Ingi Gislason <bjarniig>

 

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

Attach Files:
   
   
Comment:
   

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by gbranden (Posted a comment)
  • -email is unavailable- added by bjarniig (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 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2023-03-30 gbranden StatusRejected Duplicate
        Summary[PATCH] macro files: Protect space and its visibility protect spaces before comments from the stripper
    2022-03-05 gbranden Item GroupFeature change Lint
    2018-11-05 gbranden Severity3 - Normal 1 - Wish
        StatusNone Rejected
        Assigned toNone gbranden
        Open/ClosedOpen Closed
    2018-04-28 bjarniig Attached File- Added 0001-macro-files-Protect-space-and-its-visibility.txt, #44052

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code