Update of bug #67082 (group groff):
Severity: 3 - Normal => 2 - Minor
_______________________________________________________
Follow-up Comment #1:
[http://git.savannah.gnu.org/cgit/groff.git/commit/?id=6e4d2554e Commit
6e4d2554e] (from August 2023, after 1.23's release) fixed half of this
problem. The "Next" nodes now correctly traverse from "if-then" (5.23.2) to
"if-else" (5.23.3) to "Conditional Blocks" (5.23.4) to "while" (5.23.5).
But going the other direction is still broken: selecting "Previous" from
"while" (5.23.5) goes to "if-else" (5.23.3), skipping the correct previous
node, "Conditional Blocks" (5.23.4). The rest of the "Previous" links in
section 5.23 are correct.
The offending line in doc/groff.texi.in appears to be:
@node while, , if-else, Conditionals and Loops
It looks to me like these fields are ordered: current node, next node,
previous node. So this line incorrectly indicates that "if-else" is previous
to "while". (This line has remained unchanged since 1999, so you've found a
very old bug.)
As an aside, it astounds me that every "Previous" and "Next" field has to be
explicitly specified in the source code, an error-prone process (as Branden
discovered in 2023, and you discovered this year) and major DRY violation. In
a sane design, the manual's structure would be specified in one place, and all
previous/next links would be populated automatically from that.
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/bugs/?67082>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/
signature.asc
Description: PGP signature
