[Cbor] draft-ietf-cbor-7049bis-05 comments

Jim Schaad <ietf@augustcellars.com> Thu, 17 January 2019 00:58 UTC

Return-Path: <ietf@augustcellars.com>
X-Original-To: cbor@ietfa.amsl.com
Delivered-To: cbor@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 5F171131236; Wed, 16 Jan 2019 16:58:59 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id aGgTeXbIEtP7; Wed, 16 Jan 2019 16:58:56 -0800 (PST)
Received: from mail2.augustcellars.com (augustcellars.com [50.45.239.150]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id C2095126F72; Wed, 16 Jan 2019 16:58:53 -0800 (PST)
Received: from Jude (73.180.8.170) by mail2.augustcellars.com (192.168.0.56) with Microsoft SMTP Server (TLS) id 15.0.1395.4; Wed, 16 Jan 2019 16:58:47 -0800
From: Jim Schaad <ietf@augustcellars.com>
To: draft-ietf-cbor-7049bis@ietf.org
CC: cbor@ietf.org
Date: Wed, 16 Jan 2019 16:58:44 -0800
Message-ID: <045001d4adff$ccfdd540$66f97fc0$@augustcellars.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
X-Mailer: Microsoft Outlook 16.0
Content-Language: en-us
Thread-Index: AdSt/eqtWROsJX2EQiuoLVyh+ZNxDw==
X-Originating-IP: [73.180.8.170]
Archived-At: <https://mailarchive.ietf.org/arch/msg/cbor/rbGc2MaJIISLcuYK4MlVnOTDzhs>
Subject: [Cbor] draft-ietf-cbor-7049bis-05 comments
X-BeenThere: cbor@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "Concise Binary Object Representation \(CBOR\)" <cbor.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/cbor>, <mailto:cbor-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/cbor/>
List-Post: <mailto:cbor@ietf.org>
List-Help: <mailto:cbor-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/cbor>, <mailto:cbor-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 17 Jan 2019 00:58:59 -0000

These two issues are based on reading the area around the diffs and not on a
full reading of the document.  I plan to do that some time in the next 3
weeks.

1.  Having pulled the fact that a start for indefinite encoding is no longer
before the definition of "break", I kept finding this a pain as I wanted to
know how to start at the same time that finish is being documented.  I do
realize that this is in both of the following sections but still found it
jarring.

2.  I found the entire paragraph on how ending of False, etc. very
confusing.  I especially did not know what the rule was going to be for
dealing with simple values from 32 and up as these are two-byte sequences.
I think the entire paragraph could be more simply specified as saying that
all simple values must be encoded in the minimal number of bytes and say
that a decode MUST treat encodings which are not of minimal length as being
errors.  I think do not have the problem of how to encode simple value 32 as
0xf820 is well understood as being the one and only encoding.

3.  I am not sure how to react to the following potentially contradictory
statements:
    * The data model says that bignum and integer encodings are distinct for
the generic data model.  (para #1 section 3.4.3)
   * The preferred encoding for small bignum values is as integers and not
as bignum values.  (para #2)
   This means that if for some reason I want to use bignums as the keys then
I need to violate one of those two suggestions.  I don't think that this
would be a good idea but is may want to be clarified.

4.  The re-write of the last paragraph in section 3.4.6.2 had me looking for
the sentence that dealt with the third tag for padding purposes.  This was
not really discordant in the old text because only one item was dealt with,
but for the new text two out of three items are dealt with and thus it
appears that one of them is missing.

Jim