Hi,

Here is a proposed patch for bug #389786.

Apparently, the regression was introduced with revision 3396 of uri.c which
was checked-in with following comment
"* uri.c: applied patch from Rob Richards fixing the URI regressions
 tests on Windows which seems to indicate bad escaping.
Daniel"

That revision 3396 effectively defeated the part of code a couple of lines
before that replaces the \ with / (for relative path only).

Of course my patch needs to be reviewed thoroughly, since I don't really
understand the URI and path normalization code in uri.c, and I just provided
a local correction to make that particular test happy. If someone can point
me how to run the URI regression tests on windows, I would be glad to test
that first, however.

-sbi
Index: uri.c
===================================================================
--- uri.c       (revision 3575)
+++ uri.c       (working copy)
@@ -2433,7 +2433,7 @@
     }
 
     if (uri->scheme == NULL) {
-       ret = xmlStrdup((const xmlChar *) path);
+       ret = xmlStrdup((const xmlChar *) uri->path);
     } else {
        ret = xmlSaveUri(uri);
     }
_______________________________________________
xml mailing list, project page  http://xmlsoft.org/
[email protected]
http://mail.gnome.org/mailman/listinfo/xml

Reply via email to