[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: Atom Tombstones - Last Call



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. Get busy.