[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Sip] Revised xcapevent to -06
I've revised xcapevent to fix the XML syntax errors to the best of my
ability, and to correct the grammar error pointed out by Pasi Eronen
during IESG review. There may be subtle XML issues about namespaces or
other stuff beyond my ken. More on that later.
Thew new grammar is:
diff-processing = "diff-processing" EQUAL (
"no-patching" /
"xcap-patching" /
"aggregate" /
token )
Before, we lacked the parentheses, so the production was wrong due to
operation precedence.
A review note from Pasi says:
"The examples in Appendix A contain some invalid XML -- I didn't
actually run a validator, but e.g. "" is clearly
wrong, and in some cases (e.g. last example in A.4), handling of
namespaces doesn't look OK."
I've actually run a validator on every XML block in this draft, by
cutting and pasting from the text into the validator. This took about
an hour. I did find a couple of missing ">" characters, which I have
fixed. I THINK I got everything. Anybody have an automated tool for a
recheck?
As for the namespace handling:
I presume Pasi's referring to:
<?xml version="1.0" encoding="UTF-8"?>
<xcap-diff xmlns="urn:ietf:params:xml:ns:xcap-diff"
xcap-root="http://xcap.example.com/">
<document previous-etag="7ahggs3"
sel="tests/users/sip:joe at example.com/index"
new-etag="63hjjsll"/>
</xcap-diff>
This looks like the same xmlns as every other example in the text, so
I could be missing something (like THAT would be a first!). Anybody
hhave a cluebat to hit me with here?
And in the process of writing this note up, I realized that a change I
made in section 4.7 at Jari's request somehow fell out of the edit
buffer. So I guess a -07 is on the way.
--
Dean Willis