draft-ietf-idnabis-tables-06a.txt (Substantive Problem 1)

[Patrik Fältström]

On 21 jul 2009, at 02.20, Kenneth Whistler wrote:

> Patrik,
>
>> http://stupid.domain.name/stuff/draft-ietf-idnabis-tables-06a.txt
>
> Now for some substantive problems, particularly in Appendix A.
>
> First problem: 2nd paragraph of Appendix A.
>
> "Script(cp) return the name of the Script cp is of, while
> Class(cp) return the Class."
>
> This sentence is full of problems.
>
> 1. Class(cp) is nowhere used in Appendix A, so it should be
> deleted from this paragraph describing the pseudo code.
> It may be a holdover from before the correction to
> use the full property name Canonical_Combining_Class(cp).
> If that is the intent, then it should instead describe
> that particular property, rather than just an unspecified "Class".

Changed to Canonical_Combining_Class(cp) (but (2) below override).

> 2. Script(cp) is not specifically tied to any specification.
> As stated right now people might assume it relies on
> the Unicode Standard's Script property, but they might
> also decide to determine script some other way, which
> would make the specifications of the context rules
> indeterminate. I suggest tightening this up to be
> very specific about this referring to the Unicode
> Script property, if that is indeed the intent here.
>
> So a potentially much more rigorous rewrite would be:
>
> "Script(cp) returns the value of the Unicode Script property,
> as defined in Scripts.txt in the Unicode Character Database.
> Canonical_Combining_Class(cp) returns the value of
> the Unicode Canonical_Combining_Class property,
> as defined in UnicodeData.txt in the Unicode Character
> Database."
>
> If stated that way, implementers will get determinant
> values for the functions, and will all get the same
> results when they attempt to evaluate these context rules.

Much better! Changed!

> Editorial: I noticed another instance of "FALSE" that should
> change to "False" two sentences earlier in that same paragraph.


Fixed!

Many thanks Ken!

    paf