From 84d6fdefaa5aa03e452616e153ad9c3c1338297f Mon Sep 17 00:00:00 2001 From: Alexander Kurtakov Date: Fri, 9 Mar 2018 10:40:53 +0200 Subject: [PATCH] Sonar fixes in Meson plugins. Change-Id: I27a75d6dac41ae820d6af0b7c30922eb5bac58f7 Signed-off-by: Alexander Kurtakov --- .../meson/core/MesonToolChainManager.java | 2 +- .../org/eclipse/cdt/meson/core/Activator.java | 12 +++++---- .../cdt/meson/core/IMesonConstants.java | 12 ++++----- .../cdt/internal/meson/ui/MesonBuildTab.java | 9 ++++--- .../internal/meson/ui/SWTImagesFactory.java | 7 +++--- .../commands/AbstractMesonCommandHandler.java | 25 +++++++++---------- .../meson/ui/wizards/WizardMessages.java | 2 +- .../properties/IMesonPropertyPageControl.java | 5 ++-- .../ui/properties/MesonPropertyPage.java | 7 ------ 9 files changed, 38 insertions(+), 43 deletions(-) diff --git a/build/org.eclipse.cdt.meson.core/src/org/eclipse/cdt/internal/meson/core/MesonToolChainManager.java b/build/org.eclipse.cdt.meson.core/src/org/eclipse/cdt/internal/meson/core/MesonToolChainManager.java index e7c7ade2670..bce48e85afb 100644 --- a/build/org.eclipse.cdt.meson.core/src/org/eclipse/cdt/internal/meson/core/MesonToolChainManager.java +++ b/build/org.eclipse.cdt.meson.core/src/org/eclipse/cdt/internal/meson/core/MesonToolChainManager.java @@ -214,7 +214,7 @@ public class MesonToolChainManager implements IMesonToolChainManager { for (IMesonToolChainListener listener : listeners) { SafeRunner.run(new ISafeRunnable() { @Override - public void run() throws Exception { + public void run() { listener.handleMesonToolChainEvent(event); } diff --git a/build/org.eclipse.cdt.meson.core/src/org/eclipse/cdt/meson/core/Activator.java b/build/org.eclipse.cdt.meson.core/src/org/eclipse/cdt/meson/core/Activator.java index 9708b196c12..eae68bbb85f 100644 --- a/build/org.eclipse.cdt.meson.core/src/org/eclipse/cdt/meson/core/Activator.java +++ b/build/org.eclipse.cdt.meson.core/src/org/eclipse/cdt/meson/core/Activator.java @@ -136,13 +136,15 @@ public class Activator implements BundleActivator { } public static void log(Throwable e) { - if (e instanceof InvocationTargetException) + if (e instanceof InvocationTargetException) { e = ((InvocationTargetException) e).getTargetException(); + } IStatus status = null; - if (e instanceof CoreException) + if (e instanceof CoreException) { status = ((CoreException) e).getStatus(); - else + } else { status = new Status(IStatus.ERROR, getUniqueIdentifier(), IStatus.OK, e.getMessage(), e); + } log(status); } @@ -153,12 +155,12 @@ public class Activator implements BundleActivator { } @Override - public void start(BundleContext bundleContext) throws Exception { + public void start(BundleContext bundleContext) { Activator.context = bundleContext; } @Override - public void stop(BundleContext bundleContext) throws Exception { + public void stop(BundleContext bundleContext) { Activator.context = null; plugin = null; } diff --git a/build/org.eclipse.cdt.meson.core/src/org/eclipse/cdt/meson/core/IMesonConstants.java b/build/org.eclipse.cdt.meson.core/src/org/eclipse/cdt/meson/core/IMesonConstants.java index 3f79e47d4b7..9b9870e6af8 100644 --- a/build/org.eclipse.cdt.meson.core/src/org/eclipse/cdt/meson/core/IMesonConstants.java +++ b/build/org.eclipse.cdt.meson.core/src/org/eclipse/cdt/meson/core/IMesonConstants.java @@ -12,12 +12,12 @@ package org.eclipse.cdt.meson.core; public interface IMesonConstants { - public static final String MESON_ARGUMENTS = "meson.arguments"; //$NON-NLS-1$ - public static final String MESON_ENV = "meson.environment"; //$NON-NLS-1$ - public static final String MESON_PROJECT_OPTIONS = "meson.project.options"; //$NON-NLS-1$ - public static final String NINJA_ENV = "meson.ninja.environment"; //$NON-NLS-1$ - public static final String NINJA_ARGUMENTS = "meson.ninja.arguments"; //$NON-NLS-1$ - public static final String MESON_ENV_SEPARATOR = "|"; //$NON-NLS-1$ + String MESON_ARGUMENTS = "meson.arguments"; //$NON-NLS-1$ + String MESON_ENV = "meson.environment"; //$NON-NLS-1$ + String MESON_PROJECT_OPTIONS = "meson.project.options"; //$NON-NLS-1$ + String NINJA_ENV = "meson.ninja.environment"; //$NON-NLS-1$ + String NINJA_ARGUMENTS = "meson.ninja.arguments"; //$NON-NLS-1$ + String MESON_ENV_SEPARATOR = "|"; //$NON-NLS-1$ String MESON_GENERATOR = "meson.generator"; //$NON-NLS-1$ String BUILD_COMMAND = "meson.command.build"; //$NON-NLS-1$ String CLEAN_COMMAND = "meson.command.clean"; //$NON-NLS-1$ diff --git a/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/MesonBuildTab.java b/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/MesonBuildTab.java index b1635f56401..cdddf4951af 100644 --- a/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/MesonBuildTab.java +++ b/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/MesonBuildTab.java @@ -31,6 +31,7 @@ import org.eclipse.swt.widgets.Text; public class MesonBuildTab extends CommonBuildTab { + private static final String NINJA = "Ninja"; private Button unixGenButton; private Button ninjaGenButton; private Text mesonArgsText; @@ -131,7 +132,7 @@ public class MesonBuildTab extends CommonBuildTab { } private void updateGeneratorButtons(String generator) { - if (generator == null || generator.equals("Ninja")) { //$NON-NLS-1$ + if (generator == null || generator.equals(NINJA)) { //$NON-NLS-1$ ninjaGenButton.setSelection(true); } else { unixGenButton.setSelection(true); @@ -145,7 +146,7 @@ public class MesonBuildTab extends CommonBuildTab { ICBuildConfiguration buildConfig = getBuildConfiguration(); buildConfig.setProperty(IMesonConstants.MESON_GENERATOR, - ninjaGenButton.getSelection() ? "Ninja" : "Unix Makefiles"); //$NON-NLS-1$ //$NON-NLS-2$ + ninjaGenButton.getSelection() ? NINJA : "Unix Makefiles"); //$NON-NLS-1$ //$NON-NLS-2$ String mesonArgs = mesonArgsText.getText().trim(); if (!mesonArgs.isEmpty()) { @@ -173,7 +174,7 @@ public class MesonBuildTab extends CommonBuildTab { protected void saveProperties(Map properties) { super.saveProperties(properties); properties.put(IMesonConstants.MESON_GENERATOR, - ninjaGenButton.getSelection() ? "Ninja" : "Unix Makefiles"); //$NON-NLS-1$ //$NON-NLS-2$ + ninjaGenButton.getSelection() ? NINJA : "Unix Makefiles"); //$NON-NLS-1$ //$NON-NLS-2$ properties.put(IMesonConstants.MESON_ARGUMENTS, mesonArgsText.getText().trim()); properties.put(IMesonConstants.BUILD_COMMAND, buildCommandText.getText().trim()); @@ -187,7 +188,7 @@ public class MesonBuildTab extends CommonBuildTab { String gen = properties.get(IMesonConstants.MESON_GENERATOR); if (gen != null) { switch (gen) { - case "Ninja": //$NON-NLS-1$ + case NINJA: //$NON-NLS-1$ ninjaGenButton.setSelection(true); unixGenButton.setSelection(false); break; diff --git a/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/SWTImagesFactory.java b/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/SWTImagesFactory.java index 72a7f02911b..9158be1bd4e 100644 --- a/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/SWTImagesFactory.java +++ b/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/SWTImagesFactory.java @@ -48,8 +48,8 @@ public class SWTImagesFactory { private static ImageDescriptor createManaged(ImageRegistry registry, String prefix, String name) { - ImageDescriptor result = ImageDescriptor.createFromURL(makeIconFileURL( - prefix, name.substring(NAME_PREFIX_LENGTH))); + ImageDescriptor result = ImageDescriptor + .createFromURL(makeIconFileURL(prefix, name.substring(NAME_PREFIX_LENGTH))); registry.put(name, result); return result; } @@ -85,8 +85,9 @@ public class SWTImagesFactory { */ public static void setImageDescriptors(IAction action, String type, String relPath) { - if (relPath.startsWith(NAME_PREFIX)) + if (relPath.startsWith(NAME_PREFIX)) { relPath = relPath.substring(NAME_PREFIX_LENGTH); + } action.setDisabledImageDescriptor(create("d" + type, relPath)); //$NON-NLS-1$ action.setImageDescriptor(create("e" + type, relPath)); //$NON-NLS-1$ diff --git a/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/commands/AbstractMesonCommandHandler.java b/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/commands/AbstractMesonCommandHandler.java index ea3b887e28f..0775c352ec7 100644 --- a/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/commands/AbstractMesonCommandHandler.java +++ b/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/commands/AbstractMesonCommandHandler.java @@ -55,8 +55,9 @@ public abstract class AbstractMesonCommandHandler extends AbstractHandler { if (obj instanceof Collection) { Collection c = (Collection)obj; Object[] objArray = c.toArray(); - if (objArray.length > 0) + if (objArray.length > 0) { obj = objArray[0]; + } } if (obj instanceof ICElement) { if ( obj instanceof ICContainer || obj instanceof ICProject) { @@ -79,8 +80,6 @@ public abstract class AbstractMesonCommandHandler extends AbstractHandler { return fContainer; } - public final String SHELL_COMMAND = "sh"; //$NON-NLS-1$ - protected void showError(String title, String content) { MessageDialog.openError(new Shell(), title, content); } @@ -174,10 +173,11 @@ public abstract class AbstractMesonCommandHandler extends AbstractHandler { // Skip white-space while (Character.isWhitespace(ch)) { ++i; - if (i < rawArgList.length()) + if (i < rawArgList.length()) { ch = rawArgList.charAt(i); - else // Otherwise we are done + } else { // Otherwise we are done return argList; + } } // Simplistic parser. We break up into strings delimited @@ -188,22 +188,21 @@ public abstract class AbstractMesonCommandHandler extends AbstractHandler { boolean inString = false; while (i < rawArgList.length()) { ch = rawArgList.charAt(i); - if (ch == '\\') // escape character + if (ch == '\\') { // escape character ++i; // skip over the next character - else if (ch == '\"') { // double quotes + } else if (ch == '\"') { // double quotes inString = !inString; - } else if (Character.isWhitespace(ch)) { - if (!inString) { - argList.add(rawArgList.substring(lastArgIndex, i)); - break; - } + } else if (Character.isWhitespace(ch) && !inString) { + argList.add(rawArgList.substring(lastArgIndex, i)); + break; } ++i; } // Look for the case where we ran out of chars for the last // token. - if (i >= rawArgList.length()) + if (i >= rawArgList.length()) { argList.add(rawArgList.substring(lastArgIndex)); + } ++i; } return argList; diff --git a/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/wizards/WizardMessages.java b/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/wizards/WizardMessages.java index 51c2d3c9880..17b98a584e7 100644 --- a/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/wizards/WizardMessages.java +++ b/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/internal/meson/ui/wizards/WizardMessages.java @@ -12,7 +12,7 @@ package org.eclipse.cdt.internal.meson.ui.wizards; import org.eclipse.osgi.util.NLS; -public class WizardMessages extends NLS { +public final class WizardMessages extends NLS { public static String RunNinjaPage_name; public static String RunNinjaPage_description; diff --git a/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/meson/ui/properties/IMesonPropertyPageControl.java b/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/meson/ui/properties/IMesonPropertyPageControl.java index df3b3ab6358..757214687c6 100644 --- a/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/meson/ui/properties/IMesonPropertyPageControl.java +++ b/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/meson/ui/properties/IMesonPropertyPageControl.java @@ -40,7 +40,7 @@ public interface IMesonPropertyPageControl { * Get the command line parameter if already configured * @return String containing command-line for configured build dir */ - public default String getConfiguredString() { + default String getConfiguredString() { return "-D" + getFieldName() + "=" + getFieldValue(); //$NON-NLS-1$ //$NON-NLS-2$ } @@ -48,8 +48,7 @@ public interface IMesonPropertyPageControl { * Get the command line parameter if never configured * @return String containing command-line parm for configured build dir */ - public default String getUnconfiguredString() { - // TODO Auto-generated method stub + default String getUnconfiguredString() { return "--" + getFieldName() + "=" + getFieldValue(); //$NON-NLS-1$ //$NON-NLS-2$ } diff --git a/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/meson/ui/properties/MesonPropertyPage.java b/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/meson/ui/properties/MesonPropertyPage.java index 03a930d4467..601284734c5 100644 --- a/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/meson/ui/properties/MesonPropertyPage.java +++ b/build/org.eclipse.cdt.meson.ui/src/org/eclipse/cdt/meson/ui/properties/MesonPropertyPage.java @@ -339,15 +339,8 @@ public class MesonPropertyPage extends PropertyPage { Pattern descPattern1 = Pattern.compile("([^\\.]+).*"); Pattern descPattern = Pattern.compile("([^\\(]*)(\\(default\\:\\s+([^\\)]+)\\).*)"); for (String line : lines) { -// System.out.println(line); Matcher optionMatcher = optionPattern.matcher(line); if (optionMatcher.matches() && !optionMatcher.group(2).equals("help")) { -// System.out.println("group 1 is " + (optionMatcher.group(1) != null ? optionMatcher.group(1).trim() : null)); -// System.out.println("group 2 is " + (optionMatcher.group(2) != null ? optionMatcher.group(2).trim() : null)); -// System.out.println("group 3 is " + (optionMatcher.group(3) != null ? optionMatcher.group(3).trim() : null)); -// System.out.println("group 4 is " + (optionMatcher.group(4) != null ? optionMatcher.group(4).trim() : null)); -// System.out.println("group 5 is " + (optionMatcher.group(5) != null ? optionMatcher.group(5).trim() : null)); -// System.out.println("group 6 is " + (optionMatcher.group(6) != null ? optionMatcher.group(6).trim() : null)); if (optionMatcher.group(3) != null) { String defaultValue = argMap.get(optionMatcher.group(2)); String description = optionMatcher.group(6);