- From: Henry S. Thompson <ht@inf.ed.ac.uk>
- Date: Tue, 28 Aug 2007 12:46:52 +0100
- To: Norman Walsh <ndw@nwalsh.com>
- Cc: public-xml-processing-model-wg@w3.org
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Norman Walsh writes:
> | 4.6.1
> |
> | Major confusion wrt prefixes (c: vs err:) and links (c:error in the
> | first liine of 4.6.1.2 links to the p:error section!)
> |
> | I have some vague memory of a decision to abandon a namespace and
> | collapse names from that namespace into some other one -- was it the
> | err: namespace? If so, not only do the uses of err: in this section
> | and in 4.6.2 need to be changed, but also the entry for err: should
> | be removed from section 3.1
>
> The error vocabulary has been moved to the c: namespace, but the err:
> namespace has been retained to identify static and dynamic errors.
OK, but
a) c:error links to the anchor 'c.error', which is on 7.1.7 in the
document: it should linke to 'cv.error';
b) <err:error> still occurs in Example 9 in 4.6.2.
> | 4.7
> |
> | As long as we leave the tableau for them here, we can't say
> | _nothing_ about other-compound-steps! I would prefer to see them
> | removed from this section (see previous discussion), since they have
> | to be extensions, and as such they don't have to conform to our
> | syntax at all (note for instance that as written the production
> | given here would rule out a p:choose-type extension,
> | e.g. ipfx:case).
> |
> | At the very least, we need something along the lines of "The
> | 'other-compound-step' tableau is included above as an illustrative
> | place-holder for future expansion of the language, which might be
> | explored via extensions in the first instance."
>
> Well, I removed the tableaus. Maybe that helps?
Well, I thought the tableau for other_atomic_step was useful -- why
did you remove _it_? And there's still the sentence "Other steps can
be atomic or compound", followed by a discussion of atomic, but
nothing about compound. . .
> | 5.5
> |
> | A note about security implications?
>
> Such as?
"Note: This element represents a potential security risk: Running
unexamined 3rd-party pipelines could result in vital system
resources being overwritten."
Something similar probably belongs in 7.1.23 Store.
> | 5.7.1.1
> |
> | Sigh, this highlights a general problem. We use 'QName' all over
> | this spec., and sometimes we specify in detail where the namespace
> | bindings come from to expand them (most notably for option values,
> | but also, e.g. for c:parameter names), in which case one should
> | understand QName to mean "QName as specified in XML Namespaces 1.1".
> | But in other places, e.g. here, we just say "must be a QName".
> | _Either_ we should specify where the namespace bindings come from,
> | _or_ we should say "must be a QName as specified in XML Schema 1.0"
> | (but that wouldn't _really_ do that job, because the Schema spec. is
> | weak here).
> |
> | I guess I think we should make QName a defined term, use it
> | everywhere, and define it to mean "must be a QName as specified in
> | XML Namespaces 1.1. If unprefixed, in no namespace. If prefixed,
> | prefix *must* be bound in the in-scope namespaces of the containing
> | element, unless some other source of namespace bindings is
> | specified." and specify one or two static errors to use for failures
> | of the two "*must*"s.
>
> Do you mean "XML Namespaces 1.1" literally, or do you mean "XML Namespaces
> 1.0 or XML Namespaces 1.1, as appropriate"?
Well, I probably meant "[Namespaces 1.0] or [Namespaces 1.1], as
appropriate", but both you and I are assuming something in document
about 1.0 vs. 1.1 flexibility, which isn't actually there yet, as far
as I can see. . .
> | 5.11
> |
> | I'm unhappy with a perhaps-unintended consequence of the way output
> | ports on compound steps are handled here. Seems to me this is a
> | perfectly good p:for-each:
> |
> | <p:for-each name="saveThem">
> |
> | <p:output port="result">
> | <p:pipe step="saveThem" port="current"/>
> | </p:output>
>
> That makes my brain hurt. Output ports bind to step outputs. If you
> want that, stick in p:identity step that reads the input and produces
> an output.
Sigh -- I don't see why we should treat these specially, but I don't
suppose it makes much difference. If no-one else cares, I'll let this
go.
> | 5.12
> |
> | "It is a static error (err:XS0024) if the content of the p:inline
> | element is not a well-formed XML document."
> |
> | How could any processor ever detect this error?
>
> By looking to see if p:inline has two or more top-level element children.
Ah, _that's_ the point. Well, it can't be a document, it can't
include a doctype. . . I guess we need to be careful to say what we
really mean here:
"It is a static error (err:XS0024) if the content of the p:inline
element does not consist of exactly one element, optionally
preceded and/or followed by any number of processing instructions,
comments or whitespace characters."
I don't know how else to say this, since there is no notion of
well-formed infoset to appeal to, otherwise we could say "the content
. . . must be such as to produce a well-formed infoset when installed
as the [children] of a Document InfoItem".
> | 5.13
> |
> | "The parser which the p:document element employs must be conformant
> | to Namespaces in XML.
> |
> | -->
> |
> | "The parser which the p:document element employs must be conformant
> | to [Namespaces 1.1]."
>
> See earlier question about NS 1.0 vs. NS 1.1.
See earlier response.
ht
- --
Henry S. Thompson, HCRC Language Technology Group, University of Edinburgh
Half-time member of W3C Team
2 Buccleuch Place, Edinburgh EH8 9LW, SCOTLAND -- (44) 131 650-4440
Fax: (44) 131 650-4587, e-mail: ht@inf.ed.ac.uk
URL: http://www.ltg.ed.ac.uk/~ht/
[mail really from me _always_ has this .sig -- mail without it is forged spam]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.6 (GNU/Linux)
iD8DBQFG1AsskjnJixAXWBoRApjJAJ9cfVUz7wrpMk357tJH2bcUFGVShQCeKNBV
GsxHRhxRbzFDThYCFFF4yvw=
=GWtw
-----END PGP SIGNATURE-----
Received on Tuesday, 28 August 2007 11:48:16 UTC