[xml] Proposed patch for #389786



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

Attachment: uri-patch.txt
Description: Text document



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]