sm-02 nits
From: Jari Arkko (jari.arkkopiuha.net)
Date: Thu, 18 Mar 2004 02:26:43 -0500 (EST)
- Title should expand "EAP" per RFC editor
  requirements.

- Abstract should expand AAA and replace
  RADIUS with AAA so that it doesn't have to be
  expanded also.

- Move the second last paragraph after the text
  that talks about the switch.

- Delete last paragraph of the abstract,
  it is in part duplication and in part contain
  temporary status information about the document.

- Section 2, s/out of scope for/outside the scope of/
  I think, English is not my native language...

- For some reason, the bottom page heading is right on
  the next line after text ends. E.g. Section 4.1.
  I find this a bit confusing, it takes a moment to
  realize that your eyes have read into the heading and
  not a continuation of the paragraph. If you can change
  this easily, please do. If not, don't worry about it...

- Inconsistent capitalization of the sentences explaining
  variables, see e.g. Sections 4.1.1 vs 4.1.2.

- s/wasn't/was not/
  s/don't/do not/
  s/can't/can not/
  s/hasn't/has not/

- In Section 5.2, I think it would be useful to add
  the following just before "m.init": "In the following
  we will discuss procedure calls that the state machine
  can make to methods."

- As discussed with the RFC editor and in IETF-59,
  we need a .txt version with the state machine
  as a table.

--Jari



Results generated by Tiger Technologies using MHonArc.