Re: [rtcweb] AD Review: draft-ietf-rtcweb-fec-07

On Thu, Mar 1, 2018 at 3:40 PM, Adam Roach <adam@nostrum.com> wrote:

> I have completed my AD review of draft-ietf-rtcweb-fec-07, and found only
> very minor editorial nits, which should be treated as normal last call
> comments.
>

Thanks for the review, and agree with the comments, with one exception.
Should I publish a new version of the document with corrections, or wait
for the document to progress further?

>
> Due to this document's close relationship with
> draft-ietf-payload-flexible-fec-scheme, I plan to wait until both
> documents are ready to progress before placing draft-ietf-rtcweb-fec into
> IETF Last Call.
>
> Thanks to everyone who contributed to this document over its lifetime.
>
> Nits follow.
>
> ------------------------------------------------------------
> ---------------
> §2
>
> Since this document uses "should" in non-normative sentences, please use
> the
> RFC 8174 boilerplate.
>
> ------------------------------------------------------------
> ---------------
> §3.3
>
>    Some audio codecs, notably Opus [RFC6716] and AMR [RFC4867] support
>
> Insert comma before "support"
>
> ------------------------------------------------------------
> ---------------
> §4.1
>
>    against individual losses, with minimal overhead.  Note that as
>    indicated above the built-in Opus FEC only provides single-frame
>
> Insert commas before and after the phrase "as indicated above"
>
> ------------------------------------------------------------
> ---------------
> §7
>
>    Implementations MAY support additional FEC mechanisms if desired,
>    e.g.  [RFC5109].
>
> - Change the comma after "desired" to a semicolon.

I think a comma is appropriate here; this is also consistent with how e.g.
is used elsewhere in the document.

> - Insert a comma after "e.g."
>
> ------------------------------------------------------------
> ---------------
>
>
> /a
>
> _______________________________________________
> rtcweb mailing list
> rtcweb@ietf.org
> https://www.ietf.org/mailman/listinfo/rtcweb
>