Hi Martin,
Thanks for your review.
> 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.
Updated in my local copy.
> The algorithm talks specifically about space (' ') rather than
> whitespace.
True and, FWIW, this was the only instance of the word "whitespace" in the
document.
However, I wonder if there might be an issue lurking here. Already the
algorithm eliminates the potential for TAB, and implicitly eliminates NL and CR
through the use of the word "line", but I wonder if there are other characters
that we wish to skip over... e.g. vertical tab?
> o 6.1
>
> s/is exists/exists/
Fixed.
> 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.
s/are likely to/may/
> 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.
OLD:
This section describes the process for folding and unfolding long
lines when they are encountered in a single instance of text content.
It is assumed that another process inserts/extracts the individual
text content instances to/from an Internet-Draft or RFC. For
example, the `xiax` utility [xiax] does this.
NEW:
This section describes a process for folding and unfolding long lines
when they are encountered in text content.
The steps are complete, but implementations MAY achieve the same
result in other ways.
When a larger document contains multiple instances of text content
that may need to be folded or unfolded, it is assumed that another
process inserts/extracts the individual text content instances to/
from the larger document prior to utilizing the algorithms described
in this section. For example, the `xiax` utility [xiax] does this.
> 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.
I looked at this for a little while, but didn't see how it could be improved.
Can you provide some text?
> 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...)
This is because we never defined how to handle folding the header itself. I
wrote about this a while back and no-one seemed bothered by the limitation.
The effort/value ration isn't there. The need to fold less than 69-characters
is unlikely, and less than 46-characters seems even more so.
> o 7.2.1 / 7.2.2
>
> I don't think the text should assume that folding/unfolding is
> "automated".
Both sections clearly state that authors may do the equivalent manually, or do
you mean that the word "automated" in these sections isn't adding much value
and could/should be removed?
> o 7.2.1
>
> Perhaps add to bullet 1:
>
> If no such location can be found, then exit (this text content
> cannot be folded)
Added in my local copy.
>
>
> o 7.2.2
>
> s/maximum../maximum./
Fixed (in both locations)
> o Appendix A
>
> Consider using the command "tempfile" instead of /tmp/wip*
Used `mktemp -d` instead.
Kent // author
_______________________________________________
netmod mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/netmod
