[PATCH v2] msxml3: all string passed to IXMLDOMDocument_load() need to be URL-unescaped

[Damjan Jovanovic]

msxml3 allows URL escape sequences even for C:\ style paths.
eg. C:\Program%20Files\...

Try 2 determines URL length dynamically, suppresses some compiler
const warnings, and cleans up tests.

Signed-off-by: Damjan Jovanovic <damjan.jov(a)gmail.com>
---
 dlls/msxml3/bsc.c          | 33 ++++++++++++++++++++++++++++-----
 dlls/msxml3/tests/domdoc.c | 17 +++++++++++++++++
 2 files changed, 45 insertions(+), 5 deletions(-)

[Nikolay Sivov]

This again is too complicated and unescapes every url, which is 
unnecessary. Does attached patch work for you?

[Damjan Jovanovic]

Yes that patch works too :)


On Sun, Dec 8, 2019 at 3:56 PM Nikolay Sivov <nsivov(a)codeweavers.com> wrote:
> This again is too complicated and unescapes every url, which is
> unnecessary. Does attached patch work for you?

[Nikolay Sivov]

Great. Could you resend this patch together with your tests as v3, or I 
can do it.

On 12/8/19 5:29 PM, Damjan Jovanovic wrote:
> Yes that patch works too :)
>
> On Sun, Dec 8, 2019 at 3:56 PM Nikolay Sivov <nsivov(a)codeweavers.com 
> <mailto:nsivov(a)codeweavers.com>> wrote:
>
> This again is too complicated and unescapes every url, which is
>     unnecessary. Does attached patch work for you?

[Damjan Jovanovic]

Resent.

On Sun, Dec 8, 2019 at 7:04 PM Nikolay Sivov <nsivov(a)codeweavers.com> wrote:
> Great. Could you resend this patch together with your tests as v3, or I
> can do it.
>
> On 12/8/19 5:29 PM, Damjan Jovanovic wrote:
>
> Yes that patch works too :)
>
> On Sun, Dec 8, 2019 at 3:56 PM Nikolay Sivov <nsivov(a)codeweavers.com>
> wrote:
> > This again is too complicated and unescapes every url, which is
> > unnecessary. Does attached patch work for you?