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

[Philipp Kewisch <[e-mail]>]

Ah ok, just listing the names again in a comment in the ABNF is easy enough, thats totally fine with me.

Thanks,
Philipp


> On Mar 26, 2014, at 14:22, Robert Sparks <[e-mail]> wrote:
> 
> I'll let you work that out with Pete.
> 
> If you do decide to list them, I wouldn't repeat their definition.
> The important thing is that you verified there wasn't anything left dangling.
> Thanks for taking the time to go through that again.
> 
> RjS
> 
>> On 3/26/14, 5:03 AM, Philipp Kewisch wrote:
>>> 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
>