[cp-patches] Re: Absolute URL parsing bug

[Mark Wielaard]

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, 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.

Cheers,

Mark
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://gcc.gnu.org/pipermail/java-patches/attachments/20050705/3c58bb81/attachment.sig>