[Uri-review] draft-larmouth-oid-iri-01 editorials

Alfred Hönes <ah@TR-Sys.de> Tue, 22 September 2009 17:46 UTC

Return-Path: <A.Hoenes@TR-Sys.de>
X-Original-To: uri-review@core3.amsl.com
Delivered-To: uri-review@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 4496F3A691D for <uri-review@core3.amsl.com>; Tue, 22 Sep 2009 10:46:53 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 2.368
X-Spam-Level: **
X-Spam-Status: No, score=2.368 tagged_above=-999 required=5 tests=[AWL=1.117, BAYES_00=-2.599, CHARSET_FARAWAY_HEADER=3.2, HELO_EQ_DE=0.35, MIME_8BIT_HEADER=0.3]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id kBu9Bhx9dd+K for <uri-review@core3.amsl.com>; Tue, 22 Sep 2009 10:46:51 -0700 (PDT)
Received: from TR-Sys.de (gateway.tr-sys.de [213.178.172.147]) by core3.amsl.com (Postfix) with ESMTP id 402013A6809 for <uri-review@ietf.org>; Tue, 22 Sep 2009 10:45:20 -0700 (PDT)
Received: from ZEUS.TR-Sys.de by w. with ESMTP ($Revision: 1.37.109.26 $/16.3.2) id AA101941366; Tue, 22 Sep 2009 19:42:46 +0200
Received: (from ah@localhost) by z.TR-Sys.de (8.9.3 (PHNE_25183)/8.7.3) id TAA01616; Tue, 22 Sep 2009 19:41:19 +0200 (MESZ)
From: Alfred Hönes <ah@TR-Sys.de>
Message-Id: <200909221741.TAA01616@TR-Sys.de>
To: j.larmouth@btinternet.com, olivier.dubuisson@orange-ftgroup.com
Date: Tue, 22 Sep 2009 19:41:18 +0200
X-Mailer: ELM [$Revision: 1.17.214.3 $]
Mime-Version: 1.0
Content-Type: text/plain; charset="hp-roman8"
Content-Transfer-Encoding: 8bit
Cc: uri-review@ietf.org
Subject: [Uri-review] draft-larmouth-oid-iri-01 editorials
X-BeenThere: uri-review@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: Proposed URI Schemes <uri-review.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/uri-review>, <mailto:uri-review-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/uri-review>
List-Post: <mailto:uri-review@ietf.org>
List-Help: <mailto:uri-review-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/uri-review>, <mailto:uri-review-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 22 Sep 2009 17:46:53 -0000

Hello,
I have followed up to the long expected update to your OID URI scheme
draft, draft-larmouth-oid-iri-01, and have a bunch of editorial
remarks.

(1)  Abstract -- spurious word duplication?

I did not really understand the second sentence, but my final
conclusion (based on the observation of more text duplication
in the draft) is that the last 3 words should simply be deleted:

   This internet draft defines the IRI/URI scheme for International
   Object Identifiers.  The syntax and semantics of the IRI is specified
   below using the International Object Identifier tree specified in
|  ITU-T X.660 International Object Identifiers.
---
   This internet draft defines the IRI/URI scheme for International
   Object Identifiers.  The syntax and semantics of the IRI is specified
   below using the International Object Identifier tree specified in
|  ITU-T X.660.


(2)  Section 1

(2a)  1st para

The draft says:

   The International Object Identifier tree [ITU-T X.660] provides a
|  hierarchically based identification scheme for objects/resources,
   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   using almost all Unicode/ISO 10646 characters to identify arcs in the
   tree.  [...]

The use of "based" in this context without designating the base does
not make proper sense to me.
Thus, again I suggest to simplify the language:

   The International Object Identifier tree [ITU-T X.660] provides a
|  hierarchical identification scheme for objects/resources, using
              ^^
   almost all Unicode/ISO 10646 characters to identify arcs in the tree.
   [...]

(2b)  3rd para

For improved readability, I suggest to add a comma as indicated
below; also, I'd insert a hyphen in "% encoding" (or spell it out:
"percent-encoding") :

   [...]
   For ITU-T purposes, it is essential to be able to use names in any
|  language as well as numbers, without resort to any %-encoding or
                                v                      ^
|  restriction to numeric values, where an IRI can be used.  [...]

(2c)  6th para

I suggest to enhance the readability by swapping detailed references.
The first instance is in this paragraph:

      ... [RFC3987], Section 3.1 ...
---
      ... Section 3.1 of [RFC3987] ...

A Similar change should be applied in the last para of section 2.1.

(2d)  7th and 8th para -- duplicate text

These two paragraphs are (up to the additional paragraph break)
a literal repetition of the 3rd para:

|  There has been in existence for some time a URN scheme of "urn:oid",
|  but this is based entirely on numeric identification of each arc.
|
|  For ITU-T purposes, it is essential to be able to use names in any
|  [...]
|  A separate new 'oid' IRI scheme was considered far preferable.

Please strike out both paragraphs!


(3)  Section 2.1

(3a)  1st para -- typo in citation (spurious spacing)

The ref tag for the ABNF RFC contains a spurious blank
[ proof of non-use of xml2rfc!  :-) ].
Please correct:   s/[RFC 5234]/[RFC5234]/

(3b)  1st para -- missref

The internal pointer has not been adjusted during the restructuring
of the memo.  Please correct:     s/2.5/2.2/

(3c)  ABNF spec

According to RFC 5234, the  '<...>'  notation for ABNF non-termoinals
should be used in the prose and avoided in the formal ABNF.
Therefore, I suggest to remove the angle braces, including the
clerical error introduced in the course of their introduction.
Furthermore, columnar alignment and some more indentation for the
ABNF 'code' would perhaps improve the readability and make that
 important part more outstanding.

|  <oidiri> = "oid:/" <firstarcid> <subsequentarcid>
|  <firstarcid> = <unicodelabel>
|  <subsequentarcid> = 0*("/" <unicodelabel&gt);
|  <unicodelabel> = 1*(<iunreserved>)
---
|     oidiri          = "oid:/" firstarcid subsequentarcid
|     firstarcid      = unicodelabel
|     subsequentarcid = 0*("/" unicodelabel);
|     unicodelabel    = 1*(iunreserved)

(3d)  last para

See (2c) and (3a) above:

     ... [RFC 3987], Section 3.1 ...
---
     ... Section 3.1 of [RFC3987] ...


(4)  Section 2.4

Please supply the missing trailing period at the end of the last para.


(5)  Section 4.1

(5a)  1st para

I suggest be be more explicit regarding the expected IANA action:

|  (The IANA Registration Template is in 4.2)
---
|  IANA is asked to register the IRI/URI scheme 'oid' on behalf of this
|  document, using the IANA Registration Template [RFC4395] presented in
|  Section 4.2.

(5b)  3rd para

As in (3a) above:    s/[RFC 2434]/[RFC2434]/


(6)  Section 4.2

I suggest to enhance the readability by use of indentation of the
continuation lines (for "Applications ..." and "Contact:".


(7)  Section 6

It looks like ref. [ORS] is not publicly available -- at least
it is (currently) not listed on
 <http://standards.ISO.ORG/ittf/PubliclyAvailableStandards/index.html>.

This might be an obstacle for wide-spread adoption, and will
definitely exclude its use as a Normative Reference.
However, it looks like this ref. can be demoted to Informative
without negatively affecting the specification of the URI scheme.
Yet, it would be interesting to learn how the indicated use of the
DNS for [ORS] is managed.


Kind regards,
  Alfred Hönes.

-- 

+------------------------+--------------------------------------------+
| TR-Sys Alfred Hoenes   |  Alfred Hoenes   Dipl.-Math., Dipl.-Phys.  |
| Gerlinger Strasse 12   |  Phone: (+49)7156/9635-0, Fax: -18         |
| D-71254  Ditzingen     |  E-Mail:  ah@TR-Sys.de                     |
+------------------------+--------------------------------------------+