Author: lindner
Date: Mon Jun 14 22:26:11 2010
New Revision: 954662
URL: http://svn.apache.org/viewvc?rev=954662&view=rev
Log:
small performance optimization
Modified:
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/config/CoreUtilConfigContributor.java
Modified:
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/config/CoreUtilConfigContributor.java
URL:
http://svn.apache.org/viewvc/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/config/CoreUtilConfigContributor.java?rev=954662&r1=954661&r2=954662&view=diff
==============================================================================
---
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/config/CoreUtilConfigContributor.java
(original)
+++
shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/config/CoreUtilConfigContributor.java
Mon Jun 14 22:26:11 2010
@@ -10,6 +10,7 @@ import org.apache.shindig.gadgets.spec.M
import java.util.Collection;
import java.util.Map;
+import java.util.Set;
/**
* Populates the core.util configuration, which at present includes the list
@@ -31,9 +32,11 @@ public class CoreUtilConfigContributor i
ModulePrefs prefs = gadget.getSpec().getModulePrefs();
Collection<Feature> features = prefs.getFeatures().values();
Map<String, Map<String, Object>> featureMap =
Maps.newHashMapWithExpectedSize(features.size());
+ Set<String> allFeatureNames = registry.getAllFeatureNames();
+
for (Feature feature : features) {
// Skip unregistered features
- if (!registry.getAllFeatureNames().contains(feature.getName())) {
+ if (!allFeatureNames.contains(feature.getName())) {
continue;
}
// Flatten out the multimap a bit for backwards compatibility: map keys
