From c704c83cbd4edda438c056fcd2c9f27eeb2497fc Mon Sep 17 00:00:00 2001 From: Anthony Green Date: Mon, 5 Jul 1999 13:35:21 +0000 Subject: URL.java (equals): Compare strings using String.equals. * java/net/URL.java (equals): Compare strings using String.equals. * java/net/URL.java (sameFile): Ditto. From-SVN: r27947 --- libjava/java/net/URL.java | 32 ++++++++++++++++++++++---------- 1 file changed, 22 insertions(+), 10 deletions(-) (limited to 'libjava/java') diff --git a/libjava/java/net/URL.java b/libjava/java/net/URL.java index b6dc13d..d2f0bc2 100644 --- a/libjava/java/net/URL.java +++ b/libjava/java/net/URL.java @@ -189,11 +189,18 @@ public final class URL implements Serializable return false; URL uObj = (URL) obj; - if (protocol != uObj.protocol || host != uObj.host || port != uObj.port || - file != uObj.file || ref != uObj.ref) - return false; - - return true; + + // This comparison is very conservative. It assumes that any + // field can be null. + return (port == uObj.port + && ((protocol == null && uObj.protocol == null) + || (protocol != null && protocol.equals(uObj.protocol))) + && ((host == null && uObj.host == null) + || (host != null && host.equals(uObj.host))) + && ((file == null && uObj.file == null) + || (file != null && file.equals(uObj.file))) + && ((ref == null && uObj.ref == null) + || (ref != null && ref.equals(uObj.ref)))); } public final Object getContent() throws IOException @@ -258,11 +265,16 @@ public final class URL implements Serializable public boolean sameFile(URL other) { - if (other == null || protocol != other.protocol || host != other.host || - port != other.port || file != other.file) - return false; - - return true; + // This comparison is very conservative. It assumes that any + // field can be null. + return (other != null + && port == other.port + && ((protocol == null && other.protocol == null) + || (protocol != null && protocol.equals(other.protocol))) + && ((host == null && other.host == null) + || (host != null && host.equals(other.host))) + && ((file == null && other.file == null) + || (file != null && file.equals(other.file)))); } protected void set(String protocol, String host, int port, String file, -- cgit v1.1