On Jul 10, 2011, at 1:40 PM, Josh Berkus <j...@agliodbs.com> wrote:

> Hackers,
> 
>>> B. 6. Current behaviour _is intended_ (there is "if"  to check node type) 
>>> and _"natural"_. In this particular case user ask for text content of some 
>>> node, and this content is actually "<".
>> 
>> I don't buy that. The check for the node type is there because
>> two different libxml functions are used to convert nodes to
>> strings. The if has absolutely *zero* to do with escaping, expect
>> for that missing escape_xml() call in the "else" case.
>> 
>> Secondly, there is little point in having an type XML if we
>> don't actually ensure that values of that type can only contain
>> well-formed XML.
> 
> Can anyone else weigh in on this? Peter?

Unless I am missing something, Florian  is clearly correct here.

...Robert
-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to