Re: [Json] New JSON Text Sequence I-D submitted

"Manger, James" <James.H.Manger@team.telstra.com> Tue, 29 April 2014 00:41 UTC

From: "Manger, James" <James.H.Manger@team.telstra.com>
To: Nico Williams <nico@cryptonector.com>, "json@ietf.org" <json@ietf.org>
Date: Tue, 29 Apr 2014 10:41:23 +1000
Thread-Topic: [Json] New JSON Text Sequence I-D submitted
Thread-Index: Ac9jF8HYObffFIbBS+aI0nC/1vm5bwAJbbJA
Message-ID: <255B9BB34FB7D647A506DC292726F6E115456D2132@WSMSG3153V.srv.dir.telstra.com>
References: <CAK3OfOgLJDEFK_K16WhX9dm21a25r7Wr4EmZcqbGa7CLdqYWGw@mail.gmail.com>
In-Reply-To: <CAK3OfOgLJDEFK_K16WhX9dm21a25r7Wr4EmZcqbGa7CLdqYWGw@mail.gmail.com>
Accept-Language: en-US, en-AU
Content-Language: en-US
acceptlanguage: en-US, en-AU
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: http://mailarchive.ietf.org/arch/msg/json/W875G4VmWzQj0D8IbPL5k2t7IZQ
Subject: Re: [Json] New JSON Text Sequence I-D submitted
Precedence: list

> https://datatracker.ietf.org/doc/draft-ietf-json-text-sequence/

Short & simple — looks good.

1.
One missing feature is that a file containing only whitespace should be a valid JSON text sequence. Suggested changes:

FROM:  JSON-sequence = *(JSON-text 1*(ws))
TO  :  JSON-sequence = *(ws) *(JSON-text 1*(ws))

P.S. Do you need the brackets in *(ws)?

FROM:  A JSON text sequence is a sequence of JSON texts
TO  :  A JSON text sequence is a sequence of zero, one, or more JSON texts


2.
We can't have a <ws> ABNF production that is different from RFC7159 JSON. ws in JSON is unfortunate, but we have to live with it.
json-seq:  ws =    %x20 / %x09 / %x0A / %x0D
json    :  ws = *( %x20 / %x09 / %x0A / %x0D )

I would be tempted to write the ABNF using <value>, instead of <JSON-text>.

  JSON-sequence = *whitespace *(value 1*whitespace)
  whitespace = %x20 / %x09 / %x0A / %x0D
  value = <given by RFC7159>

--
James Manger

[Json] New JSON Text Sequence I-D submitted Nico Williams
Re: [Json] New JSON Text Sequence I-D submitted Manger, James
Re: [Json] New JSON Text Sequence I-D submitted Martin J. Dürst
Re: [Json] New JSON Text Sequence I-D submitted Nico Williams