Re: [netmod] Barry Leiba's No Objection on draft-ietf-netmod-artwork-folding-08: (with COMMENT)

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 <noreply@ietf.org> 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
> netmod@ietf.org
> https://www.ietf.org/mailman/listinfo/netmod