xinclude errata document review

 
> -----Original Message-----
> From: Daniel Veillard [mailto:daniel@veillard.com] 

>   Done, I recorded all potential errata where the resolution led
> to an update to the spec, i.e.:
>    PEX1, PEX6, PEX7, PEX10, PEX11, PEX15, PEX16
> 
> I uploaded the resulting errata document:
>    http://www.w3.org/2004/12/xinclude-errata

Here's my review.

In general, looks good, except I'm not able to
correspond PEX10 in the errata document to PEX10
in the PE document.  I think you copied PEX7 to
PEX10 and then didn't update the text.

Editorial
---------
PEX6, Rationale, s/specidied, t/specified. T/
(Note both a typo and a punctuation change.)

PEX11, Rationale, s/superceeded/superceded/

PEX15:

Suggested new text for improved wording, change:

  Since the xpointer attribute is not a URI reference,
  and that %-escaping is not done in XPointers, so '%'
  is an ordinary character in the value of the xpointer
  attribute.

to:

  Since the xpointer attribute is not a URI reference,
  and %-escaping is not done in XPointers, '%'
  is an ordinary character in the value of the xpointer
  attribute.

Rationale, s/ressource/resource/

REX16, Rationale, 
  change:
    specification, all XInclude processors
to:
    specification. All XInclude processors

paul

Received on Wednesday, 28 September 2005 18:08:21 UTC