bugGNU troff - Bugs: bug #61450, Add warning for extraneous...

 
 

bug #61450: Add warning for extraneous arguments to requests

Submitted by:  Ralph Corderoy <ralph>
Submitted on:  Tue 09 Nov 2021 11:39:37 AM UTC
 
Category:  Core Severity:  1 - Wish
Item Group:  Warning/Suspicious behaviour Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open Planned Release:  None
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       No canned response available

 

Tue 11 Jan 2022 04:06:39 AM UTC, comment #2: 

Moving back to former Item Group.  Sorry for the churn.

G. Branden Robinson <gbranden>
Project Administrator
Sun 14 Nov 2021 12:02:40 AM UTC, comment #1: 

This might be as simple as adding a conditional to `skip_line()`.

`token::next()` already takes care of escape sequence processing, so comments still work as documented.

So do other escape sequences.  Here are some interesting things you can do today, though I don't recall seeing any input that exploits the rules this aggressively.

$ cat EXPERIMENTS/post-request-funny-business.groff
.ft R \R'a 5'
.ps 10 \s'24z'
.tm a=\na, .ps=\n[.ps]u
$ groff EXPERIMENTS/post-request-funny-business.groff
a=5, .ps=24000u

G. Branden Robinson <gbranden>
Project Administrator
Tue 09 Nov 2021 11:39:37 AM UTC, original submission:  

No warning is issued when a request has extraneous arguments, e.g. ‘.ce 3 1 4 1 5’.  A warning should be added and included in the ‘all’ grouping of warnings.  Experience from deploying that warning will allow a future choice on whether extraneous arguments should be upgraded to an error, e.g. there's no widespread historical habit of extra arguments being provided knowing that they'll be ignored.

Conversation causing this bug report occurred on the groff mailing list in 2021-11 under the subject ‘Two trivial questions’.

Ralph Corderoy <ralph>

 

(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 (Posted a comment)
  • -email is unavailable- added by ralph (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
    2022-01-11 gbranden Item GroupFeature change => Warning/Suspicious behaviour
    2022-01-11 gbranden Item GroupWarning/Suspicious behaviour => Feature change
    2021-11-14 gbranden Severity3 - Normal => 1 - Wish

    Back to the top


    Powered by Savane 3.9