[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [cp-patches] Re: Absolute URL parsing bug
From: |
Andrew Haley |
Subject: |
Re: [cp-patches] Re: Absolute URL parsing bug |
Date: |
Tue, 5 Jul 2005 14:45:16 +0100 |
Mark Wielaard writes:
> Hi,
>
> On Mon, 2005-07-04 at 18:07 +0100, Andrew Haley wrote:
> > When creating a URL: if the spec is absolute and a context URL is
> > supplied, we should inherit the host:port part of the context if its
> > protocol the same as that of the spec, but we should not inherit its
> > file path.
>
> Have you added the tests from your previous email to mauve?
> There are a lot of URL tests in mauve,
Ah, I didn't know that. OK, I'll try running mauve. Sigh.
> any new PASSes or FAILs?
>
> > *************** public final class URL implements Serial
> > *** 409,413 ****
> > host = context.host;
> > port = context.port;
> > - file = context.file;
> > userInfo = context.userInfo;
> > if (file == null || file.length() == 0)
> > --- 409,412 ----
>
> Isn't that last if statement now unnecessary?
> It seems you can set file to "/" unconditionally here.
True, yes. I'm finding it pretty hard to get inside the thinking of
whoever wrote this.
Andrew.
- [cp-patches] Absolute URL parsing bug, Andrew Haley, 2005/07/01
- [cp-patches] Re: Absolute URL parsing bug, Per Bothner, 2005/07/01
- [cp-patches] Re: Absolute URL parsing bug, Andrew Haley, 2005/07/02
- [cp-patches] Re: Absolute URL parsing bug, Per Bothner, 2005/07/02
- [cp-patches] Re: Absolute URL parsing bug, Andrew Haley, 2005/07/04
- [cp-patches] Re: Absolute URL parsing bug, Andrew Haley, 2005/07/04
- Re: [cp-patches] Re: Absolute URL parsing bug, Mark Wielaard, 2005/07/05
- Re: [cp-patches] Re: Absolute URL parsing bug,
Andrew Haley <=
- Re: [cp-patches] Re: Absolute URL parsing bug, Andrew Haley, 2005/07/07
- [cp-patches] Re: Absolute URL parsing bug, Per Bothner, 2005/07/05
- [cp-patches] Re: Absolute URL parsing bug, Tom Tromey, 2005/07/02
- Re: [cp-patches] Re: Absolute URL parsing bug, Chris Burdess, 2005/07/02
- Re: [cp-patches] Re: Absolute URL parsing bug, Andrew Haley, 2005/07/03