Re: [xml] [PATCH v2] xmllint: Fix validation of URIs with ampersands

2015-10-29 Thread Liam R E Quin
On Tue, 27 Oct 2015 15:09:57 -0600 Alex Henrie wrote: > 2015-10-26 19:31 GMT-06:00 Liam R E Quin : > > URIs with unescaped ampersands are a syntax error, however... > > Invalid XML is rejected with or without this patch. The effect of this > patch is to make xmllint --stream produce the same out

Re: [xml] [PATCH v2] xmllint: Fix validation of URIs with ampersands

2015-10-27 Thread Alex Henrie
2015-10-26 19:31 GMT-06:00 Liam R E Quin : > URIs with unescaped ampersands are a syntax error, however... Invalid XML is rejected with or without this patch. The effect of this patch is to make xmllint --stream produce the same output as plain xmllint. Please read the bug report at https://bugzil

Re: [xml] [PATCH v2] xmllint: Fix validation of URIs with ampersands

2015-10-27 Thread Alex Henrie
I just realized that I put "xmllint" as the component in the patch title. If you accept this patch (and I hope you do), you'll probably want to change that to "xmlschemas". -Alex ___ xml mailing list, project page http://xmlsoft.org/ xml@gnome.org https

Re: [xml] [PATCH v2] xmllint: Fix validation of URIs with ampersands

2015-10-27 Thread Liam R E Quin
URIs with unescaped ampersands are a syntax error, however... Liam -- Liam Quin - XML Activity Lead, W3C, http://www.w3.org/People/Quin/ Pictures from old books: http://fromoldbooks.org/ ___ xml mailing list, project page http://xmlsoft.org/ xml@gnom

[xml] [PATCH v2] xmllint: Fix validation of URIs with ampersands

2015-10-26 Thread Alex Henrie
This makes xmlSchemaSAXHandleStartElementNs pass attributes through xmlStringDecodeEntities, similar to how xmlSchemaVDocWalk passes them through xmlNodeListGetString. Fixes https://bugzilla.gnome.org/show_bug.cgi?id=709171 You can see my first try and the feedback I got on it here and here: http