Hi Barry, I addressed all these issues in the latest update:
https://tools.ietf.org/html/draft-ietf-netmod-artwork-folding-09 <https://tools.ietf.org/html/draft-ietf-netmod-artwork-folding-09> For the "2nd line needing to be blank" issue, I also updated the code to test for it. Thanks, Kent > On Aug 22, 2019, at 12:41 AM, Barry Leiba via Datatracker <[email protected]> > wrote: > > Barry Leiba has entered the following ballot position for > draft-ietf-netmod-artwork-folding-08: No Objection > > When responding, please keep the subject line intact and reply to all > email addresses included in the To and CC lines. (Feel free to cut this > introductory paragraph, however.) > > > Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html > for more information about IESG DISCUSS and COMMENT positions. > > > The document, along with other ballot positions, can be found here: > https://datatracker.ietf.org/doc/draft-ietf-netmod-artwork-folding/ > > > > ---------------------------------------------------------------------- > COMMENT: > ---------------------------------------------------------------------- > > — Section 4.1 — > > I find the BCP 14 “SHOULD” in this section to be odd, and would lower-case > them. > > When needed, this effort again > SHOULD be automated to reduce effort and errors resulting from manual > processing. > > This sentence is really awkward: “when needed”, the use of “effort” twice, and > the uncertainty of whether the clause “resulting from manual processing” > applies to both effort and errors, or only to the latter. I would say it this > way: > > NEW > This work should also be automated to reduce the effort and to reduce errors > resulting from manual processing. END > > — Section 6 — > > assumes that the continuation begins at the character that is > not a space character (' ') on the following line. > > Should be “at the first character”. > > — Section 7.1.1 — > > The second line is a blank line. > > The code in the appendix generates an *empty* line (no text). Is that what > you > mean by “blank line”? Will a line that contains only space characters > (*looks* > the same) work also? The code in the appendix appears to discard the second > line without checking its content at all. I think you should be clearer about > what qualifies as a “blank line”. (This also applies to Section 8.1.1.) > > — Section 7.2.1 — > > If this text content needs to and can be folded, insert the header > described in Section 7.1.1, ensuring that any additional printable > characters surrounding the header does not result in a line exceeding > the desired maximum. > > Should be “do not result” (to match the plural “printable characters”). > > > _______________________________________________ > netmod mailing list > [email protected] > https://www.ietf.org/mailman/listinfo/netmod
_______________________________________________ netmod mailing list [email protected] https://www.ietf.org/mailman/listinfo/netmod
