Skip to content

Commit 4c1a034

Browse files
eclipse-pde-botlaeubi
authored andcommitted
Perform clean code of ui/org.eclipse.pde.ui
1 parent bce79ce commit 4c1a034

15 files changed

+32
-30
lines changed

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/bndtools/BndScanner.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,12 +40,12 @@
4040
public class BndScanner extends RuleBasedScanner {
4141
final Set<String> instructions;
4242
final Set<String> directives = new HashSet<>();
43-
private Token T_DEFAULT;
44-
private Token T_KEY;
45-
private Token T_ERROR;
46-
private Token T_COMMENT;
47-
private Token T_INSTRUCTION;
48-
private Token T_OPTION;
43+
private final Token T_DEFAULT;
44+
private final Token T_KEY;
45+
private final Token T_ERROR;
46+
private final Token T_COMMENT;
47+
private final Token T_INSTRUCTION;
48+
private final Token T_OPTION;
4949

5050
public BndScanner(IColorManager colorManager) {
5151
T_DEFAULT = new Token(new TextAttribute(colorManager.getColor(IJavaColorConstants.JAVA_DEFAULT)));

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/correction/java/AddPdeClasspathContainerClasspathFixProposal.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626

2727
public class AddPdeClasspathContainerClasspathFixProposal extends ClasspathFixProposal {
2828

29-
private IJavaProject project;
29+
private final IJavaProject project;
3030

3131
public AddPdeClasspathContainerClasspathFixProposal(IJavaProject project) {
3232
this.project = project;

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/editor/plugin/JavaAttributeWizardPage.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ public class JavaAttributeWizardPage extends NewClassWizardPage {
4444
private final ISchemaAttribute attInfo;
4545
private final IPluginModelBase model;
4646
private final InitialClassProperties initialValues;
47-
private IJavaProject javaProject;
47+
private final IJavaProject javaProject;
4848

4949
static class InitialClassProperties {
5050
// populate new wizard page

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/parts/PluginVersionPart.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ public Object[] getElements(Object inputElement) {
179179
private VersionRange fVersionRange;
180180
private boolean fIsRanged;
181181
private final boolean fRangeAllowed;
182-
private boolean isPlugin;
182+
private final boolean isPlugin;
183183

184184
public PluginVersionPart(boolean rangeAllowed) {
185185
this(rangeAllowed, false);

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/shared/target/RepositoryBundleContainerAdapterFactory.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,10 +117,12 @@ public boolean canEdit(ITargetDefinition target, TreePath treePath) {
117117
return segment instanceof RepositoryBundleContainer || segment instanceof RequirementNode;
118118
}
119119

120+
@Override
120121
public boolean canRemove(ITargetDefinition target, TreePath treePath) {
121122
return treePath.getLastSegment() instanceof RequirementNode;
122123
}
123124

125+
@Override
124126
public IStatus remove(ITargetDefinition target, TreePath[] treePaths) {
125127
boolean reload = false;
126128
for (TreePath path : treePaths) {

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/shared/target/UpdateTargetJob.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,9 @@ public class UpdateTargetJob extends Job {
3939

4040
private final IJobFunction action;
4141

42-
private boolean update;
42+
private final boolean update;
4343

44-
private Job cancelJob;
44+
private final Job cancelJob;
4545

4646
/**
4747
* Schedules a new update job that will update all selected children of the

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/wizards/plugin/TemplateListSelectionPage.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ public class TemplateListSelectionPage extends WizardListSelectionPage {
5757
private final ContentPage fContentPage;
5858
private Button fUseTemplate;
5959
private String fInitialTemplateId;
60-
private Map<Template, CompletableFuture<String>> templateTextLoadings = new HashMap<>();
60+
private final Map<Template, CompletableFuture<String>> templateTextLoadings = new HashMap<>();
6161

6262
class WizardFilter extends ViewerFilter {
6363
@Override

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/wizards/plugin/TemplatePluginContentWizard.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@
4545
public class TemplatePluginContentWizard extends Wizard implements IPluginContentWizard {
4646

4747
private static final String DEFAULT_BND_INSTRUCTION = "bnd.bnd"; //$NON-NLS-1$
48-
private Template template;
48+
private final Template template;
4949
private IFieldData data;
5050
private TemplateParamsWizardPage paramsWizardPage;
5151

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/wizards/tools/ConvertAutomaticManifestProcessor.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@
5656

5757
public class ConvertAutomaticManifestProcessor extends RefactoringProcessor {
5858

59-
private List<IProject> projects;
59+
private final List<IProject> projects;
6060
private boolean useProjectRoot;
6161
private boolean keepRequireBundle;
6262
private boolean keepImportPackage;

ui/org.eclipse.pde.ui/src/org/eclipse/pde/internal/ui/wizards/tools/ConvertAutomaticManifestsWizardSettingsPage.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public class ConvertAutomaticManifestsWizardSettingsPage extends UserInputWizard
3232
private static final String KEY_KEEP_IMPORT_PACKAGE = "keep_import_package"; //$NON-NLS-1$
3333
private static final String KEY_KEEP_EXPORT_PACKAGE = "keep_export_package"; //$NON-NLS-1$
3434
private static final String KEY_KEEP_REQUIREDEXECUTIONENVIRONMENT = "keep_requiredexecutionenvironment_package"; //$NON-NLS-1$
35-
private ConvertAutomaticManifestProcessor processor;
35+
private final ConvertAutomaticManifestProcessor processor;
3636

3737
public ConvertAutomaticManifestsWizardSettingsPage(ConvertAutomaticManifestProcessor processor) {
3838
super(PDEUIMessages.ConvertAutomaticManifestWizardPage_title);

0 commit comments

Comments
 (0)