Replace anonymous classes by lambda expressions in Platform Runtime
Increases the performance, because lesser objects need to be created.
Improves readability because a lot of boilerplate code will be removed.
jSparrow Clean Up.
Change-Id: If24161d3f066332d10f7e84e749432ab4112a8e9
Signed-off-by: Ardit Ymeri <ardit.ymeri@splendit.at>
diff --git a/bundles/org.eclipse.core.expressions/src/org/eclipse/core/internal/expressions/Expressions.java b/bundles/org.eclipse.core.expressions/src/org/eclipse/core/internal/expressions/Expressions.java
index cd65153..3ba6ab5 100644
--- a/bundles/org.eclipse.core.expressions/src/org/eclipse/core/internal/expressions/Expressions.java
+++ b/bundles/org.eclipse.core.expressions/src/org/eclipse/core/internal/expressions/Expressions.java
@@ -138,15 +138,12 @@
fgKnownClasses= new WeakHashMap<>();
fgNotFoundClasses = new WeakHashMap<>();
BundleContext bundleContext= ExpressionPlugin.getDefault().getBundleContext();
- BundleListener listener= new BundleListener() {
- @Override
- public void bundleChanged(BundleEvent event) {
- // Invalidate the caches if any of the bundles is stopped
- if (event.getType() == BundleEvent.STOPPED) {
- synchronized (Expressions.class) {
- fgKnownClasses.clear();
- fgNotFoundClasses.clear();
- }
+ BundleListener listener= (BundleEvent event) -> {
+ // Invalidate the caches if any of the bundles is stopped
+ if (event.getType() == BundleEvent.STOPPED) {
+ synchronized (Expressions.class) {
+ fgKnownClasses.clear();
+ fgNotFoundClasses.clear();
}
}
};
diff --git a/bundles/org.eclipse.e4.core.services/src/org/eclipse/e4/core/internal/services/about/InstalledBundles.java b/bundles/org.eclipse.e4.core.services/src/org/eclipse/e4/core/internal/services/about/InstalledBundles.java
index d61a1fd..ffede0d 100644
--- a/bundles/org.eclipse.e4.core.services/src/org/eclipse/e4/core/internal/services/about/InstalledBundles.java
+++ b/bundles/org.eclipse.e4.core.services/src/org/eclipse/e4/core/internal/services/about/InstalledBundles.java
@@ -56,17 +56,13 @@
private Comparator<Bundle> createComparator(Map<String, String> names) {
final Collator collator = Collator.getInstance(Locale.getDefault());
- return new Comparator<Bundle>() {
-
- @Override
- public int compare(Bundle o1, Bundle o2) {
- String id1 = identify(o1);
- String id2 = identify(o2);
- if (!id1.equals(id2)) {
- return collator.compare(id1, id2);
- }
- return collator.compare(names.get(id1), names.get(id2));
+ return (Bundle o1, Bundle o2) -> {
+ String id1 = identify(o1);
+ String id2 = identify(o2);
+ if (!id1.equals(id2)) {
+ return collator.compare(id1, id2);
}
+ return collator.compare(names.get(id1), names.get(id2));
};
}
diff --git a/bundles/org.eclipse.e4.core.services/src/org/eclipse/e4/core/internal/services/about/InstalledFeatures.java b/bundles/org.eclipse.e4.core.services/src/org/eclipse/e4/core/internal/services/about/InstalledFeatures.java
index 38215f5..9b57d82 100644
--- a/bundles/org.eclipse.e4.core.services/src/org/eclipse/e4/core/internal/services/about/InstalledFeatures.java
+++ b/bundles/org.eclipse.e4.core.services/src/org/eclipse/e4/core/internal/services/about/InstalledFeatures.java
@@ -44,17 +44,13 @@
private Comparator<IBundleGroup> createComparator() {
final Collator collator = Collator.getInstance(Locale.getDefault());
- return new Comparator<IBundleGroup>() {
-
- @Override
- public int compare(IBundleGroup o1, IBundleGroup o2) {
- String id1 = o1.getIdentifier();
- String id2 = o2.getIdentifier();
- if (!id1.equals(id2)) {
- return collator.compare(id1, id2);
- }
- return collator.compare(o1.getName(), o2.getName());
+ return (IBundleGroup o1, IBundleGroup o2) -> {
+ String id1 = o1.getIdentifier();
+ String id2 = o2.getIdentifier();
+ if (!id1.equals(id2)) {
+ return collator.compare(id1, id2);
}
+ return collator.compare(o1.getName(), o2.getName());
};
}
