Follow-up Comment #6, bug #67939 (group groff): [comment #4 comment #4:] > [comment #3 comment #3:] >> Thus an updated patch is in the attachment. > > Thanks, Bjarni. This new patch addresses one of the three concerns I raised > in comment #1. To recap, the other two were: > * Only a subset of sed works across all implementations. What steps have you > taken to verify that your sed is portable?
None.
Solutions to such issues exist already. The people at the other
end are responsible for their application.
> * Alternately, Deri and I both offered sed-free methods to accomplish the
> same thing.
Sed is the simplest and an appropriate tool.
> * The patch contains another set of changes that appear to be unrelated to
> the update in how the page count is attained. These should either be
> separate patches, or you should justify why they should remain in the same
> one.
The changes are not caused by a classical bug, but are a revision, to
make better choices concerning speed, generality, usability, and
maintenance.
I see no valid reason to make two or more tickets for such changes as
they are confined within a single (small) file, that does not interact with
other files (is labour-saving, economical).
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/bugs/?67939>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/
signature.asc
Description: PGP signature
