RE: Review comments from Basavaraj Patel on CTP
[email protected] Fri, 24 Oct 2003 18:05:32 +0300
| Newsgroups | gmane.ietf.seamoby |
|---|---|
| Message-ID | <[email protected]> |
> Editorial: > > 1. Can the problem description text in the Introduction be a > subsection? The current Into section begins with a reference to a > problem statement described in RFC3374. > Nit: The open quote "Problem... in Section 1 is not terminated. Done > 2. s/This section provides a protocol overview/This section provides the > CT protocol overview. Done > 3. Section 2: (References) > "In response to a CT-Activate Request message or to the CT trigger, pAR > predictively transmits a Context Transfer Data (CTD) message that > contains feature contexts. This message, described in Section > 2.4.2" > Incorrect reference. > > "In the second scenario, pAR receives a Context Transfer Request (CT > Request) described in Section 2.4.5, message from nAR." > Incorrect reference; Should be 2.4.6 > > Nit: Check all references Done > 4. Section 2: > "[1].* contexts, pAR verifies authorization token before transmitting > the [2]algorithm [3] When context transfer takes place without the > nAR requesting it (scenario one above), nAR requires MN to present > its authorization token" > > I could not make out what this statement says. I am not even sure > if the authors can understand it. Done, I've deleted it, based upon reorganizing some of the text > 5. Section 2: > Can you create subsections for scenarios 1 and 2? Would make it > easier to reference. done. > 6. s/Additially/Additionally Done > 7. Section 2.3 > a. Would be good to describe each field separately. Goes for all > subsequent sections where message formats are described. > CXT Type - > Length - > P bit - Will do > b. "When the presence vector is in use, the Presentation Vector is > interpreted.." > What is this Presentation vector? (Typo) fixed > c. Last paragraph has % chars done > 8. Sec 2.4.1 > "If an acknowledgement for this message is needed, the MN sets the > A flag to 1, " > > Missing "A" flag in CTAR. Fixed, it was listed as R > 9. Sec 2.4.2 > Describe the fields. Will do > 10. Is there a need for specifying the status codes used in CTDR and CTAA? Will do. John