Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLars Vogel2021-04-01 10:07:21 +0000
committerLars Vogel2021-04-01 10:07:21 +0000
commit5b945471cd20dc1f537651f9584c14e258dce07a (patch)
tree4d9f5bf1b71f82a9cd95897bfa393de7123177ba
parent8e1803d13078546b39de9293d3638f482974e7bf (diff)
downloadeclipse.platform.team-5b945471cd20dc1f537651f9584c14e258dce07a.tar.gz
eclipse.platform.team-5b945471cd20dc1f537651f9584c14e258dce07a.tar.xz
eclipse.platform.team-5b945471cd20dc1f537651f9584c14e258dce07a.zip
[dogfooding][cleanup] Use static inner class clean up for compare cocde
Using this cleanup brings: - It reviews the feature - For optimization: small memory usage reduction as the outer pointer is removed Change-Id: I8f1bdcb09e0e654683c0cb5e26c8370d6a47638e Signed-off-by: Lars Vogel <Lars.Vogel@vogella.com>
-rw-r--r--bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/CompareFilterDescriptor.java2
-rw-r--r--bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/CompareWithOtherResourceDialog.java6
-rw-r--r--bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/EditionAction.java2
-rw-r--r--bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/ResourceCompareInput.java2
4 files changed, 6 insertions, 6 deletions
diff --git a/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/CompareFilterDescriptor.java b/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/CompareFilterDescriptor.java
index b5d2d88e8..9bb3d5dd7 100644
--- a/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/CompareFilterDescriptor.java
+++ b/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/CompareFilterDescriptor.java
@@ -40,7 +40,7 @@ public class CompareFilterDescriptor {
private ResourceBundle fResourceBundle;
private ImageDescriptor fImageDescriptor;
- private class ConfigurationKeysEnumeration implements Enumeration<String> {
+ private static class ConfigurationKeysEnumeration implements Enumeration<String> {
private String[] keySet;
private int cursor = 0;
diff --git a/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/CompareWithOtherResourceDialog.java b/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/CompareWithOtherResourceDialog.java
index 4f4103c3d..8f1d0d55d 100644
--- a/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/CompareWithOtherResourceDialog.java
+++ b/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/CompareWithOtherResourceDialog.java
@@ -76,7 +76,7 @@ public class CompareWithOtherResourceDialog extends TitleAreaDialog {
private int MIN_HEIGHT_WITH_ANCESTOR = 320;
private int MIN_HEIGHT_WITHOUT_ANCESTOR = 238;
- private class FileTextDragListener implements DragSourceListener {
+ private static class FileTextDragListener implements DragSourceListener {
private ContentTypeElement element;
@@ -101,7 +101,7 @@ public class CompareWithOtherResourceDialog extends TitleAreaDialog {
}
}
- private class FileTextDropListener implements DropTargetListener {
+ private static class FileTextDropListener implements DropTargetListener {
private ContentTypeElement element;
private ResourceTransfer resourceTransfer;
@@ -181,7 +181,7 @@ public class CompareWithOtherResourceDialog extends TitleAreaDialog {
}
- private abstract class ContentTypeElement {
+ private abstract static class ContentTypeElement {
private Button radioButton;
protected Button mainButton;
diff --git a/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/EditionAction.java b/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/EditionAction.java
index 5619280a9..b68ce8fa9 100644
--- a/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/EditionAction.java
+++ b/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/EditionAction.java
@@ -56,7 +56,7 @@ public class EditionAction extends BaseCompareAction {
* Implements the IStreamContentAccessor and ITypedElement protocols
* for a Document.
*/
- class DocumentBufferNode implements ITypedElement, IEncodedStreamContentAccessor {
+ static class DocumentBufferNode implements ITypedElement, IEncodedStreamContentAccessor {
private static final String UTF_16= "UTF-16"; //$NON-NLS-1$
private IDocument fDocument;
private IFile fFile;
diff --git a/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/ResourceCompareInput.java b/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/ResourceCompareInput.java
index 8f4604d39..b718f97df 100644
--- a/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/ResourceCompareInput.java
+++ b/bundles/org.eclipse.compare/compare/org/eclipse/compare/internal/ResourceCompareInput.java
@@ -175,7 +175,7 @@ class ResourceCompareInput extends CompareEditorInput {
return fDiffViewer;
}
- class SelectAncestorDialog extends MessageDialog {
+ static class SelectAncestorDialog extends MessageDialog {
private IResource[] theResources;
IResource ancestorResource;
IResource leftResource;

Back to the top