Attachment 156039 Details for Bug 299551 – patch

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] patch

299533-352.txt (text/plain), 4.16 KB, created by Andrew Niefer

on 2010-01-13 14:59:38 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Andrew Niefer

Created: 2010-01-13 14:59:38 EST

Size: 4.16 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.equinox.p2.metadata.generator
>Index: src/org/eclipse/equinox/internal/p2/metadata/generator/features/SiteFeature.java
>===================================================================
>RCS file: /cvsroot/rt/org.eclipse.equinox/p2/bundles/org.eclipse.equinox.p2.metadata.generator/src/org/eclipse/equinox/internal/p2/metadata/generator/features/SiteFeature.java,v
>retrieving revision 1.4
>diff -u -r1.4 SiteFeature.java
>--- src/org/eclipse/equinox/internal/p2/metadata/generator/features/SiteFeature.java	20 May 2009 04:10:55 -0000	1.4
>+++ src/org/eclipse/equinox/internal/p2/metadata/generator/features/SiteFeature.java	13 Jan 2010 19:45:22 -0000
>@@ -39,30 +39,42 @@
> 	 * Return false if one of them is null
> 	 */
> 	public static boolean sameURL(URL url1, URL url2) {
>-
>-		if (url1 == null || url2 == null)
>-			return false;
> 		if (url1 == url2)
> 			return true;
>-		if (url1.equals(url2))
>-			return true;
>+		if (url1 == null ^ url2 == null)
>+			return false;
> 
> 		// check if URL are file: URL as we may
> 		// have 2 URL pointing to the same featureReference
> 		// but with different representation
> 		// (i.e. file:/C;/ and file:C:/)
>-		if (!"file".equalsIgnoreCase(url1.getProtocol())) //$NON-NLS-1$
>-			return false;
>-		if (!"file".equalsIgnoreCase(url2.getProtocol())) //$NON-NLS-1$
>-			return false;
>-
>-		File file1 = new File(url1.getFile());
>-		File file2 = new File(url2.getFile());
>-
>-		if (file1 == null)
>+		final boolean isFile1 = "file".equalsIgnoreCase(url1.getProtocol());//$NON-NLS-1$
>+		final boolean isFile2 = "file".equalsIgnoreCase(url2.getProtocol());//$NON-NLS-1$
>+		if (isFile1 && isFile2) {
>+			File file1 = new File(url1.getFile());
>+			File file2 = new File(url2.getFile());
>+			return file1.equals(file2);
>+		}
>+		// URL1 xor URL2 is a file, return false. (They either both need to be files, or neither)
>+		if (isFile1 ^ isFile2)
> 			return false;
>+		return getExternalForm(url1).equals(getExternalForm(url2));
>+	}
> 
>-		return (file1.equals(file2));
>+	/**
>+	 * Gets the external form of this URL. In particular, it trims any white space, 
>+	 * removes a trailing slash and creates a lower case string.
>+	 */
>+	private static String getExternalForm(URL url) {
>+		String externalForm = url.toExternalForm();
>+		if (externalForm == null)
>+			return ""; //$NON-NLS-1$
>+		externalForm = externalForm.trim();
>+		if (externalForm.endsWith("/")) { //$NON-NLS-1$
>+			// Remove the trailing slash
>+			externalForm = externalForm.substring(0, externalForm.length() - 1);
>+		}
>+		return externalForm.toLowerCase();
> 	}
> 
> 	/**
>@@ -107,18 +119,44 @@
> 	 * <code>false</code> otherwise
> 	 */
> 	public boolean equals(Object object) {
>-
> 		if (object == null)
> 			return false;
>-		if (getURL() == null)
>-			return false;
>-
> 		if (!(object instanceof SiteFeature))
> 			return false;
>-
>-		SiteFeature f = (SiteFeature) object;
>-
>-		return sameURL(getURL(), f.getURL());
>+		SiteFeature that = (SiteFeature) object;
>+		if (this.featureId == null) {
>+			if (that.featureId != null)
>+				return false;
>+		} else if (!this.featureId.equals(that.featureId))
>+			return false;
>+		if (this.featureVersion == null) {
>+			if (that.featureVersion != null)
>+				return false;
>+		} else if (!this.featureVersion.equals(that.featureVersion))
>+			return false;
>+		return sameURL(this.getURL(), that.getURL());
>+	}
>+
>+	/* (non-Javadoc)
>+	 * @see java.lang.Object#hashCode()
>+	 */
>+	public int hashCode() {
>+		final int prime = 31;
>+		int result = 1;
>+		result = prime * result + (featureId == null ? 0 : featureId.hashCode());
>+		result = prime * result + (featureVersion == null ? 0 : featureVersion.hashCode());
>+		if (this.getURL() == null)
>+			return result;
>+
>+		if ("file".equalsIgnoreCase(getURL().getProtocol())) {//$NON-NLS-1$ 
>+			// If the URL is a file, then create the HashCode from the file
>+			File f = new File(getURL().getFile());
>+			if (f != null)
>+				result = prime * result + f.hashCode();
>+		} else
>+			// Otherwise create it from the External form of the URL (in lower case)
>+			result = prime * result + getExternalForm(this.getURL()).hashCode();
>+		return result;
> 	}
> 
> 	/**

Actions: View | Diff

Attachments on bug 299551: 156039