Re: xinclude errata document review

On Wed, Sep 28, 2005 at 02:08:19PM -0400, Grosso, Paul wrote:
> 
>  
> > -----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.

  Dohh ! Fixed online !

> 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

  All fixed, thanks Paul !

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel@veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | 

Received on Wednesday, 28 September 2005 21:27:37 UTC