Thanks Francis. We will address the comments and upload in a day or two. Regards, Dhananjay
Sent from Samsung Mobile -------- Original message -------- From: "Sri Gundavelli (sgundave)" <[email protected]> Date:27/02/2017 23:42 (GMT+05:30) To: Francis Dupont <[email protected]>, [email protected] Cc: [email protected] Subject: Re: review of draft-ietf-dmm-lma-controlled-mag-params-03.txt Thanks Francis. On 2/27/17, 3:30 AM, "Francis Dupont" <[email protected]> wrote: >I am the assigned Gen-ART reviewer for this draft. The General Area >Review Team (Gen-ART) reviews all IETF documents being processed >by the IESG for the IETF Chair. Please treat these comments just >like any other last call comments. > >For more information, please see the FAQ at > ><https://trac.ietf.org/trac/gen/wiki/GenArtfaq>. > >Document: draft-ietf-dmm-lma-controlled-mag-params-03.txt >Reviewer: Francis Dupont >Review Date: 20170217 >IETF LC End Date: 20170224 >IESG Telechat date: unknown > >Summary: Ready > >Major issues: None > >Minor issues: None > >Nits/editorial comments: > - ToC page 2 and 9 (title) page 8: Acknowledgements -> Acknowledgments > > - 1 page 2: you must expand the abbrevs at first use in the body of > the document: the abstract is not a part of the body. > > - 1 page 2: singaling -> signaling > > - 3 page 3: suboptions -> sub-options > (because the dash was at all other uses) > > - 3 page 4: the wording "The sub-options are optional" is ambiguous: > it can mean that there is no mandatory sub-options or that the > field can be empty. Obviously it is the first according to the > section 3 intro "MUST contain one or more". This container is for carrying one or more sub-options, but inclusion of any specific sub-option is optional. DJ can reword this to clarify this more clearly. Ack to other comments. Sri > > - 3.1 page 4: Binding Refresh Control Sub-Option -> > Binding Re-registration Control Sub-Option > (it is the official name of the option cf section 6) > > - authors' addresses page 12: China -> P.R. China or ISO 3166 CN > BTW I have a question about the "Kansas": I can understand > to put only the state for California or Texas but Kansas? (:-) > >Regars > >[email protected]
_______________________________________________ Gen-art mailing list [email protected] https://www.ietf.org/mailman/listinfo/gen-art
