Re: [CCAMP] 答复: RtgDir review: draft-ietf-ccamp-lmp-behavior-negotiation-09

Lou Berger <lberger@labn.net> Tue, 22 January 2013 14:25 UTC

Return-Path: <lberger@labn.net>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 6FAB121F8937 for <ccamp@ietfa.amsl.com>; Tue, 22 Jan 2013 06:25:48 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -101.006
X-Spam-Level:
X-Spam-Status: No, score=-101.006 tagged_above=-999 required=5 tests=[AWL=0.807, BAYES_00=-2.599, IP_NOT_FRIENDLY=0.334, MIME_8BIT_HEADER=0.3, SARE_SUB_ENC_UTF8=0.152, 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 gepZnxtMOvfQ for <ccamp@ietfa.amsl.com>; Tue, 22 Jan 2013 06:25:47 -0800 (PST)
Received: from oproxy7-pub.bluehost.com (oproxy7-pub.bluehost.com [67.222.55.9]) by ietfa.amsl.com (Postfix) with SMTP id 92D7321F87FB for <ccamp@ietf.org>; Tue, 22 Jan 2013 06:25:47 -0800 (PST)
Received: (qmail 13441 invoked by uid 0); 22 Jan 2013 14:25:24 -0000
Received: from unknown (HELO box313.bluehost.com) (69.89.31.113) by oproxy7.bluehost.com with SMTP; 22 Jan 2013 14:25:24 -0000
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=labn.net; s=default; h=Content-Transfer-Encoding:Content-Type:In-Reply-To:References:Subject:CC:To:MIME-Version:From:Date:Message-ID; bh=UrxBi/j/0MewN3J9XQLWguN2GzhRnxkI4Wd42wYbJRs=; b=OKOVZisOgxDJH8I7WPw9PVjBjdxVCwtyk5nbdACsjR6FddiQkwhFenusj9RtqeHUCC777OgAwxSnVvgdTtbeD9vTZBEOp/BB2H0tBgs157QGMTrg6nRF77ncq0dGU07l;
Received: from box313.bluehost.com ([69.89.31.113]:46948 helo=[127.0.0.1]) by box313.bluehost.com with esmtpa (Exim 4.80) (envelope-from <lberger@labn.net>) id 1Txemy-0003aV-8d; Tue, 22 Jan 2013 07:25:24 -0700
Message-ID: <50FEA15E.6040801@labn.net>
Date: Tue, 22 Jan 2013 09:25:34 -0500
From: Lou Berger <lberger@labn.net>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130107 Thunderbird/17.0.2
MIME-Version: 1.0
To: Julien Meuric <julien.meuric@orange.com>
References: <50F97FAC.3010903@orange.com> <92A1F6CF27D54D4DA5364E59D892A02A3884F3F1@szxeml555-mbx.china.huawei.com> <50FE56F5.8040806@orange.com>
In-Reply-To: <50FE56F5.8040806@orange.com>
X-Enigmail-Version: 1.5
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8bit
X-Identified-User: {1038:box313.bluehost.com:labnmobi:labn.net} {sentby:smtp auth 69.89.31.113 authed with lberger@labn.net}
Cc: "draft-ietf-ccamp-lmp-behavior-negotiation.all@tools.ietf.org" <draft-ietf-ccamp-lmp-behavior-negotiation.all@tools.ietf.org>, "rtg-dir@ietf.org" <rtg-dir@ietf.org>, "ccamp@ietf.org" <ccamp@ietf.org>, "rtg-ads@tools.ietf.org" <rtg-ads@tools.ietf.org>
Subject: Re: [CCAMP] 答复: RtgDir review: draft-ietf-ccamp-lmp-behavior-negotiation-09
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: Discussion list for the CCAMP working group <ccamp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ccamp>, <mailto:ccamp-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ccamp>
List-Post: <mailto:ccamp@ietf.org>
List-Help: <mailto:ccamp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ccamp>, <mailto:ccamp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 22 Jan 2013 14:25:48 -0000

Julien/Dan,

Some comments below (as coauthor).

On 1/22/2013 4:08 AM, Julien Meuric wrote:
> Hi Dan.
> 
> You're welcome.
> 
> I've added "very" on purpose, to emphasize what document the sentence 
> was referring to (i.e. the RFC mentioned at the beginning of that very 
> same sentence, not at the end of the previous one). It was just a 
> suggestion, it isn't mandatory to keep it if you feel it's more 
> confusing than clarifying. You may also check with a native English 
> speaker (e.g. the RFC Editor), who I'm not.
> 

IMO the use of "very" is 100% stylistic.  I don't think it causes any
harm or provides any substantive benefit.  It also doesn't matter to me
if it's included or not...

> Regards,
> 
> Julien
> 
> 
> Le 22/01/2013 08:21, Lidan (Dan) a écrit :
>> Julien,
>>
>> Thanks for the review, we will update the draft according to your comments.
>>
>> One comment needs your clarification:
>> - s/functions defined in that document/functions specified in that very document/  [repetition of "defined"]
>> For adding "very", is it a typo?
>>
>> For the reference [LMP-TEST], we will check with the authors of draft-cecczhang-ccamp-gmpls-g709v3-lmp, to see what's their intention.

Rather than reference an individual draft, my suggestion is to just drop
the whole sentence.  It doesn't really add much in any case.

The other comments all look good to me.  Much thanks!

Lou

>>
>> Thank you again!
>>
>> Dan
>>
>> -----邮件原件-----
>> 发件人: ccamp-bounces@ietf.org [mailto:ccamp-bounces@ietf.org] 代表 Julien Meuric
>>
>> Hello,
>>
>> I have been selected as the Routing Directorate reviewer for this draft.
>> The Routing Directorate seeks to review all routing or routing-related
>> drafts as they pass through IETF last call and IESG review. The purpose
>> of the review is to provide assistance to the Routing ADs. For more
>> information about the Routing Directorate, please see
>> http://www.ietf.org/iesg/directorate/routing.html
>>
>> Although these comments are primarily for the use of the Routing ADs, it
>> would be helpful if you could consider them along with any other IETF
>> Last Call comments that you receive, and strive to resolve them through
>> discussion or by updating the draft.
>>
>> Document: draft-ietf-ccamp-lmp-behavior-negotiation-09
>> Reviewer: Julien Meuric
>> Review Date: 18 January 2013
>> IETF LC End Date: 21 January 2013
>> Intended Status: Standards Track
>>
>> *Summary:*
>> This document is basically ready for publication, but has nits that
>> should be considered prior to it.
>>
>> *Comments:*
>> This document is clearly written and easy to understand. The defined
>> mechanism is simple and well specified.
>>
>> *Nits:*
>> Abstract
>> -Instead of "GMPLS networks", "GMPLS-controlled networks" reads more
>> accurate to me. It would also align on the phrase used in the
>> introduction. The resulting "Generalized Multiprotocol Label Switching
>> (GMPLS)-controlled networks" may be checked with the RFC Editor.
>>
>> Introduction
>> - s/LMP node/LMP-capable node/
>> - s/functions defined in that document/functions specified in that very
>> document/  [repetition of "defined"]
>> - s/the message types from/the types from/  [repetition of "message"]
>>
>> Section 2.2
>> - s/MAY include, HelloConfig/MAY include HelloConfig/
>>
>> Section 3.1
>> - s/number of MBZ bits field/number of bits in MBZ field/
>>
>> Section 4
>> - s/[RFC4202] defined/[RFC4202]-defined/
>>
>> Section 9.2
>> - I tend to think that [LMP TEST] should now reference
>> draft-cecczhang-ccamp-gmpls-g709v3-lmp
>>
>>
>> Enjoy the week-end,
>>
>> Julien
>>
>> _______________________________________________
>> CCAMP mailing list
>> CCAMP@ietf.org
>> https://www.ietf.org/mailman/listinfo/ccamp
> 
> 
> 
> 
>