Re: Review of draft-ietf-mmusic-4572-update-11

Christer Holmberg <christer.holmberg@ericsson.com> Wed, 25 January 2017 13:41 UTC

Return-Path: <christer.holmberg@ericsson.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 51554129951; Wed, 25 Jan 2017 05:41:28 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.221
X-Spam-Level:
X-Spam-Status: No, score=-4.221 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, 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 JRT5mUHCWGJ7; Wed, 25 Jan 2017 05:41:27 -0800 (PST)
Received: from sesbmg22.ericsson.net (sesbmg22.ericsson.net [193.180.251.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 6D2841298CE; Wed, 25 Jan 2017 05:41:26 -0800 (PST)
X-AuditID: c1b4fb30-13a0498000007085-4c-5888ab04e22b
Received: from ESESSHC001.ericsson.se (Unknown_Domain [153.88.183.21]) by (Symantec Mail Security) with SMTP id CE.FE.28805.40BA8885; Wed, 25 Jan 2017 14:41:24 +0100 (CET)
Received: from ESESSMB209.ericsson.se ([169.254.9.76]) by ESESSHC001.ericsson.se ([153.88.183.21]) with mapi id 14.03.0319.002; Wed, 25 Jan 2017 14:40:33 +0100
From: Christer Holmberg <christer.holmberg@ericsson.com>
To: Elwyn Davies <elwynd@dial.pipex.com>, "gen-art@ietf.org" <gen-art@ietf.org>
Subject: Re: Review of draft-ietf-mmusic-4572-update-11
Thread-Topic: Review of draft-ietf-mmusic-4572-update-11
Thread-Index: AQHSdoIexBxSt+o33kK0m/85TkkEWaFJRV6A
Date: Wed, 25 Jan 2017 13:40:32 +0000
Message-ID: <D4AE73C2.165FA%christer.holmberg@ericsson.com>
References: <148529043511.12735.9313753556504417778.idtracker@ietfa.amsl.com>
In-Reply-To: <148529043511.12735.9313753556504417778.idtracker@ietfa.amsl.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/14.7.1.161129
x-originating-ip: [153.88.183.19]
Content-Type: text/plain; charset="Windows-1252"
Content-ID: <88571B48E596E3449D14B2D538C45CC0@ericsson.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrHIsWRmVeSWpSXmKPExsUyM2K7qC7L6o4Ig6mr9S0+nLvKbrHtuKDF 1VefWSyebZzPYjF1+WMWB1aP4yt2snssWfKTKYApissmJTUnsyy1SN8ugStj7inpgqecFR86 ZrM2MF5h72Lk5JAQMJE4dH8zcxcjF4eQwDpGiYWbbzJBOIsZJf4+bgVyODjYBCwkuv9pgzSI CARKzFw9gRWkhllgOaPEg32fGEESwgLmEi9nHmWGKLKQ+DphOhuEbSTR8n0TmM0ioCoxZ3If mM0rYC3RunA62BVCAr4Sz7Z+B5vDKeAn8bDhHVgNo4CYxPdTa5hAbGYBcYlbT+YzQVwtILFk z3lmCFtU4uXjf6wgtqiAnsTy52ug4ooS7U8bGCF6DSTen5vPDGFbS1zuesAGYWtLLFv4mhni HkGJkzOfsExgFJ+FZN0sJO2zkLTPQtI+C0n7AkbWVYyixanFSbnpRkZ6qUWZycXF+Xl6eakl mxiBsXhwy2+DHYwvnzseYhTgYFTi4f2Q3R4hxJpYVlyZe4hRgoNZSYRXfGVHhBBvSmJlVWpR fnxRaU5q8SFGaQ4WJXFes5X3w4UE0hNLUrNTUwtSi2CyTBycUg2ME0+xW6rwhuvNPlLrf6t7 2xNx7tyTVdrnbfRUs3V55PMmWWhN0BXQ7dS7y7TZ8fbC9LfZvizHFn9hjzsfa3304VZZEb6m bkmHG481lfjTpT4nHLbtXs2Zv8Jgilz5Em7HigATObWTArt/nc85/1Jw5u6yushu99+zbZm2 qPWZ7333gWX//A9KLMUZiYZazEXFiQD1aADtwQIAAA==
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/yKuqRxEjlQrqVx4XPxQ2XQ6ElB0>
Cc: "draft-ietf-mmusic-4572-update.all@ietf.org" <draft-ietf-mmusic-4572-update.all@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>, "mmusic@ietf.org" <mmusic@ietf.org>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 25 Jan 2017 13:41:28 -0000

Hi Elwyn,

Thanks for your review! See inline.

Nits/editorial comments:

>s1, para 2: s/TLS protocol/The TLS protocol/ (as per RFC 4572)

I will fix as suggested.


>s4, para 2: s/a new protocol identifier/the protocol identifier/ (it
>isn't new any more)

I am happy to remove ³new², but doesn¹t ³a² still sound better than ³the²?


>s5.1: Suggest s/m- line/"m" line/  for consistency with s3.4

I will replace with single quotes (Œm¹), for consistency with s3.4 and s4.


>s5.1, para 1: s/e.g./e.g.,/

I will fix as suggested.


>s5.1, para 4: s/that each used certificate matches/that each
>certificate used matches/

I will fix as suggested.


>s5.1, para 5: s/each used certificate matches/each certificate used
>matches/

I will fix as suggested.

>s8, para 5: ' This specification creates a new IANA registry named
>"Hash Function Textual Names".'
>The registry is no longer new.  Perhaps s/creates a new IANA
>registry/takes over the IANA registry from RFC 4572/

I suggest:

 "This specification takes over the IANA registry named "Hash Function
 Textual Names², that was created in RFC 4572.  It will not be part of
 the SDP Parameters.²


Regards,

Christer