Bug 552175 - Cache property names per EClass
JXPath creates only a single instance of EDynamicPropertyHandler. On
this instance getPropertyNames is called frequently for the same EClass,
making is a candidate for caching.
Change-Id: I0155dafc3ee5f43c00cdcadbb838d4c7fa1e2277
Signed-off-by: Karsten Thoms <karsten.thoms@itemis.de>
diff --git a/bundles/org.eclipse.e4.emf.xpath/src/org/eclipse/e4/emf/internal/xpath/helper/EDynamicPropertyHandler.java b/bundles/org.eclipse.e4.emf.xpath/src/org/eclipse/e4/emf/internal/xpath/helper/EDynamicPropertyHandler.java
index 17a5677..d4d9e51 100644
--- a/bundles/org.eclipse.e4.emf.xpath/src/org/eclipse/e4/emf/internal/xpath/helper/EDynamicPropertyHandler.java
+++ b/bundles/org.eclipse.e4.emf.xpath/src/org/eclipse/e4/emf/internal/xpath/helper/EDynamicPropertyHandler.java
@@ -13,6 +13,9 @@
******************************************************************************/
package org.eclipse.e4.emf.internal.xpath.helper;
+import java.util.HashMap;
+import java.util.Map;
+
import org.apache.commons.jxpath.DynamicPropertyHandler;
import org.eclipse.core.runtime.Assert;
import org.eclipse.emf.ecore.EClass;
@@ -25,12 +28,14 @@
* @author Karsten Thoms <karsten.thoms@itemis.de>
*/
public class EDynamicPropertyHandler implements DynamicPropertyHandler {
+ private Map<EClass, String[]> eClass2PropNames = new HashMap<>();
+
@Override
public String[] getPropertyNames(Object object) {
Assert.isLegal(object instanceof EObject);
EClass eClass = ((EObject) object).eClass();
- String[] propNames = eClass.getEAllStructuralFeatures().stream().map(f -> f.getName()).toArray(String[]::new);
- return propNames;
+ return eClass2PropNames.computeIfAbsent(eClass,
+ clz -> clz.getEAllStructuralFeatures().stream().map(f -> f.getName()).toArray(String[]::new));
}
@Override