Re: Last Call: <draft-ietf-appsawg-json-patch-08.txt> (JSON Patch) to Proposed Standard

Mark Nottingham <mnot@mnot.net> Mon, 17 December 2012 04:19 UTC

Return-Path: <mnot@mnot.net>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id DD5AC21F8518 for <ietf@ietfa.amsl.com>; Sun, 16 Dec 2012 20:19:23 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -103.829
X-Spam-Level:
X-Spam-Status: No, score=-103.829 tagged_above=-999 required=5 tests=[AWL=-1.230, BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([64.170.98.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id K09qlvywAoz0 for <ietf@ietfa.amsl.com>; Sun, 16 Dec 2012 20:19:22 -0800 (PST)
Received: from mxout-08.mxes.net (mxout-08.mxes.net [216.86.168.183]) by ietfa.amsl.com (Postfix) with ESMTP id C592D21F850E for <ietf@ietf.org>; Sun, 16 Dec 2012 20:19:22 -0800 (PST)
Received: from [192.168.1.80] (unknown [118.209.33.170]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (No client certificate requested) by smtp.mxes.net (Postfix) with ESMTPSA id 6FCA6509B8; Sun, 16 Dec 2012 23:19:21 -0500 (EST)
Content-Type: text/plain; charset="us-ascii"
Mime-Version: 1.0 (Mac OS X Mail 6.2 \(1499\))
Subject: Re: Last Call: <draft-ietf-appsawg-json-patch-08.txt> (JSON Patch) to Proposed Standard
From: Mark Nottingham <mnot@mnot.net>
In-Reply-To: <p1sj78lfq4.fsf@d72933.na.sas.com>
Date: Mon, 17 Dec 2012 15:19:17 +1100
Content-Transfer-Encoding: quoted-printable
Message-Id: <BB1AAE6C-B3B5-418D-A367-D22366EDE86A@mnot.net>
References: <p1sj78lfq4.fsf@d72933.na.sas.com>
To: "David J. Biesack" <David.Biesack@sas.com>
X-Mailer: Apple Mail (2.1499)
Cc: ietf@ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ietf>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 17 Dec 2012 04:19:24 -0000

On 15/12/2012, at 4:13 AM, David J. Biesack <David.Biesack@sas.com> wrote:

> 
> "4.1 add" and "4.2 remove" refers to "specified index", but I do not see name of the index member specified in the draft, or how to represent it.
> (Other sections are more explicit, i.e. "from" for the "move" op.)
> 
> i.e. "index" as in 
> 
>  "index" : integer 
> 
> The phrasing 
> 
>   If the "-" character is used to index the end of the array, this has the effect of appending the value to the array.
> 
> is not clear. Where does this "-" character appear? Does one use just
> 
>  "index" : "-" 
> 
> in which case the spec should be 
> 
>  If the "-" string is used ...
> 
> If instead you mean a negative integer, i.e.
> 
>  "index" : negative-integer
> 
> and not just "-", does the spec interpret all negative integers to indicate appending to the end?

I think a reference to the JSON-Pointer spec at an appropriate place here will help readers.

Thanks,



--
Mark Nottingham   http://www.mnot.net/