Update of bug #60233 (group groff):

                  Status:               Need Info => None
                 Summary: doc/groff.texi: ambiguity in .tkf documentation =>
[PATCH] doc/groff.texi: ambiguity in .tkf documentation

    _______________________________________________________

Follow-up Comment #5:

I've attached a patch for feedback.  It encompasses points made in all the
previous comments.

In addition to the content changes, this patch rebreaks lines following
current convention.[1]  Recent commits seem to refer to this as breaking lines
"in a roff-friendly way," but it's really for the benefit of diff, not roff.
(It is following the convention used in man pages, so "in a
man-page-conventional way" would also be accurate.  But a major reason for the
man page convention is how it clarifies content changes in diffs by omitting
consequent reflowing, so I feel diff is at the root of the rationale for it.)

Accordingly, I first rebroke the existing lines, then edited the wording, then
ran a diff between those two steps to show the content changes my patch
makes:

 Enable track kerning for font@tie{}@var{f}.
 If the current font is@tie{}@var{f}
-the width of every glyph
+the width taken up by every glyph
 is increased by an amount
 between @var{n1} and @var{n2}
 (@var{n1}, @var{n2} can be negative);
-if the current type size is less than or equal to @var{s1}
+if the current type size is less than or equal to @var{s1},
+or if @var{s2} is less than or equal to @var{s1},
 the width is increased by @var{n1};
-if it is greater than or equal to @var{s2}
+if the current type size is greater than or equal to @var{s2},
 the width is increased by @var{n2};
-if the type size is greater than or equal to @var{s1}
-and less than or equal to @var{s2}
+otherwise,
 the increase in width
 is a linear function of the type size.

[1] My idea of "current convention" is merely to break lines between sentences
and at logical places within them.  If a more specific convention has been
followed thus far (e.g. always break before a conjunction), this patch should
be reflowed again to reflect that.

(file #58528)

    _______________________________________________________

Additional Item Attachment:

Name: 60233.patch                    Size: 1.5KiB
    <https://file.savannah.gnu.org/file/60233.patch?file_id=58528>


    AGPL NOTICE

These attachments are served by Savane. You can download the corresponding
source code of Savane at
https://savannah.gnu.org/source/savane-c36938be85ff6c1b727bc7dd7fd30e48f9142870.tar.gz


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?60233>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/

Attachment: signature.asc
Description: PGP signature

Reply via email to