Re: [Gen-art] review of draft-ietf-mmusic-sdp-g723-g729-04.txt

"Muthu Arul Mozhi Perumal (mperumal)" <mperumal@cisco.com> Wed, 29 January 2014 05:07 UTC

Return-Path: <mperumal@cisco.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 9189C1A0280 for <gen-art@ietfa.amsl.com>; Tue, 28 Jan 2014 21:07:29 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -10.036
X-Spam-Level:
X-Spam-Status: No, score=-10.036 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RP_MATCHES_RCVD=-0.535, SPF_PASS=-0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham
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 vqh3BVF4qH-0 for <gen-art@ietfa.amsl.com>; Tue, 28 Jan 2014 21:07:26 -0800 (PST)
Received: from alln-iport-7.cisco.com (alln-iport-7.cisco.com [173.37.142.94]) by ietfa.amsl.com (Postfix) with ESMTP id BB5861A0193 for <gen-art@ietf.org>; Tue, 28 Jan 2014 21:07:26 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=3559; q=dns/txt; s=iport; t=1390972044; x=1392181644; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=Hbh9WYg9M986x/fxwrUFqeeyyDPA7kI3qEC6QQJ+IFk=; b=QLwOGwlns0SnrLjdT6mh/6IT7iRRb4lB0FrotoZpJqf6hE03ZlhlLdns ZxlhztoCJdy2mbckiVNRXf6YD3FfXZoKOpG6tTlXxSOaRHnbI/vDxcuUA /c+vId/jDF3pG8jjGnh1ShS4agPRnIUF2hEZGqCdXAC0EMKgbiEFo2Emh M=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: Ag0FAHuL6FKtJXHB/2dsb2JhbABagww4Vr0EgQ8WdIIlAQEBBCdSDAQCAQgQAQMBAQELHQcyFAkIAQEEAQ0FCId9DcksF40NgUExBwaDHoEUBIkRkEmQbYMtgio
X-IronPort-AV: E=Sophos;i="4.95,740,1384300800"; d="scan'208";a="16282791"
Received: from rcdn-core2-6.cisco.com ([173.37.113.193]) by alln-iport-7.cisco.com with ESMTP; 29 Jan 2014 05:07:23 +0000
Received: from xhc-aln-x01.cisco.com (xhc-aln-x01.cisco.com [173.36.12.75]) by rcdn-core2-6.cisco.com (8.14.5/8.14.5) with ESMTP id s0T57NIX023043 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Wed, 29 Jan 2014 05:07:23 GMT
Received: from xmb-rcd-x02.cisco.com ([169.254.4.29]) by xhc-aln-x01.cisco.com ([173.36.12.75]) with mapi id 14.03.0123.003; Tue, 28 Jan 2014 23:07:23 -0600
From: "Muthu Arul Mozhi Perumal (mperumal)" <mperumal@cisco.com>
To: Parthasarathi R <partha@parthasarathi.co.in>, "Francis.Dupont@fdupont.fr" <Francis.Dupont@fdupont.fr>, "gen-art@ietf.org" <gen-art@ietf.org>
Thread-Topic: review of draft-ietf-mmusic-sdp-g723-g729-04.txt
Thread-Index: Ac7p5lgX/OVwt+Nw0E+9CmXB4mmOXQAEYCHQDK21nNA=
Date: Wed, 29 Jan 2014 05:07:22 +0000
Message-ID: <E721D8C6A2E1544DB2DEBC313AF54DE2243BE719@xmb-rcd-x02.cisco.com>
References: <201311251357.rAPDvjtA037008@givry.fdupont.fr> <003101cee9f8$63890b50$2a9b21f0$@co.in>
In-Reply-To: <003101cee9f8$63890b50$2a9b21f0$@co.in>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [72.163.208.169]
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Mailman-Approved-At: Wed, 29 Jan 2014 05:10:35 -0800
Cc: "Flemming Andreasen (fandreas)" <fandreas@cisco.com>, "Ari Keränen (ari.keranen@ericsson.com)" <ari.keranen@ericsson.com>, "draft-ietf-mmusic-sdp-g723-g729.all@tools.ietf.org" <draft-ietf-mmusic-sdp-g723-g729.all@tools.ietf.org>
Subject: Re: [Gen-art] review of draft-ietf-mmusic-sdp-g723-g729-04.txt
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 29 Jan 2014 05:07:29 -0000

Hi Francis,

We have incorporated your comments in the next revision (will be publishing it shortly). 

|- Abstract page 1: usually the Abstract should not reference an RFC
|  by its number. IMHO here it is the exception: the I-D will be
|  included into the next revision of the RFC.

We have removed the reference to RFC4856 from the Abstract.

|- I don't like the annexa/annexb name (nor my spell checker) but
|  they are the names used by the RFC...

Right, annexa/annexb name is unavoidable.

|- ToC page 2 and 7 page 7: Acknowledgement -> Acknowledgment

Addressed.

|- 1 page 3 (wording suggestion): implied if -> implied when

Addressed.

|- 1 page 3: BTW IMHO "use or preferred" should be interpreted
|  as preferred in the offer and use in the answer so the RFC is
|  correct. But as you mentioned some implementations didn't follow
|  the interpretation so I understand why a clarification new
|  document (this I-D) is needed. And of course I fully agree with
|  3.1 and 3.2.

You got it right..

|- 7 page 7: Note I checked the spelling of "Harprit S. Chhatwal
| (InnoMedia)" (uncommon for our eyes but correct).

Thank you for that..

Muthu

|-----Original Message-----
|From: Parthasarathi R [mailto:partha@parthasarathi.co.in]
|Sent: Monday, November 25, 2013 9:37 PM
|To: Francis.Dupont@fdupont.fr; gen-art@ietf.org
|Cc: draft-ietf-mmusic-sdp-g723-g729.all@tools.ietf.org
|Subject: RE: review of draft-ietf-mmusic-sdp-g723-g729-04.txt
|
|Hi Francis.Dupont@fdupont.fr,
|
|Thanks a lot for the detailed review comments.
|
|I'll discuss with Muthu (Co-author) and then incorporate your comments in
|the next revision.
|
|Thanks
|Partha
|
|> -----Original Message-----
|> From: Francis.Dupont@fdupont.fr [mailto:Francis.Dupont@fdupont.fr]
|> Sent: Monday, November 25, 2013 7:28 PM
|> To: gen-art@ietf.org
|> Cc: draft-ietf-mmusic-sdp-g723-g729.all@tools.ietf.org
|> Subject: review of draft-ietf-mmusic-sdp-g723-g729-04.txt
|>
|> I am the assigned Gen-ART reviewer for this draft. For background on
|> Gen-ART, please see the FAQ at
|>
|> <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
|>
|> Please resolve these comments along with any other Last Call comments
|> you may receive.
|>
|> Document: draft-ietf-mmusic-sdp-g723-g729-04.txt
|> Reviewer: Francis Dupont
|> Review Date: 20131120
|> IETF LC End Date: 20131127
|> IESG Telechat date: unknown
|>
|> Summary: Ready
|>
|> Major issues: None
|>
|> Minor issues: None
|>
|> Nits/editorial comments:
|>  - Abstract page 1: usually the Abstract should not reference an RFC
|>   by its number. IMHO here it is the exception: the I-D will be
|>   included into the next revision of the RFC.
|>
|>  - I don't like the annexa/annexb name (nor my spell checker) but
|>   they are the names used by the RFC...
|>
|>  - ToC page 2 and 7 page 7: Acknowledgement -> Acknowledgment
|>
|>  - 1 page 3 (wording suggestion): implied if -> implied when
|>
|>  - 1 page 3: BTW IMHO "use or preferred" should be interpreted
|>   as preferred in the offer and use in the answer so the RFC is
|>   correct. But as you mentioned some implementations didn't follow
|>   the interpretation so I understand why a clarification new
|>   document (this I-D) is needed. And of course I fully agree with
|>   3.1 and 3.2.
|>
|>  - 7 page 7: Note I checked the spelling of "Harprit S. Chhatwal
|>   (InnoMedia)" (uncommon for our eyes but correct).
|>
|> Regards
|>
|> Francis.Dupont@fdupont.fr