Attachment 257296 Details for Bug 472778 – Fix for this issue

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

[patch] Fix for this issue

472778_launch_validation.patch (text/plain), 1.83 KB, created by Vikas Chandra

on 2015-10-16 07:18:10 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Vikas Chandra

Created: 2015-10-16 07:18:10 EDT

Size: 1.83 KB

patch

obsolete

>diff --git a/ui/org.eclipse.pde.launching/src/org/eclipse/pde/internal/launching/launcher/BundleLauncherHelper.java b/ui/org.eclipse.pde.launching/src/org/eclipse/pde/internal/launching/launcher/BundleLauncherHelper.java
>index 876ff69..33ade48 100644
>--- a/ui/org.eclipse.pde.launching/src/org/eclipse/pde/internal/launching/launcher/BundleLauncherHelper.java
>+++ b/ui/org.eclipse.pde.launching/src/org/eclipse/pde/internal/launching/launcher/BundleLauncherHelper.java
>@@ -1,5 +1,5 @@
> /*******************************************************************************
>- * Copyright (c) 2007, 2013 IBM Corporation and others.
>+ * Copyright (c) 2007, 2015 IBM Corporation and others.
>  * All rights reserved. This program and the accompanying materials
>  * are made available under the terms of the Eclipse Public License v1.0
>  * which accompanies this distribution, and is available at
>@@ -177,6 +177,7 @@
> 
> 			//remove conflicting duplicates - if they have same version or both are singleton
> 			HashMap<String, IPluginModelBase> pluginMap = new HashMap<String, IPluginModelBase>();
>+			Set<IPluginModelBase> pluginSet = new HashSet<IPluginModelBase>();
> 			List<IPluginModelBase> workspaceModels = null;
> 			for (Iterator<IPluginModelBase> iterator = launchPlugins.iterator(); iterator.hasNext();) {
> 				IPluginModelBase model = iterator.next();
>@@ -192,11 +193,12 @@
> 						}
> 					}
> 				}
>-				pluginMap.put(id, model);
>+				pluginSet.add(model);
> 			}
>+			pluginMap.clear();
> 
> 			// Create the start levels for the selected plugins and add them to the map
>-			for (Iterator<IPluginModelBase> iterator = pluginMap.values().iterator(); iterator.hasNext();) {
>+			for (Iterator<IPluginModelBase> iterator = pluginSet.iterator(); iterator.hasNext();) {
> 				IPluginModelBase model = iterator.next();
> 				addBundleToMap(map, model, "default:default"); //$NON-NLS-1$
> 			}

Actions: View | Diff

Attachments on bug 472778: 255584 | 257296 | 257328