[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
groff-manpage.patch
Description: Text document
- [PATCH] man page typo in spelling of lsn and lss registers?,
Alexis <=