Re: [css-syntax] Removed <unicode-range-token>, please review

On Thu, Nov 13, 2014 at 9:03 PM, Tab Atkins Jr. <jackalmage@gmail.com> wrote:
> On Thu, Nov 13, 2014 at 7:42 PM, John Daggett <jdaggett@mozilla.com> wrote:
>> Tab Atkins wrote:
>>> In section 7.1:
>>>
>>> To determine what codepoints the <urange> represents:
>>>
>>> 1. If end value is greater than the maximum allowed code point, set
>>>    it to the maximum allowed code point.
>>
>> No, this is invalid syntax and the descriptor defintion should be rejected.
>>
>>> 2. If start value is greater than end value, the <urange>
>>>    represents an empty range of codepoints.
>>
>> Ditto. Plus you introduce serialization problems by allowing "empty
>> range". I went through this with the Fonts spec, that's why it isn't
>> defined this way. :)
>
> Sure, I can just make those both invalid <urange> rather than empty or
> truncated.  Not a problem.

And done.  Now those errors make the <urange> invalid and a syntax
error, rather than triggering error-correction.

~TJ

Received on Friday, 14 November 2014 19:21:14 UTC