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
_______________________________________________
netmod mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/netmod
