groff
[Top][All Lists]
Advanced

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

[PATCH] man page typo in spelling of lsn and lss registers?


From: Alexis
Subject: [PATCH] man page typo in spelling of lsn and lss registers?
Date: Wed, 12 Jul 2023 13:14:59 +0200

Hello all,

reading through the groff(7) manpage I noticed an inconsistency in the
spelling of the \n[lsn] and \n[lss] registers:

The "Writable predefined registers" section references

  `\n[lsn]` and `\n[lss]`

whereas the paragraphs under the "Traps" section mention

  `\n[.lsn]` and `\n[.lss]` (notice the leading period `.`)

A quick grep over the sources and groff.texi reveals several occurrences
of `lsn` and `lss` yet  the period-prefixed variant only appears in
groff.7.man:

% grep -R 'ls[sn]' src doc/groff.texi
src/roff/troff/input.cpp:  register_dictionary.define("lsn", new 
variable_reg(&leading_spaces_number));
src/roff/troff/input.cpp:  register_dictionary.define("lss", new 
variable_reg(&leading_spaces_space));
doc/groff.texi:@DefregItemx {lsn}
doc/groff.texi:@DefregListEndx {lss}
doc/groff.texi:@code{lsn}, and the amount of corresponding horizontal motion in
doc/groff.texi:register @code{lss}, irrespective of whether a leading space 
trap is

% grep -Rl '\.ls[ns]' .
./man/groff.7.man

My current understanding of groff's code base is virtually non-existent
and I may have missed some crucial aspects and things in regards to
the aforementioned inconsistency and things are as they should be.
In that case I'd appreciate pointers that would help me understand why
the things are the way they are.

Otherwise please find attached a patch that removes the leading period
`.` from the \n[lsn] and \n[lss] registers in the "Traps" section in
groff.7.man.


Best
Alexis

Attachment: groff-manpage.patch
Description: Text document


reply via email to

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