public inbox for [email protected]  

help / color / mirror / Atom feed

From: Nikolay Samokhvalov <[email protected]>
To: Peter Eisentraut <[email protected]>
Cc: Bruce Momjian <[email protected]>
Cc: PostgreSQL-development <[email protected]>
Cc: PostgreSQL-documentation <[email protected]>
Subject: Re: [HACKERS] New XML section for documentation
Date: Tue, 5 Sep 2006 11:58:36 +0400
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>

On 8/26/06, Peter Eisentraut <[email protected]> wrote:
> Bruce Momjian wrote:
> > I made it clear in the section that the XML syntax was being checked,
> > not validation against a schema.  You want Check and Validation
> > sections?
>
> "Valid" and "well-formed" have very specific distinct meanings in XML.
> (Note that "check" doesn't have any meaning there.)  We will eventually
> want a method to verify both the validity and the well-formedness.
>
> I think that a function called xml_valid checks for well-formedness is
> an outright bug and needs to be fixed.

That's exactly what I'm talking about. xml_valid() is wrong name and
it may confuse people.
I what to add that, with XML section in the documentation, this bug
becomes more significant.

Bruce suggested to use overload to keep backward compat. - in other
words, 1-arg function for checking for well-formedness and 2-arg
function for validation process. That's bad too:
 - two _different_ actions for one function => another confusion
  - I (as a user) would think that 1-arg function is designed for
validation process for cases when XML document contains a reference to
DTD (as an example).

I stand for fixing it via renaming, breaking backward compatibility.
Later it will be more painful.

BTW, what is the deadline for changes (additions) in docs? I would add
general XML terms (such as what is XML, what is well-formed document,
what is validation; short overview of XML standards and SQL/XML as a
part of SQL:200n, etc Maybe about contrib/xml2 installation process -
actually, XSLT support requires additional lib). Moreover, if SQL/XML
patch will be accepted it will require several words too.
-- 
Best regards,
Nikolay

view thread (30+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected]
  Subject: Re: [HACKERS] New XML section for documentation
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox