Attachment 23466 Details for Bug 100624 – Fix to removing sites

[patch] Fix to removing sites

org.eclipse.update.ui.patch (text/plain), 1.12 KB, created by Dorian Birsan

on 2005-06-17 14:49:00 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Dorian Birsan

Created: 2005-06-17 14:49:00 EDT

Size: 1.12 KB

patch

obsolete

>Index: src/org/eclipse/update/internal/ui/wizards/TargetPage.java
>===================================================================
>RCS file: /home/eclipse/org.eclipse.update.ui/src/org/eclipse/update/internal/ui/wizards/TargetPage.java,v
>retrieving revision 1.72
>diff -u -r1.72 TargetPage.java
>--- src/org/eclipse/update/internal/ui/wizards/TargetPage.java	18 May 2005 18:52:40 -0000	1.72
>+++ src/org/eclipse/update/internal/ui/wizards/TargetPage.java	17 Jun 2005 18:46:41 -0000
>@@ -388,11 +388,12 @@
>     
>     void removeAddedSites() {
>         if (added != null) {
>-            while (!added.isEmpty()) {
>-                Iterator it = added.iterator(); 
>-                if (it.hasNext())
>-                    config.removeConfiguredSite((IConfiguredSite) it.next());
>+        	Iterator it = added.iterator(); 
>+            while (it.hasNext()) {
>+                 config.removeConfiguredSite((IConfiguredSite) it.next());
>+                // the config listener no longer removes the site so we need to cleanup the collection at the end
>             }
>+            added.clear();
>         }           
>     }
>

Actions: View | Diff

Attachments on bug 100624: 23466