[Gen-art] review of draft-ietf-ccamp-general-constraint-encode-16.txt

[Francis Dupont <[e-mail]>]

I am the assigned Gen-ART reviewer for this draft. For background on
Gen-ART, please see the FAQ at

<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Please resolve these comments along with any other Last Call comments
you may receive.

Document: draft-ietf-ccamp-general-constraint-encode-16.txt
Reviewer: Francis Dupont
Review Date: 20150107
IETF LC End Date: 20150117
IESG Telechat date: unknown

Summary: Ready

Major issues: None

Minor issues: None

Nits/editorial comments:
 - you use a strange indentation for sub sections... Perhaps it is from
  the tempate?? Anyway it will be handled/fixed by the RFC Editor.

 - 2 page 6:
  1 -- the device is switched(e.g., ROADM/OXC)
                             ^ add a space here

 - 2.2 page 7: OxFF -> 0xff (letter O -> digit 0)

 - 2.2 page 8: RestrictType -> RestrictionType

 - 2.3 page 11:
  0x01(Inclusive Range)
      ^ add a space here

 - 2.3 page 11: I can't understand the unnumbered in this statement:
  "... Note
   that the Action field can be set to 0x01 (Inclusive Range) only when
   unnumbered link identifier is used."

 - 2.6.1 page 16: I suggest: Num Labels -> Num Labels = N
  in the schema

 - 2.6.2 page 16: I suggest: Num Labels -> Num Labels = 2
  in the schema

 - 2.6.3 page 17: missing parenthesis:
  "positions (Num Labels) and beyond SHOULD be set to zero"
                                    ^ add ) here?

 - 6 page 28: I don't fully understand why RFC 5307 is an
  informative reference and RFC 4203 (same with OSPF in place
  of IS-IS) is a normative one.

 - pages 31 and 32: perhaps another side-effect from the template:
  inusual (for a draft) IP statement & co...

Thanks

[e-mail]