Haven't had a chance to re-read the whole document, but I just noticed 
something which I thought I should mention before I forgot about it.

I don't know RELAX NG syntax very well, but it seems to me that the schema you 
have for the deletedEntry isn't quite right.

     deletedEntry = element at:deleted-entry {
       atomCommonAttributes,
       attribute ref { atomUri },
       attribute when { atomDateConstruct },
       ( element at:by { atomPersonConstruct}?,
       & element at:comment {atomTextConstruct}?,
       & element atom:link*,
       & element atom:source?,
       & extensionElement* )
     }

Surely those element lines shouldn't have a comma on the end? You already have 
an ampersand connector and you wouldn't want a comma connector anyway since 
order isn't significant. Or have I got it all wrong?

Also, not a big deal, but while you're looking at that, the spacing inside the 
braces could do with some consistency.

Regards
James

                                          
_________________________________________________________________
The New Busy think 9 to 5 is a cute idea. Combine multiple calendars with 
Hotmail. 
http://www.windowslive.com/campaign/thenewbusy?tile=multicalendar&ocid=PID28326::T:WLMTAGL:ON:WL:en-US:WM_HMP:042010_5

Reply via email to