[cp-patches] Re: Absolute URL parsing bug

[Andrew Haley]

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.