Re: [Jcardcal] Genart LC review: draft-ietf-jcardcal-jcal-09

[Philipp Kewisch <[e-mail]>]

On 3/21/14, 2:16 PM, Robert Sparks wrote:
>
> On 3/20/14, 6:26 PM, Philipp Kewisch wrote:
>> On 3/20/14, 11:20 PM, Robert Sparks wrote:
>>>> value-separator is defined in section 2 of rfc 7159.
>>> Provide that reference at the point it's used in the document please.
>> Just above the ABNF in the introduction to the section, I write "ABNF
>> Symbols not described here are taken from [RFC7159]". Isn't that
>> sufficient?
> I missed it. Others are likely to as well.
> Building an explicit list of what symbols you're counting on from that
> document
> would let you check that you haven't missed defining any others.
> Putting the list
> in the document will help others later find where to go when they're
> missing the
> symbol by using grep on this document.
> (You should have that list already from the ABNF verification step you
> performed).
>>
>> Philipp
>
That list is quite a bit, I personally think the note at the beginning
of the section should be sufficient. Don't you think most implementers
will assume these rules belong to JSON and re-read the section if they
are really looking for why the rule is missing? If you think this is a
blocking issue, I'm happy to include it though.

; These symbols originate from RFC7159 and are only
begin-array     = ws %x5B ws  ; [ left square bracket
begin-object    = ws %x7B ws  ; { left curly bracket
end-array       = ws %x5D ws  ; ] right square bracket
end-object      = ws %x7D ws  ; } right curly bracket
name-separator  = ws %x3A ws  ; : colon
value-separator = ws %x2C ws  ; , comma
string = quotation-mark *char quotation-mark
quotation-mark = %x22      ; " 
ws = *( %x20 / %x09 / %x0A / %x0D ) ; Space, Horizontal tab, line feed
                                    ; or new line, carriage return
false = %x66.61.6c.73.65   ; false
true  = %x74.72.75.65      ; true
number = [ minus ] int [ frac ] [ exp ]
decimal-point = %x2E       ; . 
digit1-9 = %x31-39         ; 1-9
e = %x65 / %x45            ; e E
exp = e [ minus / plus ] 1*DIGIT
frac = decimal-point 1*DIGIT
int = zero / ( digit1-9 *DIGIT )
minus = %x2D               ; - 
plus = %x2B                ; + 
zero = %x30                ; 0