From 06980d14fb10df91f3580a98ececf765106714e4 Mon Sep 17 00:00:00 2001 From: James Blackburn Date: Wed, 10 Mar 2010 20:42:21 +0000 Subject: [PATCH] Tidy: using CopyOnWriteArrayList no need to duplicate listener list during notify. --- .../managedbuilder/internal/core/NotificationManager.java | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/build/org.eclipse.cdt.managedbuilder.core/src/org/eclipse/cdt/managedbuilder/internal/core/NotificationManager.java b/build/org.eclipse.cdt.managedbuilder.core/src/org/eclipse/cdt/managedbuilder/internal/core/NotificationManager.java index ac0bb65b86c..4e3c5166e21 100644 --- a/build/org.eclipse.cdt.managedbuilder.core/src/org/eclipse/cdt/managedbuilder/internal/core/NotificationManager.java +++ b/build/org.eclipse.cdt.managedbuilder.core/src/org/eclipse/cdt/managedbuilder/internal/core/NotificationManager.java @@ -42,10 +42,8 @@ public class NotificationManager /*implements ISettingsChangeListener */{ } private void notifyListeners(SettingsChangeEvent event){ - ISettingsChangeListener listeners[] = fListeners.toArray(new ISettingsChangeListener[fListeners.size()]); - for(int i = 0; i < listeners.length; i++){ - listeners[i].settingsChanged(event); - } + for (ISettingsChangeListener listener : fListeners) + listener.settingsChanged(event); } private static SettingsChangeEvent createOptionChangedEvent(IResourceInfo rcInfo, IHoldsOptions holder, IOption option, Object oldValue){