Document: draft-cridland-urlfetch-binary-03.txt
REviewer: Pete McCann
Review Date: 02 Dec 2008
IESG Telechat Date: 04 Dec 2008


Summary:
ABNF might have some problems, unless I am confused.  Otherwise looks ok. 

Technical
=========

I could not parse the ABNF as some productions seemed to be missing; I looked for them in the referenced documents RFC 3501, RFC 3516, and RFC 4467.  Specifically, I cannot find a definition for "url-meta-bodystruct", "url-meta-body", or "url-meta-binary".
Did you mean for the last three productions ("url-bodystruct", "url-binary", and "url-body") to have the word "meta" in them?  Those three seem to be

unused elsewhere, and fit the examples given earlier.  Or, did I just miss the definitions from somewhere else?  In that case, it might be nice to have a pointer to where the production is actually defined, in a comment

within the ABNF.

Editorial
=========

In the Abstract:
OLD:
user's private store, however,
NEW:
user's private store; however,

Section 3:
OLD:
additional, per-URL, parameters
NEW:
additional, per-URL parameters

OLD:
command, and will provide,
NEW:
command and will provide,

Section 3.2:
OLD:
   follows:
   BODYPARTSTRUCTURE
NEW:
   follows:

   BODYPARTSTRUCTURE