Re: [Ltru] Fwd: draft-davis-t-langtag-ext

Mykyta Yevstifeyev <evnikita2@gmail.com> Thu, 07 July 2011 03:55 UTC

Return-Path: <evnikita2@gmail.com>
X-Original-To: ltru@ietfa.amsl.com
Delivered-To: ltru@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E9DDF21F8A85 for <ltru@ietfa.amsl.com>; Wed, 6 Jul 2011 20:55:16 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.411
X-Spam-Level:
X-Spam-Status: No, score=-2.411 tagged_above=-999 required=5 tests=[AWL=-1.113, BAYES_00=-2.599, HTML_MESSAGE=0.001, MANGLED_TEXT=2.3, RCVD_IN_DNSWL_LOW=-1]
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 2NMnUj9mGb1Q for <ltru@ietfa.amsl.com>; Wed, 6 Jul 2011 20:55:16 -0700 (PDT)
Received: from mail-fx0-f54.google.com (mail-fx0-f54.google.com [209.85.161.54]) by ietfa.amsl.com (Postfix) with ESMTP id E568121F8621 for <ltru@ietf.org>; Wed, 6 Jul 2011 20:55:15 -0700 (PDT)
Received: by fxe4 with SMTP id 4so909380fxe.27 for <ltru@ietf.org>; Wed, 06 Jul 2011 20:55:15 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type; bh=tzEhVaNLIGl/cAQZKuGL+/6Bb+3jjZIPKph4HyExlRs=; b=lxCcwIvD3Mu4l7ux7kJHlh+5J3UUdpBWRr90nRAwDawvC02WnIM8Y23tFQsZNrQHMy xM92wdyuxzTKeYHdaHBFi4FFrdIYNypgS935TzrGzzLFLS698tL+kXk5X+YU1VM7bgCh OXJSdB+tV/CPM+E/1KmYY9ftaVIdsj7NUFZ0c=
Received: by 10.223.76.219 with SMTP id d27mr514397fak.87.1310010913402; Wed, 06 Jul 2011 20:55:13 -0700 (PDT)
Received: from [127.0.0.1] ([195.191.104.224]) by mx.google.com with ESMTPS id b3sm6436722fao.44.2011.07.06.20.55.11 (version=SSLv3 cipher=OTHER); Wed, 06 Jul 2011 20:55:12 -0700 (PDT)
Message-ID: <4E152E4F.9070203@gmail.com>
Date: Thu, 07 Jul 2011 06:55:59 +0300
From: Mykyta Yevstifeyev <evnikita2@gmail.com>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; ru; rv:1.9.2.18) Gecko/20110616 Thunderbird/3.1.11
MIME-Version: 1.0
To: ltru@ietf.org
References: <4E14F473.6030101@qualcomm.com>
In-Reply-To: <4E14F473.6030101@qualcomm.com>
Content-Type: multipart/alternative; boundary="------------060509090204040700060801"
Cc: Pete Resnick <presnick@qualcomm.com>
Subject: Re: [Ltru] Fwd: draft-davis-t-langtag-ext
X-BeenThere: ltru@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Language Tag Registry Update working group discussion list <ltru.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ltru>, <mailto:ltru-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ltru>
List-Post: <mailto:ltru@ietf.org>
List-Help: <mailto:ltru-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ltru>, <mailto:ltru-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 07 Jul 2011 03:55:17 -0000

Hello,

I've identified the following issue in the draft.

Section 2.2 says:

>     The subtags in the 't' extension are of the following form:
>
>       +--------+-------------------------+----------------------------+
>       | Label  | ABNF                    | Comment                    |
>       +--------+-------------------------+----------------------------+
>       | t_ext= | "t"                     | Extension                  |
>       |        | ("-" lang *("-" field)  | Source + optional field(s) |
>       |        | / 1*("-" field))        | Field(s) only (no source)  |
>       | lang=  | language                | [BCP47  <http://tools.ietf.org/html/draft-davis-t-langtag-ext-01#ref-BCP47>], with restrictions |
>       |        | ["-" script]            |                            |
>       |        | ["-" region]            |                            |
>       |        | *("-" variant)          |                            |
>       | field= | sep 1*("-" 3*8alphanum) | With restrictions          |
>       | sep=   | 1ALPHA 1DIGIT           | Subtag separators          |
>       +--------+-------------------------+----------------------------+

I should note that, first of all, reference to RFC 5234 is missing; 
moreover, and this is more important, making the ABNF definition in the 
form of table makes such definition an invalid one, in terms of RFC 
5234.  Also, there are a number of ABNF nits here.  So, please consider 
changing this to:

>     The subtags in the 't' extension are of the following form, defined
>     using ABNF [RFC5234] in<t-ext>  rule:
>
>       t-ext    = "t" ("-" lang *("-" field) / 1*("-" field))
>       lang     = langtag
>       field    = sep 1*("-" 3*8alphanum)
>       sep      = ALPHA DIGIT
>       alphanum = ALPHA / DIGIT
>
>     where<langta>  rule is specified in BCP 47 [BCP47],<ALPHA>  and<DIGIT>
>     rules - in RFC 5234 [RFC5234].
Also, the minors comments on references.  Reference to BCP 47 should 
include both references to RFC 5646 and RFC 4647, like:
>     [BCP47]    Phillips, A. and M. Davis, "Matching of Language Tags",
>                BCP 47, RFC 4647, September 2006.
>
>                Phillips, A., Ed., and M. Davis, Ed., "Tags for Identifying
>                Languages", BCP 47, RFC 5646, September 2009.
...and, referencing UTS 35 you shouldn't reference specific parts of the 
document; this should be done in the text.  Finally, I don't see where 
[US-ASCII] is used in the text.

Thanks,
Mykyta Yevstifeyev

07.07.2011 2:49, Pete Resnick wrote:
> Most of the people on the ietf-languages list are probably on the 
> ltru@ietf.org list as well, but I wanted to confirm that everyone got 
> a chance to review this before it proceeded to the IESG. Please have a 
> look at the ltru archive 
> <http://www.ietf.org/mail-archive/web/ltru/current/maillist.html> and 
> send any comments to the ltru@ietf.org list since that's where 
> discussion seems to be taking place.
>
> Thanks.
>
> pr