Re: [netmod] WG Last Call: draft-ietf-netmod-artwork-folding-02

[Martin Bjorklund <[e-mail]>]

Hi,

I have reviewed draft-ietf-netmod-artwork-folding-02, and here are my
comments:


o  6

  Perhaps:

  OLD:

   begins at the first non-
   whitespace character on the following line.

  NEW:

   begins at the first character that is not a space character (' ')
   on the following line.


  The algorithm talks specifically about space (' ') rather than
  whitespace.


o  6.1

  s/is exists/exists/


o  6.1 / 6.2

  6.2 says (correctly!):

   It is RECOMMENDED for implementations to first attempt to fold
   content using the single backslash strategy and, only in the unlikely
   event that it cannot fold the input or the folding logic is unable to
   cope with a contingency occurring on the desired folding column, then
   fallback to the double backslash strategy.

  But 6.1 says about the Single Backslash Strategy:

   automation implementations are likely to encounter scenarios that
   will produce errors without special care


  So it 6.1 thinks it is likely that SBS won't work, but 6.2 says it
  is unlikely.  IMO 6.2 is correct - it is extremely unlikely that SBS
  won't work.


o  7.1.2 / 7.2

  I would prefer if the format is defined with descriptive text,
  rather than with an algorithm.  It is the end result that matters,
  not which algorithm an implementation uses to get to the result.

  I suggest the algorithm is moved to an appendix, and/or a sentence
  is added that explains that the algorithm is just an example.

  Also expand the descriptive text in 7.1.2; I think that the text in
  section 6 is probably enough.  However, there are some important
  details buried in the desciption of the algorithm; specifically the
  cases where SBS can't be used.


o  7.2.1

  I don't understand why there is a min limit of 46 characters for
  folding to work.  If the only reason is for the non-normative script
  to be able to center the header line, then I think this limitation
  should be removed.  (I would even prefer less flexibility in the
  header line syntax...)


o  7.2.1 / 7.2.2

  I don't think the text should assume that folding/unfolding is
  "automated".


o  7.2.1

  Perhaps add to bullet 1:

    If no such location can be found, then exit (this text content
    cannot be folded)


o  7.2.2

  s/maximum../maximum./


o  Appendix A

  Consider using the command "tempfile" instead of /tmp/wip*




/martin