[apps-discuss] Review of draft-ietf-appsawg-json-pointer

[Paul Hoffman <[e-mail]>]

This document mostly looks fine.

- I do not understand the use of "-"; at least one use of it in an application should be given.

- If "-" is useful, wouldn't it also be useful to have something that indicates the (non-existant) member before the first array element?

- Editorial nit in section 4:
      *  characters that represent an unsigned base-10 integer value
         (possibly with leading zeros), making the new referenced value
         is the array element with the zero-based index identified by
         the token, or
That should either be "the new referenced value is the" or "making the new referenced value the", I believe.

- One of the security considerations is too limited.
   Note that JSON pointers can contain the NUL (Unicode U+0000)
   character, which may not be representable in all programming
   languages.
The same is true for all control characters, and for some programming languages, all non-ASCII characters. Proposed rewording:
   Note that JSON pointers can contain the NUL (Unicode U+0000)
   character, control characters, non-ASCII characters, and so
   on. These characters may not be representable in all programming
   languages.

--Paul Hoffman