Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLars Vogel2021-05-05 09:27:59 +0000
committerLars Vogel2021-05-10 17:48:43 +0000
commitff7f0d838fb6cd3a7091dd3365d0ae3d2417b69b (patch)
tree0dcac87eb77144d97511fbfb8c7bd74466654546
parentf97e0374e552bf072482a60d176dc230071a1006 (diff)
downloadeclipse.platform.team-ff7f0d838fb6cd3a7091dd3365d0ae3d2417b69b.tar.gz
eclipse.platform.team-ff7f0d838fb6cd3a7091dd3365d0ae3d2417b69b.tar.xz
eclipse.platform.team-ff7f0d838fb6cd3a7091dd3365d0ae3d2417b69b.zip
Use JDT static inner class clean up for team.coreI20210510-1800
This ticket uses and tests the "Use static inner class" cleanup feature: - It reviews the feature - It reduces the memory consumption as it avoids the pointer to the outer class Example: Before: public class Outer { public class InnerClass { int i; public boolean anotherMethod() { return true; } } } After: public class Outer { public static class InnerClass { int i; public boolean anotherMethod() { return true; } } } Change-Id: I3982b15f62d5e42da12f3ee114712002655651c7 Signed-off-by: Lars Vogel <Lars.Vogel@vogella.com> Reviewed-on: https://git.eclipse.org/r/c/platform/eclipse.platform.team/+/180222 Tested-by: Platform Bot <platform-bot@eclipse.org>
-rw-r--r--bundles/org.eclipse.team.core/src/org/eclipse/team/internal/core/mapping/ScopeManagerEventHandler.java2
-rw-r--r--bundles/org.eclipse.team.core/src/org/eclipse/team/internal/core/subscribers/BatchingLock.java2
2 files changed, 2 insertions, 2 deletions
diff --git a/bundles/org.eclipse.team.core/src/org/eclipse/team/internal/core/mapping/ScopeManagerEventHandler.java b/bundles/org.eclipse.team.core/src/org/eclipse/team/internal/core/mapping/ScopeManagerEventHandler.java
index 95ac2ef9a..d658266f7 100644
--- a/bundles/org.eclipse.team.core/src/org/eclipse/team/internal/core/mapping/ScopeManagerEventHandler.java
+++ b/bundles/org.eclipse.team.core/src/org/eclipse/team/internal/core/mapping/ScopeManagerEventHandler.java
@@ -33,7 +33,7 @@ public class ScopeManagerEventHandler extends BackgroundEventHandler {
private Set<ResourceMapping> toRefresh = new HashSet<>();
private ISynchronizationScopeManager manager;
- class ResourceMappingEvent extends Event {
+ static class ResourceMappingEvent extends Event {
private final ResourceMapping[] mappings;
public ResourceMappingEvent(ResourceMapping[] mappings) {
super(REFRESH);
diff --git a/bundles/org.eclipse.team.core/src/org/eclipse/team/internal/core/subscribers/BatchingLock.java b/bundles/org.eclipse.team.core/src/org/eclipse/team/internal/core/subscribers/BatchingLock.java
index 0c4939d91..63a569c17 100644
--- a/bundles/org.eclipse.team.core/src/org/eclipse/team/internal/core/subscribers/BatchingLock.java
+++ b/bundles/org.eclipse.team.core/src/org/eclipse/team/internal/core/subscribers/BatchingLock.java
@@ -59,7 +59,7 @@ public class BatchingLock {
}
};
- public class ThreadInfo {
+ public static class ThreadInfo {
private Set<IResource> changedResources = new HashSet<>();
private IFlushOperation operation;
private List<ISchedulingRule> rules = new ArrayList<>();

Back to the top