Follow-up Comment #8, bug #44018 (group groff):
commit cdcb5c244a33abe12f7c544620afe0a5906fd20a
Author: G. Branden Robinson <g.branden.robin...@gmail.com>
Date: Thu Apr 3 21:54:44 2025 -0500
[groff]: Improve CJK break/adjustment test case.
* src/roff/groff/tests/do-not-loop-infinitely-when-breaking-cjk.sh: Make
test case more illustrative. Conform better to modern groff test
conventions.
commit 22f5b0512b6983843aac72bd92426fcbc182bc8b
Author: G. Branden Robinson <g.branden.robin...@gmail.com>
Date: Fri Apr 4 02:38:07 2025 -0500
[troff]: Slightly refactor to clarify logic.
...in difficult adjustment cases.
* src/roff/troff/env.cpp (distribute_space): Promote return type from
`void` to `bool`, communicating whether this function successfully
adjusted spaces. Add/update `return` statements.
(environment::possibly_break_line): Populate `output_width` variable
in stages, adding `extra_space` only if the `distribute_space()` call
was successful. Annotate early scenario return for comprehensibility.
(environment::wrap_up_field): Cast return value of
`distribute_space()` to `void`; since that's what was happening
implicitly before, this is a trivial type-correctness change.
This change might finally solve the mystery of Savannah #44018.
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/bugs/?44018>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/
signature.asc
Description: PGP signature
