Re: [xml] empty CDATA sections

[Daniel Veillard <veillard redhat com>]

On Thu, Mar 06, 2008 at 09:58:38AM -0500, Daniel Veillard wrote:
> On Mon, Mar 03, 2008 at 11:06:08PM -0800, Mark Rowe wrote:
> > On 03/03/2008, at 22:47, Daniel Veillard wrote:
> >
> > > On Mon, Mar 03, 2008 at 09:45:18AM -0800, Darin Adler wrote:
> > > > Hi folks.
> > > >
> > > > As you may know, WebKit currently uses libxml2 to handle XML
> > > > documents, including XHTML. I recently discovered that a number of  
> > > > Ian
> > > > Hickson's test cases that work fine in other browsers fail in  
> > > > browsers
> > > > based on WebKit. That's because libxml2 discards empty CDATA  
> > > > sections.
> > > > I filed this bug:
> > > >
> > > >    http://bugzilla.gnome.org/show_bug.cgi?id=514181
> > > >
> > > > And Mark Rowe attached a patch that resolves the issue.
> > > >
> > > > Daniel pointed out that some clients of libxml2 may be depending on
> > > > the current behavior; some may see this as a feature rather than a  
> > > > bug.
> > > >
> > > > Does anyone have any comments, objections to this change perhaps or
> > > > support for the change?
>
>   Since nobody seems to have objections about this I will look at the
> changes needed and do some testing when I find a bit of free time.

  Okay, done. Actually the normal parsing code was preserving empty CDATA
nodes, so it's not a change of behaviour, the push and streaming parser
now all work alike, and the serialization code has been updated to save 
them back. An example was added to the regression suite,

  thanks,

Daniel

-- 
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard      | virtualization library  http://libvirt.org/
veillard redhat com  | libxml GNOME XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine  http://rpmfind.net/