bug-groff
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug #59608] sample_docs.mom: error: an argument has a 'p' unit attached


From: Bjarni Ingi Gislason
Subject: [bug #59608] sample_docs.mom: error: an argument has a 'p' unit attached
Date: Sun, 17 Jan 2021 20:22:31 -0500 (EST)
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Firefox/78.0

Follow-up Comment #6, bug #59608 (project groff):


[comment #5 comment #5:]
> 1) Mark significant trailing space as such, that is not just "...abc \"
>   but
> "...abc \"significant trailing space
> 
> Marking significant trailing space with just the slash-quote follows the
recommendation made by Werner some years ago.  With nothing after it, it
already means "the foregoing space is significant."  Adding the comment
afterwards would be tautological.
> 
  If the reader already recognizes its meaning.

  That does not apply to readers,
that see such a "construct" for the first time,
or have forgotten what it is supposed to mean.

I find an explicit "warning text (sign)" better and more future proof.

N.B.  Starting each sentence on a new line is better to make clear to
what part an answer applies to.

> Not sure what you mean by "2) Show all arguments in a diagnostic message." 
Do you mean every argument that was passed to the macro?  What would be the
advantage?  A user needs 

a) Every argument

b) The reader sees all (variable) information;
otherwise it is obscured (censored).

Adding one line to all instances is simpler
than to edit each relevant line.

.tm The arguments are: \\@


>only know the offending calling macro, its position in the file, which arg is
bad ("first," "second"... or "arg to <keyword"), and why. 

How do you know in advance,
what a user needs to fix the error in the shortest time.
The content of an argument is now not always shown.


> Listing additional correct arguments serves no purpose and could, in some
instances, add a huge amount noise.  Some of the _STYLE macros have over ten
keyword/value pairs as arguments.    

The number of arguments to be displayed does not matter,
their content could.

The simplest cure as stated above:
add a single line to all diagnostic blocks,
like

.tm The arguments are: \\@



    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?59608>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]