From 45152b52080a71b8ef2c6d904375085fa3763e8e Mon Sep 17 00:00:00 2001 From: Stefan Bodewig Date: Wed, 21 Dec 2016 18:50:29 +0100 Subject: [PATCH 1/6] Sonar meets backwards compatibility requirements --- src/main/org/apache/tools/ant/DirectoryScanner.java | 2 +- src/main/org/apache/tools/ant/taskdefs/Javadoc.java | 2 +- src/main/org/apache/tools/ant/taskdefs/Length.java | 2 +- .../tools/ant/taskdefs/optional/javacc/JavaCC.java | 4 ++-- .../ant/taskdefs/optional/jsp/JspNameMangler.java | 2 +- .../apache/tools/ant/taskdefs/optional/net/FTP.java | 6 +++--- .../tools/ant/taskdefs/optional/net/FTPTask.java | 6 +++--- .../org/apache/tools/ant/util/Base64Converter.java | 2 +- .../org/apache/tools/ant/util/CollectionUtils.java | 5 +---- src/main/org/apache/tools/bzip2/BZip2Constants.java | 2 +- src/main/org/apache/tools/bzip2/CRC.java | 2 +- src/main/org/apache/tools/zip/ZipOutputStream.java | 12 ++++++------ 12 files changed, 22 insertions(+), 25 deletions(-) diff --git a/src/main/org/apache/tools/ant/DirectoryScanner.java b/src/main/org/apache/tools/ant/DirectoryScanner.java index 709779a73..6b5289792 100644 --- a/src/main/org/apache/tools/ant/DirectoryScanner.java +++ b/src/main/org/apache/tools/ant/DirectoryScanner.java @@ -146,7 +146,7 @@ public class DirectoryScanner * method instead. */ @Deprecated - protected static final String[] DEFAULTEXCLUDES = { + protected static final String[] DEFAULTEXCLUDES = { //NOSONAR // Miscellaneous typical temporary files SelectorUtils.DEEP_TREE_MATCH + "/*~", SelectorUtils.DEEP_TREE_MATCH + "/#*#", diff --git a/src/main/org/apache/tools/ant/taskdefs/Javadoc.java b/src/main/org/apache/tools/ant/taskdefs/Javadoc.java index 9fd3c8a3f..ebdce4965 100644 --- a/src/main/org/apache/tools/ant/taskdefs/Javadoc.java +++ b/src/main/org/apache/tools/ant/taskdefs/Javadoc.java @@ -1313,7 +1313,7 @@ public class Javadoc extends Task { * cannot be static in inner classes.) The first letter * from each element is used to build up the scope string. */ - static final String[] SCOPE_ELEMENTS = { + static final String[] SCOPE_ELEMENTS = { //NOSONAR "overview", "packages", "types", "constructors", "methods", "fields" }; diff --git a/src/main/org/apache/tools/ant/taskdefs/Length.java b/src/main/org/apache/tools/ant/taskdefs/Length.java index bd4da4424..68fb20a34 100644 --- a/src/main/org/apache/tools/ant/taskdefs/Length.java +++ b/src/main/org/apache/tools/ant/taskdefs/Length.java @@ -247,7 +247,7 @@ public class Length extends Task implements Condition { /** EnumeratedAttribute operation mode */ public static class FileMode extends EnumeratedAttribute { - static final String[] MODES = new String[] {EACH, ALL}; + static final String[] MODES = new String[] {EACH, ALL}; //NOSONAR /** * Return the possible values for FileMode. diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/javacc/JavaCC.java b/src/main/org/apache/tools/ant/taskdefs/optional/javacc/JavaCC.java index 219cc9c8d..999110fb6 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/javacc/JavaCC.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/javacc/JavaCC.java @@ -77,7 +77,7 @@ public class JavaCC extends Task { protected static final int TASKDEF_TYPE_JJTREE = 2; protected static final int TASKDEF_TYPE_JJDOC = 3; - protected static final String[] ARCHIVE_LOCATIONS = + protected static final String[] ARCHIVE_LOCATIONS = //NOSONAR new String[] { "JavaCC.zip", "bin/lib/JavaCC.zip", @@ -85,7 +85,7 @@ public class JavaCC extends Task { "javacc.jar", // used by jpackage for JavaCC 3.x }; - protected static final int[] ARCHIVE_LOCATIONS_VS_MAJOR_VERSION = + protected static final int[] ARCHIVE_LOCATIONS_VS_MAJOR_VERSION = //NOSONAR new int[] { 1, 2, diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/jsp/JspNameMangler.java b/src/main/org/apache/tools/ant/taskdefs/optional/jsp/JspNameMangler.java index 6e08e7d5e..850c6bca9 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/jsp/JspNameMangler.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/jsp/JspNameMangler.java @@ -33,7 +33,7 @@ public class JspNameMangler implements JspMangler { /** * this is the list of keywords which can not be used as classnames */ - public static final String[] keywords = { + public static final String[] keywords = { //NOSONAR "assert", "abstract", "boolean", "break", "byte", "case", "catch", "char", "class", diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java b/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java index b6360caa4..b78370657 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/net/FTP.java @@ -144,7 +144,7 @@ public class FTP extends Task implements FTPTaskConfig { private String initialSiteCommand = null; private boolean enableRemoteVerification = true; - protected static final String[] ACTION_STRS = { + protected static final String[] ACTION_STRS = { //NOSONAR "sending", "getting", "deleting", @@ -155,7 +155,7 @@ public class FTP extends Task implements FTPTaskConfig { "site" }; - protected static final String[] COMPLETED_ACTION_STRS = { + protected static final String[] COMPLETED_ACTION_STRS = { //NOSONAR "sent", "retrieved", "deleted", @@ -166,7 +166,7 @@ public class FTP extends Task implements FTPTaskConfig { "site command executed" }; - protected static final String[] ACTION_TARGET_STRS = { + protected static final String[] ACTION_TARGET_STRS = { //NOSONAR "files", "files", "files", diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPTask.java b/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPTask.java index 79780c78c..aa060ae2a 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPTask.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/net/FTPTask.java @@ -108,7 +108,7 @@ public class FTPTask extends Task implements FTPTaskConfig { private ClassLoader mirrorLoader; private FTPTaskMirror delegate = null; - public static final String[] ACTION_STRS = { + public static final String[] ACTION_STRS = { //NOSONAR "sending", "getting", "deleting", @@ -119,7 +119,7 @@ public class FTPTask extends Task implements FTPTaskConfig { "site" }; - public static final String[] COMPLETED_ACTION_STRS = { + public static final String[] COMPLETED_ACTION_STRS = { //NOSONAR "sent", "retrieved", "deleted", @@ -130,7 +130,7 @@ public class FTPTask extends Task implements FTPTaskConfig { "site command executed" }; - public static final String[] ACTION_TARGET_STRS = { + public static final String[] ACTION_TARGET_STRS = { //NOSONAR "files", "files", "files", diff --git a/src/main/org/apache/tools/ant/util/Base64Converter.java b/src/main/org/apache/tools/ant/util/Base64Converter.java index 5d60a145f..4f62b0357 100644 --- a/src/main/org/apache/tools/ant/util/Base64Converter.java +++ b/src/main/org/apache/tools/ant/util/Base64Converter.java @@ -49,7 +49,7 @@ public class Base64Converter { // CheckStyle:ConstantNameCheck OFF - bc /** Provided for BC purposes */ - public static final char[] alphabet = ALPHABET; + public static final char[] alphabet = ALPHABET; //NOSONAR // CheckStyle:ConstantNameCheck ON diff --git a/src/main/org/apache/tools/ant/util/CollectionUtils.java b/src/main/org/apache/tools/ant/util/CollectionUtils.java index 03c48d937..23f67a4d8 100644 --- a/src/main/org/apache/tools/ant/util/CollectionUtils.java +++ b/src/main/org/apache/tools/ant/util/CollectionUtils.java @@ -36,12 +36,9 @@ import java.util.Vector; */ public class CollectionUtils { - /** - * Collections.emptyList() is Java5+. - */ @SuppressWarnings("rawtypes") @Deprecated - public static final List EMPTY_LIST = Collections.EMPTY_LIST; + public static final List EMPTY_LIST = Collections.EMPTY_LIST; //NOSONAR /** * Please use Vector.equals() or List.equals(). diff --git a/src/main/org/apache/tools/bzip2/BZip2Constants.java b/src/main/org/apache/tools/bzip2/BZip2Constants.java index 3a511a7aa..df32d958f 100644 --- a/src/main/org/apache/tools/bzip2/BZip2Constants.java +++ b/src/main/org/apache/tools/bzip2/BZip2Constants.java @@ -52,7 +52,7 @@ public interface BZip2Constants { *

FIXME: This array should be in a private or package private * location, since it could be modified by malicious code.

*/ - int[] rNums = { + int[] rNums = { //NOSONAR 619, 720, 127, 481, 931, 816, 813, 233, 566, 247, 985, 724, 205, 454, 863, 491, 741, 242, 949, 214, 733, 859, 335, 708, 621, 574, 73, 654, 730, 472, diff --git a/src/main/org/apache/tools/bzip2/CRC.java b/src/main/org/apache/tools/bzip2/CRC.java index 0102c8e39..0d4ada902 100644 --- a/src/main/org/apache/tools/bzip2/CRC.java +++ b/src/main/org/apache/tools/bzip2/CRC.java @@ -30,7 +30,7 @@ package org.apache.tools.bzip2; * */ final class CRC { - static final int crc32Table[] = { + static final int crc32Table[] = { //NOSONAR 0x00000000, 0x04c11db7, 0x09823b6e, 0x0d4326d9, 0x130476dc, 0x17c56b6b, 0x1a864db2, 0x1e475005, 0x2608edb8, 0x22c9f00f, 0x2f8ad6d6, 0x2b4bcb61, diff --git a/src/main/org/apache/tools/zip/ZipOutputStream.java b/src/main/org/apache/tools/zip/ZipOutputStream.java index 0b4f5f902..228ce614f 100644 --- a/src/main/org/apache/tools/zip/ZipOutputStream.java +++ b/src/main/org/apache/tools/zip/ZipOutputStream.java @@ -963,33 +963,33 @@ public class ZipOutputStream extends FilterOutputStream { * * @since 1.1 */ - protected static final byte[] LFH_SIG = ZipLong.LFH_SIG.getBytes(); + protected static final byte[] LFH_SIG = ZipLong.LFH_SIG.getBytes(); //NOSONAR /** * data descriptor signature * * @since 1.1 */ - protected static final byte[] DD_SIG = ZipLong.DD_SIG.getBytes(); + protected static final byte[] DD_SIG = ZipLong.DD_SIG.getBytes(); //NOSONAR /** * central file header signature * * @since 1.1 */ - protected static final byte[] CFH_SIG = ZipLong.CFH_SIG.getBytes(); + protected static final byte[] CFH_SIG = ZipLong.CFH_SIG.getBytes(); //NOSONAR /** * end of central dir signature * * @since 1.1 */ - protected static final byte[] EOCD_SIG = ZipLong.getBytes(0X06054B50L); + protected static final byte[] EOCD_SIG = ZipLong.getBytes(0X06054B50L); //NOSONAR /** * ZIP64 end of central dir signature */ - static final byte[] ZIP64_EOCD_SIG = ZipLong.getBytes(0X06064B50L); + static final byte[] ZIP64_EOCD_SIG = ZipLong.getBytes(0X06064B50L); //NOSONAR /** * ZIP64 end of central dir locator signature */ - static final byte[] ZIP64_EOCD_LOC_SIG = ZipLong.getBytes(0X07064B50L); + static final byte[] ZIP64_EOCD_LOC_SIG = ZipLong.getBytes(0X07064B50L); //NOSONAR /** * Writes next block of compressed data to the output stream. From 0555c6d5f6d17e6861ef5c92d21d19caf2fe4721 Mon Sep 17 00:00:00 2001 From: Stefan Bodewig Date: Wed, 21 Dec 2016 18:50:53 +0100 Subject: [PATCH 2/6] "hard coded password" --- src/main/org/apache/tools/ant/taskdefs/optional/sos/SOS.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/sos/SOS.java b/src/main/org/apache/tools/ant/taskdefs/optional/sos/SOS.java index 03031d5ed..c6614cbdd 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/sos/SOS.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/sos/SOS.java @@ -41,7 +41,7 @@ public abstract class SOS extends Task implements SOSCmd { private String sosCmdDir = null; private String sosUsername = null; - private String sosPassword = ""; + private String sosPassword = null; private String projectPath = null; private String vssServerPath = null; private String sosServerPath = null; @@ -269,7 +269,7 @@ public abstract class SOS extends Task implements SOSCmd { * @return empty string if it wasn't set. */ protected String getPassword() { - return sosPassword; + return sosPassword == null ? "" : sosPassword; } /** From 4cecc03af634d050ab17667ab63e22a843b88f45 Mon Sep 17 00:00:00 2001 From: Stefan Bodewig Date: Wed, 21 Dec 2016 19:56:31 +0100 Subject: [PATCH 3/6] there simply is no better logger around --- src/main/org/apache/tools/ant/Main.java | 6 +++--- src/main/org/apache/tools/ant/Project.java | 2 +- .../org/apache/tools/ant/ProjectHelperRepository.java | 6 +++--- .../org/apache/tools/ant/filters/ReplaceTokens.java | 7 ++++++- src/main/org/apache/tools/ant/launch/Launcher.java | 4 ++-- src/main/org/apache/tools/ant/listener/MailLogger.java | 2 +- src/main/org/apache/tools/ant/taskdefs/Execute.java | 4 ++-- src/main/org/apache/tools/ant/taskdefs/KeySubst.java | 2 +- .../apache/tools/ant/taskdefs/ProcessDestroyer.java | 10 +++++----- .../org/apache/tools/ant/taskdefs/RecorderEntry.java | 2 +- .../tools/ant/taskdefs/optional/PropertyFile.java | 2 +- .../tools/ant/taskdefs/optional/ejb/IPlanetEjbc.java | 4 ++-- .../ant/taskdefs/optional/junit/JUnitTestRunner.java | 2 +- .../ant/taskdefs/optional/sound/AntSoundPlayer.java | 4 ++-- .../modifiedselector/PropertiesfileCache.java | 4 ++-- .../tools/ant/util/LayoutPreservingProperties.java | 2 +- 16 files changed, 34 insertions(+), 29 deletions(-) diff --git a/src/main/org/apache/tools/ant/Main.java b/src/main/org/apache/tools/ant/Main.java index 647fc9fa9..4626e14d7 100644 --- a/src/main/org/apache/tools/ant/Main.java +++ b/src/main/org/apache/tools/ant/Main.java @@ -246,7 +246,7 @@ public class Main implements AntMain { printMessage(be); } } catch (final Throwable exc) { - exc.printStackTrace(); + exc.printStackTrace(); //NOSONAR printMessage(exc); } finally { handleLogfile(); @@ -881,11 +881,11 @@ public class Main implements AntMain { // but if we don't, we lose valuable information System.err.println("Caught an exception while logging the" + " end of the build. Exception was:"); - t.printStackTrace(); + t.printStackTrace(); //NOSONAR if (error != null) { System.err.println("There has been an error prior to" + " that:"); - error.printStackTrace(); + error.printStackTrace(); //NOSONAR } throw new BuildException(t); //NOSONAR } diff --git a/src/main/org/apache/tools/ant/Project.java b/src/main/org/apache/tools/ant/Project.java index e9160024f..827183d29 100644 --- a/src/main/org/apache/tools/ant/Project.java +++ b/src/main/org/apache/tools/ant/Project.java @@ -873,7 +873,7 @@ public class Project implements ResourceFactory { try { setBasedir("."); } catch (final BuildException ex) { - ex.printStackTrace(); + ex.printStackTrace(); //NOSONAR } } return baseDir; diff --git a/src/main/org/apache/tools/ant/ProjectHelperRepository.java b/src/main/org/apache/tools/ant/ProjectHelperRepository.java index 1dd44124f..02ad62d85 100644 --- a/src/main/org/apache/tools/ant/ProjectHelperRepository.java +++ b/src/main/org/apache/tools/ant/ProjectHelperRepository.java @@ -108,7 +108,7 @@ public class ProjectHelperRepository { + e.getClass().getName() + ": " + e.getMessage() + ")"); if (DEBUG) { - e.printStackTrace(System.err); + e.printStackTrace(System.err); //NOSONAR } } } @@ -175,7 +175,7 @@ public class ProjectHelperRepository { + ProjectHelper.HELPER_PROPERTY + " (" + e.getMessage() + ")"); if (DEBUG) { - e.printStackTrace(System.err); + e.printStackTrace(System.err); //NOSONAR } } return null; @@ -203,7 +203,7 @@ public class ProjectHelperRepository { System.out.println("Unable to load ProjectHelper from service " + ProjectHelper.SERVICE_ID + " (" + e.getMessage() + ")"); if (DEBUG) { - e.printStackTrace(System.err); + e.printStackTrace(System.err); //NOSONAR } } return null; diff --git a/src/main/org/apache/tools/ant/filters/ReplaceTokens.java b/src/main/org/apache/tools/ant/filters/ReplaceTokens.java index 21ca3bc91..74da64a16 100644 --- a/src/main/org/apache/tools/ant/filters/ReplaceTokens.java +++ b/src/main/org/apache/tools/ant/filters/ReplaceTokens.java @@ -27,6 +27,7 @@ import java.util.Properties; import java.util.SortedMap; import java.util.TreeMap; +import org.apache.tools.ant.Project; import org.apache.tools.ant.types.Parameter; import org.apache.tools.ant.types.Resource; import org.apache.tools.ant.types.resources.FileResource; @@ -244,7 +245,11 @@ public final class ReplaceTokens in = resource.getInputStream(); props.load(in); } catch (IOException ioe) { - ioe.printStackTrace(); + if (getProject() != null) { + getProject().log("getProperties failed, " + ioe.getMessage(), Project.MSG_ERR); + } else { + ioe.printStackTrace(); //NOSONAR + } } finally { FileUtils.close(in); } diff --git a/src/main/org/apache/tools/ant/launch/Launcher.java b/src/main/org/apache/tools/ant/launch/Launcher.java index 364bc5820..534bbfb3d 100644 --- a/src/main/org/apache/tools/ant/launch/Launcher.java +++ b/src/main/org/apache/tools/ant/launch/Launcher.java @@ -117,7 +117,7 @@ public class Launcher { System.err.println(e.getMessage()); } catch (final Throwable t) { exitCode = EXIT_CODE_ERROR; - t.printStackTrace(System.err); + t.printStackTrace(System.err); //NOSONAR } if (exitCode != 0) { if (launchDiag) { @@ -297,7 +297,7 @@ public class Launcher { "Failed to locate" + mainClassname); thrown = cnfe; } catch (final Throwable t) { - t.printStackTrace(System.err); + t.printStackTrace(System.err); //NOSONAR thrown=t; } if(thrown!=null) { diff --git a/src/main/org/apache/tools/ant/listener/MailLogger.java b/src/main/org/apache/tools/ant/listener/MailLogger.java index 4b50547f3..3d5bcd235 100644 --- a/src/main/org/apache/tools/ant/listener/MailLogger.java +++ b/src/main/org/apache/tools/ant/listener/MailLogger.java @@ -177,7 +177,7 @@ public class MailLogger extends DefaultLogger { } } catch (Exception e) { System.out.println("MailLogger failed to send e-mail!"); - e.printStackTrace(System.err); + e.printStackTrace(System.err); //NOSONAR } } diff --git a/src/main/org/apache/tools/ant/taskdefs/Execute.java b/src/main/org/apache/tools/ant/taskdefs/Execute.java index 128caa9eb..956309a6d 100644 --- a/src/main/org/apache/tools/ant/taskdefs/Execute.java +++ b/src/main/org/apache/tools/ant/taskdefs/Execute.java @@ -113,7 +113,7 @@ public class Execute { procEnvironment = System.getenv(); return procEnvironment; } catch (Exception x) { - x.printStackTrace(); + x.printStackTrace(); //NOSONAR } } @@ -162,7 +162,7 @@ public class Execute { procEnvironment.put(var.substring(0, eq), var.substring(eq + 1)); } } catch (java.io.IOException exc) { - exc.printStackTrace(); + exc.printStackTrace(); //NOSONAR // Just try to see how much we got } return procEnvironment; diff --git a/src/main/org/apache/tools/ant/taskdefs/KeySubst.java b/src/main/org/apache/tools/ant/taskdefs/KeySubst.java index 6ff67c040..28d46b9a1 100644 --- a/src/main/org/apache/tools/ant/taskdefs/KeySubst.java +++ b/src/main/org/apache/tools/ant/taskdefs/KeySubst.java @@ -153,7 +153,7 @@ public class KeySubst extends Task { System.out.println(KeySubst.replace("$f ${VERSION} f ${b} jj $", hash)); } catch (Exception e) { - e.printStackTrace(); + e.printStackTrace(); //NOSONAR } } diff --git a/src/main/org/apache/tools/ant/taskdefs/ProcessDestroyer.java b/src/main/org/apache/tools/ant/taskdefs/ProcessDestroyer.java index ba9c9d487..bc3ff4952 100644 --- a/src/main/org/apache/tools/ant/taskdefs/ProcessDestroyer.java +++ b/src/main/org/apache/tools/ant/taskdefs/ProcessDestroyer.java @@ -84,7 +84,7 @@ class ProcessDestroyer implements Runnable { } catch (NoSuchMethodException e) { // it just won't be added as a shutdown hook... :( } catch (Exception e) { - e.printStackTrace(); + e.printStackTrace(); //NOSONAR } } @@ -100,14 +100,14 @@ class ProcessDestroyer implements Runnable { addShutdownHookMethod.invoke(Runtime.getRuntime(), args); added = true; } catch (IllegalAccessException e) { - e.printStackTrace(); + e.printStackTrace(); //NOSONAR } catch (InvocationTargetException e) { Throwable t = e.getTargetException(); if (t != null && t.getClass() == IllegalStateException.class) { // shutdown already is in progress running = true; } else { - e.printStackTrace(); + e.printStackTrace(); //NOSONAR } } } @@ -129,14 +129,14 @@ class ProcessDestroyer implements Runnable { System.err.println("Could not remove shutdown hook"); } } catch (IllegalAccessException e) { - e.printStackTrace(); + e.printStackTrace(); //NOSONAR } catch (InvocationTargetException e) { Throwable t = e.getTargetException(); if (t != null && t.getClass() == IllegalStateException.class) { // shutdown already is in progress running = true; } else { - e.printStackTrace(); + e.printStackTrace(); //NOSONAR } } // start the hook thread, a unstarted thread may not be diff --git a/src/main/org/apache/tools/ant/taskdefs/RecorderEntry.java b/src/main/org/apache/tools/ant/taskdefs/RecorderEntry.java index dfb8e8453..a49593681 100644 --- a/src/main/org/apache/tools/ant/taskdefs/RecorderEntry.java +++ b/src/main/org/apache/tools/ant/taskdefs/RecorderEntry.java @@ -111,7 +111,7 @@ public class RecorderEntry implements BuildLogger, SubBuildListener { } else { out.println(StringUtils.LINE_SEP + "BUILD FAILED" + StringUtils.LINE_SEP); - error.printStackTrace(out); + error.printStackTrace(out); //NOSONAR } } cleanup(); diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/PropertyFile.java b/src/main/org/apache/tools/ant/taskdefs/optional/PropertyFile.java index 7c778dae9..02106a97d 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/PropertyFile.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/PropertyFile.java @@ -401,7 +401,7 @@ public class PropertyFile extends Task { } catch (NullPointerException npe) { // Default to string type // which means do nothing - npe.printStackTrace(); + npe.printStackTrace(); //NOSONAR } if (newValue == null) { diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/ejb/IPlanetEjbc.java b/src/main/org/apache/tools/ant/taskdefs/optional/ejb/IPlanetEjbc.java index 984fc3785..f9c2ef1aa 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/ejb/IPlanetEjbc.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/ejb/IPlanetEjbc.java @@ -310,7 +310,7 @@ public class IPlanetEjbc { // SAXException or ParserConfigurationException may be thrown System.out.println("An exception was generated while trying to "); System.out.println("create a new SAXParser."); - e.printStackTrace(); + e.printStackTrace(); //NOSONAR return; } @@ -1484,7 +1484,7 @@ public class IPlanetEjbc { System.out.println(text); } } catch (IOException e) { - e.printStackTrace(); + e.printStackTrace(); //NOSONAR } finally { FileUtils.close(reader); } diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java b/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java index 0397d843e..c7d39e096 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java @@ -1023,7 +1023,7 @@ public class JUnitTestRunner implements TestListener, JUnitTaskMirror.JUnitTestR } } } catch (final IOException e) { - e.printStackTrace(); + e.printStackTrace(); //NOSONAR } } else { final JUnitTest t = new JUnitTest(args[0]); diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/sound/AntSoundPlayer.java b/src/main/org/apache/tools/ant/taskdefs/optional/sound/AntSoundPlayer.java index aef214fe0..d1675a801 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/sound/AntSoundPlayer.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/sound/AntSoundPlayer.java @@ -109,7 +109,7 @@ public class AntSoundPlayer implements LineListener, BuildListener { project.log("Audio format is not yet supported: " + uafe.getMessage()); } catch (IOException ioe) { - ioe.printStackTrace(); + ioe.printStackTrace(); //NOSONAR } if (audioInputStream != null) { @@ -125,7 +125,7 @@ public class AntSoundPlayer implements LineListener, BuildListener { project.log("The sound device is currently unavailable"); return; } catch (IOException e) { - e.printStackTrace(); + e.printStackTrace(); //NOSONAR } if (duration != null) { diff --git a/src/main/org/apache/tools/ant/types/selectors/modifiedselector/PropertiesfileCache.java b/src/main/org/apache/tools/ant/types/selectors/modifiedselector/PropertiesfileCache.java index 1446e8904..e8cf3fea4 100644 --- a/src/main/org/apache/tools/ant/types/selectors/modifiedselector/PropertiesfileCache.java +++ b/src/main/org/apache/tools/ant/types/selectors/modifiedselector/PropertiesfileCache.java @@ -137,7 +137,7 @@ public class PropertiesfileCache implements Cache { cache.load(bis); bis.close(); } catch (Exception e) { - e.printStackTrace(); + e.printStackTrace(); //NOSONAR } } // after loading the cache is up to date with the file @@ -164,7 +164,7 @@ public class PropertiesfileCache implements Cache { bos.flush(); bos.close(); } catch (Exception e) { - e.printStackTrace(); + e.printStackTrace(); //NOSONAR } } cacheDirty = false; diff --git a/src/main/org/apache/tools/ant/util/LayoutPreservingProperties.java b/src/main/org/apache/tools/ant/util/LayoutPreservingProperties.java index aed6f3717..eceb9a0b2 100644 --- a/src/main/org/apache/tools/ant/util/LayoutPreservingProperties.java +++ b/src/main/org/apache/tools/ant/util/LayoutPreservingProperties.java @@ -700,7 +700,7 @@ public class LayoutPreservingProperties extends Properties { dolly = super.clone(); } catch (final CloneNotSupportedException e) { // should be fine - e.printStackTrace(); + e.printStackTrace(); //NOSONAR } return dolly; } From 008f1c8be554f9ed8928fdf6596a6c6c3fa5b063 Mon Sep 17 00:00:00 2001 From: Stefan Bodewig Date: Wed, 21 Dec 2016 19:57:36 +0100 Subject: [PATCH 4/6] embrace StringUtils#getStackTrace --- src/main/org/apache/tools/ant/AntClassLoader.java | 7 ++++--- src/main/org/apache/tools/ant/taskdefs/Classloader.java | 3 ++- src/main/org/apache/tools/ant/taskdefs/Java.java | 7 ++----- src/main/org/apache/tools/ant/taskdefs/KeySubst.java | 4 +++- src/main/org/apache/tools/ant/taskdefs/SubAnt.java | 4 ++-- src/main/org/apache/tools/ant/taskdefs/XSLTProcess.java | 3 ++- .../tools/ant/taskdefs/optional/ccm/CCMCreateTask.java | 4 ++-- .../tools/ant/taskdefs/optional/ejb/IPlanetEjbc.java | 3 ++- .../apache/tools/ant/taskdefs/optional/image/Image.java | 3 ++- .../tools/ant/taskdefs/optional/junit/FailureRecorder.java | 3 ++- .../tools/ant/taskdefs/optional/junit/JUnitTask.java | 3 ++- .../junit/org/apache/tools/ant/AntClassLoaderTest.java | 4 ---- 12 files changed, 25 insertions(+), 23 deletions(-) diff --git a/src/main/org/apache/tools/ant/AntClassLoader.java b/src/main/org/apache/tools/ant/AntClassLoader.java index 96b92fc0d..9fa9eae2b 100644 --- a/src/main/org/apache/tools/ant/AntClassLoader.java +++ b/src/main/org/apache/tools/ant/AntClassLoader.java @@ -49,6 +49,7 @@ import org.apache.tools.ant.util.CollectionUtils; import org.apache.tools.ant.util.FileUtils; import org.apache.tools.ant.util.JavaEnvUtils; import org.apache.tools.ant.util.LoaderUtils; +import org.apache.tools.ant.util.StringUtils; import org.apache.tools.ant.util.VectorSet; import org.apache.tools.zip.ZipLong; @@ -401,6 +402,8 @@ public class AntClassLoader extends ClassLoader implements SubBuildListener, Clo protected void log(final String message, final int priority) { if (project != null) { project.log(message, priority); + } else if (priority < Project.MSG_INFO) { + System.err.println(message); } } @@ -1020,7 +1023,6 @@ public class AntClassLoader extends ClassLoader implements SubBuildListener, Clo final String msg = "CLASSPATH element " + file + " is not a JAR."; log(msg, Project.MSG_WARN); - System.err.println(msg); return null; } jarFile = new JarFile(file); @@ -1043,8 +1045,7 @@ public class AntClassLoader extends ClassLoader implements SubBuildListener, Clo } catch (final Exception e) { final String msg = "Unable to obtain resource from " + file + ": "; log(msg + e, Project.MSG_WARN); - System.err.println(msg); - e.printStackTrace(); + log(StringUtils.getStackTrace(e), Project.MSG_WARN); } return null; } diff --git a/src/main/org/apache/tools/ant/taskdefs/Classloader.java b/src/main/org/apache/tools/ant/taskdefs/Classloader.java index 8a5967c70..99d47a15d 100644 --- a/src/main/org/apache/tools/ant/taskdefs/Classloader.java +++ b/src/main/org/apache/tools/ant/taskdefs/Classloader.java @@ -27,6 +27,7 @@ import org.apache.tools.ant.Project; import org.apache.tools.ant.Task; import org.apache.tools.ant.types.Path; import org.apache.tools.ant.types.Reference; +import org.apache.tools.ant.util.StringUtils; /** * EXPERIMENTAL @@ -238,7 +239,7 @@ public class Classloader extends Task { // TODO add exceptions } catch (Exception ex) { - ex.printStackTrace(); + log(StringUtils.getStackTrace(ex), Project.MSG_ERR); } } } diff --git a/src/main/org/apache/tools/ant/taskdefs/Java.java b/src/main/org/apache/tools/ant/taskdefs/Java.java index ffe92cfad..ed9f90659 100644 --- a/src/main/org/apache/tools/ant/taskdefs/Java.java +++ b/src/main/org/apache/tools/ant/taskdefs/Java.java @@ -40,6 +40,7 @@ import org.apache.tools.ant.types.PropertySet; import org.apache.tools.ant.types.RedirectorElement; import org.apache.tools.ant.types.Reference; import org.apache.tools.ant.util.KeepAliveInputStream; +import org.apache.tools.ant.util.StringUtils; /** * Launcher for Java applications. Allows use of @@ -997,11 +998,7 @@ public class Java extends Task { * @since 1.6.2 */ private void log(Throwable t) { - StringWriter sw = new StringWriter(); - PrintWriter w = new PrintWriter(sw); - t.printStackTrace(w); - w.close(); - log(sw.toString(), Project.MSG_ERR); + log(StringUtils.getStackTrace(t), Project.MSG_ERR); } /** diff --git a/src/main/org/apache/tools/ant/taskdefs/KeySubst.java b/src/main/org/apache/tools/ant/taskdefs/KeySubst.java index 28d46b9a1..64fbf7cf2 100644 --- a/src/main/org/apache/tools/ant/taskdefs/KeySubst.java +++ b/src/main/org/apache/tools/ant/taskdefs/KeySubst.java @@ -28,8 +28,10 @@ import java.util.Hashtable; import java.util.StringTokenizer; import org.apache.tools.ant.BuildException; +import org.apache.tools.ant.Project; import org.apache.tools.ant.Task; import org.apache.tools.ant.util.FileUtils; +import org.apache.tools.ant.util.StringUtils; /** * Keyword substitution. Input file is written to output file. @@ -80,7 +82,7 @@ public class KeySubst extends Task { } bw.flush(); } catch (IOException ioe) { - ioe.printStackTrace(); + log(StringUtils.getStackTrace(ioe), Project.MSG_ERR); } finally { FileUtils.close(bw); FileUtils.close(br); diff --git a/src/main/org/apache/tools/ant/taskdefs/SubAnt.java b/src/main/org/apache/tools/ant/taskdefs/SubAnt.java index 35109faa5..c374d8326 100644 --- a/src/main/org/apache/tools/ant/taskdefs/SubAnt.java +++ b/src/main/org/apache/tools/ant/taskdefs/SubAnt.java @@ -34,7 +34,7 @@ import org.apache.tools.ant.types.Path; import org.apache.tools.ant.types.PropertySet; import org.apache.tools.ant.types.Reference; import org.apache.tools.ant.types.ResourceCollection; - +import org.apache.tools.ant.util.StringUtils; /** * Calls a given target for all defined sub-builds. This is an extension @@ -255,7 +255,7 @@ public class SubAnt extends Task { log("Target '" + file + "' failed with message '" + thrownException.getMessage() + "'.", Project.MSG_ERR); - thrownException.printStackTrace(System.err); + log(StringUtils.getStackTrace(thrownException), Project.MSG_ERR); if (buildException == null) { buildException = new BuildException(thrownException); diff --git a/src/main/org/apache/tools/ant/taskdefs/XSLTProcess.java b/src/main/org/apache/tools/ant/taskdefs/XSLTProcess.java index 2e2f005ea..75445928a 100644 --- a/src/main/org/apache/tools/ant/taskdefs/XSLTProcess.java +++ b/src/main/org/apache/tools/ant/taskdefs/XSLTProcess.java @@ -58,6 +58,7 @@ import org.apache.tools.ant.util.ClasspathUtils; import org.apache.tools.ant.util.FileNameMapper; import org.apache.tools.ant.util.FileUtils; import org.apache.tools.ant.util.ResourceUtils; +import org.apache.tools.ant.util.StringUtils; /** * Processes a set of XML documents via XSLT. This is @@ -952,7 +953,7 @@ public class XSLTProcess extends MatchingTask implements XSLTLogger { try { resolveProcessor(PROCESSOR_TRAX); } catch (final Throwable e1) { - e1.printStackTrace(); + log(StringUtils.getStackTrace(e1), Project.MSG_ERR); handleError(e1); } } diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/ccm/CCMCreateTask.java b/src/main/org/apache/tools/ant/taskdefs/optional/ccm/CCMCreateTask.java index cda13a5cb..350d673c7 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/ccm/CCMCreateTask.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/ccm/CCMCreateTask.java @@ -30,7 +30,7 @@ import org.apache.tools.ant.Project; import org.apache.tools.ant.taskdefs.Execute; import org.apache.tools.ant.taskdefs.ExecuteStreamHandler; import org.apache.tools.ant.types.Commandline; - +import org.apache.tools.ant.util.StringUtils; /** * Creates new Continuus ccm task and sets it as the default. @@ -322,7 +322,7 @@ public class CCMCreateTask extends Continuus implements ExecuteStreamHandler { } // end of if () } catch (NullPointerException npe) { log("error procession stream , null pointer exception", Project.MSG_ERR); - npe.printStackTrace(); + log(StringUtils.getStackTrace(npe), Project.MSG_ERR); throw new BuildException(npe.getClass().getName()); } catch (Exception e) { log("error procession stream " + e.getMessage(), Project.MSG_ERR); diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/ejb/IPlanetEjbc.java b/src/main/org/apache/tools/ant/taskdefs/optional/ejb/IPlanetEjbc.java index f9c2ef1aa..76b61ae36 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/ejb/IPlanetEjbc.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/ejb/IPlanetEjbc.java @@ -43,6 +43,7 @@ import org.xml.sax.InputSource; import org.xml.sax.SAXException; import org.apache.tools.ant.util.FileUtils; +import org.apache.tools.ant.util.StringUtils; /** * Compiles EJB stubs and skeletons for the iPlanet Application @@ -443,7 +444,7 @@ public class IPlanetEjbc { p.destroy(); } catch (IOException e) { log("An IOException has occurred while trying to execute ejbc."); - e.printStackTrace(); + log(StringUtils.getStackTrace(e)); } catch (InterruptedException e) { // Do nothing } diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/image/Image.java b/src/main/org/apache/tools/ant/taskdefs/optional/image/Image.java index 162e37560..9b3766e05 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/image/Image.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/image/Image.java @@ -40,6 +40,7 @@ import org.apache.tools.ant.types.optional.image.TransformOperation; import org.apache.tools.ant.util.FileNameMapper; import org.apache.tools.ant.util.FileUtils; import org.apache.tools.ant.util.IdentityMapper; +import org.apache.tools.ant.util.StringUtils; import com.sun.media.jai.codec.FileSeekableStream; @@ -392,7 +393,7 @@ public class Image extends MatchingTask { } } catch (Exception err) { - err.printStackTrace(); + log(StringUtils.getStackTrace(err), Project.MSG_ERR); throw new BuildException(err.getMessage()); } } diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junit/FailureRecorder.java b/src/main/org/apache/tools/ant/taskdefs/optional/junit/FailureRecorder.java index 4c90631da..bfcbfedb2 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/junit/FailureRecorder.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/junit/FailureRecorder.java @@ -38,6 +38,7 @@ import org.apache.tools.ant.BuildListener; import org.apache.tools.ant.Project; import org.apache.tools.ant.ProjectComponent; import org.apache.tools.ant.util.FileUtils; +import org.apache.tools.ant.util.StringUtils; /** *

Collects all failing test cases and creates a new JUnit test class containing @@ -260,7 +261,7 @@ public class FailureRecorder extends ProjectComponent implements JUnitResultForm createClassFooter(); } catch (IOException e) { - e.printStackTrace(); + log(StringUtils.getStackTrace(e)); } finally { FileUtils.close(writer); } diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java b/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java index a9e1662e4..e15e11edb 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTask.java @@ -63,6 +63,7 @@ import org.apache.tools.ant.types.PropertySet; import org.apache.tools.ant.util.FileUtils; import org.apache.tools.ant.util.LoaderUtils; import org.apache.tools.ant.util.SplitClassLoader; +import org.apache.tools.ant.util.StringUtils; /** * Runs JUnit tests. @@ -1299,7 +1300,7 @@ public class JUnitTask extends Task { + " testcase not started or mixing ant versions?"; } } catch (final Exception e) { - e.printStackTrace(); + log(StringUtils.getStackTrace(e), Project.MSG_INFO); // ignored. } finally { FileUtils.close(br); diff --git a/src/tests/junit/org/apache/tools/ant/AntClassLoaderTest.java b/src/tests/junit/org/apache/tools/ant/AntClassLoaderTest.java index e22bd05f2..568cbc168 100644 --- a/src/tests/junit/org/apache/tools/ant/AntClassLoaderTest.java +++ b/src/tests/junit/org/apache/tools/ant/AntClassLoaderTest.java @@ -195,10 +195,6 @@ public class AntClassLoaderTest { int startMessage = log.indexOf("CLASSPATH element "); assertTrue(startMessage >= 0); assertTrue(log.indexOf("foo.jar is not a JAR", startMessage) > 0); - log = errBuffer.toString(); - startMessage = log.indexOf("CLASSPATH element "); - assertTrue(startMessage >= 0); - assertTrue(log.indexOf("foo.jar is not a JAR", startMessage) > 0); } finally { System.setErr(sysErr); } From 984a03d1ceb6e4b5d194e4d639d0b0fca46d92be Mon Sep 17 00:00:00 2001 From: Stefan Bodewig Date: Wed, 21 Dec 2016 20:11:13 +0100 Subject: [PATCH 5/6] special paths can be final --- src/main/org/apache/tools/ant/types/Path.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/org/apache/tools/ant/types/Path.java b/src/main/org/apache/tools/ant/types/Path.java index db6f5e9f6..c4f244c74 100644 --- a/src/main/org/apache/tools/ant/types/Path.java +++ b/src/main/org/apache/tools/ant/types/Path.java @@ -67,7 +67,7 @@ public class Path extends DataType implements Cloneable, ResourceCollection { // CheckStyle:VisibilityModifier OFF - bc /** The system classpath as a Path object */ - public static Path systemClasspath = + public static final Path systemClasspath = new Path(null, System.getProperty("java.class.path")); @@ -76,7 +76,7 @@ public class Path extends DataType implements Cloneable, ResourceCollection { * * @since Ant 1.6.2 */ - public static Path systemBootClasspath = + public static final Path systemBootClasspath = new Path(null, System.getProperty("sun.boot.class.path")); // CheckStyle:VisibilityModifier OFF - bc From 121043fe596097cbcdde3fa54600848b1c67935c Mon Sep 17 00:00:00 2001 From: Stefan Bodewig Date: Wed, 21 Dec 2016 20:11:36 +0100 Subject: [PATCH 6/6] false positives --- src/main/org/apache/tools/ant/ArgumentProcessorRegistry.java | 2 +- src/main/org/apache/tools/ant/ComponentHelper.java | 2 +- src/main/org/apache/tools/ant/Diagnostics.java | 2 +- src/main/org/apache/tools/ant/taskdefs/HostInfo.java | 4 ++-- .../org/apache/tools/ant/taskdefs/optional/sos/SOSCmd.java | 2 +- src/main/org/apache/tools/ant/util/ProxySetup.java | 4 ++-- src/main/org/apache/tools/ant/util/StringUtils.java | 2 +- 7 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/main/org/apache/tools/ant/ArgumentProcessorRegistry.java b/src/main/org/apache/tools/ant/ArgumentProcessorRegistry.java index 0c7c35bd0..1ede565a8 100644 --- a/src/main/org/apache/tools/ant/ArgumentProcessorRegistry.java +++ b/src/main/org/apache/tools/ant/ArgumentProcessorRegistry.java @@ -93,7 +93,7 @@ public class ArgumentProcessorRegistry { + SERVICE_ID + " (" + e.getClass().getName() + ": " + e.getMessage() + ")"); if (DEBUG) { - e.printStackTrace(System.err); + e.printStackTrace(System.err); //NOSONAR } } } diff --git a/src/main/org/apache/tools/ant/ComponentHelper.java b/src/main/org/apache/tools/ant/ComponentHelper.java index eceedeef0..4942d2407 100644 --- a/src/main/org/apache/tools/ant/ComponentHelper.java +++ b/src/main/org/apache/tools/ant/ComponentHelper.java @@ -971,7 +971,7 @@ public class ComponentHelper { Throwable t = ex.getTargetException(); out.println("Cause: The constructor threw the exception"); out.println(t.toString()); - t.printStackTrace(out); + t.printStackTrace(out); //NOSONAR } catch (NoClassDefFoundError ncdfe) { jars = true; out.println("Cause: A class needed by class " + classname diff --git a/src/main/org/apache/tools/ant/Diagnostics.java b/src/main/org/apache/tools/ant/Diagnostics.java index 6389f6eef..4ad57a063 100644 --- a/src/main/org/apache/tools/ant/Diagnostics.java +++ b/src/main/org/apache/tools/ant/Diagnostics.java @@ -471,7 +471,7 @@ public final class Diagnostics { // report error if something weird happens...this is diagnostic. if (error != null) { out.println("Error while running org.apache.env.Which"); - error.printStackTrace(); + error.printStackTrace(out); //NOSONAR } } diff --git a/src/main/org/apache/tools/ant/taskdefs/HostInfo.java b/src/main/org/apache/tools/ant/taskdefs/HostInfo.java index 5cd433cad..43a121108 100644 --- a/src/main/org/apache/tools/ant/taskdefs/HostInfo.java +++ b/src/main/org/apache/tools/ant/taskdefs/HostInfo.java @@ -43,11 +43,11 @@ import org.apache.tools.ant.Task; public class HostInfo extends Task { private static final String DEF_REM_ADDR6 = "::"; - private static final String DEF_REM_ADDR4 = "0.0.0.0"; + private static final String DEF_REM_ADDR4 = "0.0.0.0"; //NOSONAR private static final String DEF_LOCAL_ADDR6 = "::1"; - private static final String DEF_LOCAL_ADDR4 = "127.0.0.1"; + private static final String DEF_LOCAL_ADDR4 = "127.0.0.1"; //NOSONAR private static final String DEF_LOCAL_NAME = "localhost"; private static final String DEF_DOMAIN = "localdomain"; diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/sos/SOSCmd.java b/src/main/org/apache/tools/ant/taskdefs/optional/sos/SOSCmd.java index 3543c4143..9d7d54325 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/sos/SOSCmd.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/sos/SOSCmd.java @@ -53,7 +53,7 @@ public interface SOSCmd { /** The username option */ String FLAG_USERNAME = "-name"; /** The password option */ - String FLAG_PASSWORD = "-password"; + String FLAG_PASSWORD = "-password"; //NOSONAR /** The log option */ String FLAG_COMMENT = "-log"; /** The workdir option */ diff --git a/src/main/org/apache/tools/ant/util/ProxySetup.java b/src/main/org/apache/tools/ant/util/ProxySetup.java index f077f87c7..ea69e7270 100644 --- a/src/main/org/apache/tools/ant/util/ProxySetup.java +++ b/src/main/org/apache/tools/ant/util/ProxySetup.java @@ -57,7 +57,7 @@ public class ProxySetup { /** the http proxy username property */ public static final String HTTP_PROXY_USERNAME = "http.proxyUser"; /** the http proxy password property */ - public static final String HTTP_PROXY_PASSWORD = "http.proxyPassword"; + public static final String HTTP_PROXY_PASSWORD = "http.proxyPassword"; //NOSONAR /** the socks proxy host property */ public static final String SOCKS_PROXY_HOST = "socksProxyHost"; /** the socks proxy port property */ @@ -65,7 +65,7 @@ public class ProxySetup { /** the socks proxy username property */ public static final String SOCKS_PROXY_USERNAME = "java.net.socks.username"; /** the socks proxy password property */ - public static final String SOCKS_PROXY_PASSWORD = "java.net.socks.password"; + public static final String SOCKS_PROXY_PASSWORD = "java.net.socks.password"; //NOSONAR /** * create a proxy setup class bound to this project diff --git a/src/main/org/apache/tools/ant/util/StringUtils.java b/src/main/org/apache/tools/ant/util/StringUtils.java index 626fb224b..dcae2b81b 100644 --- a/src/main/org/apache/tools/ant/util/StringUtils.java +++ b/src/main/org/apache/tools/ant/util/StringUtils.java @@ -93,7 +93,7 @@ public final class StringUtils { public static String getStackTrace(Throwable t) { StringWriter sw = new StringWriter(); PrintWriter pw = new PrintWriter(sw, true); - t.printStackTrace(pw); + t.printStackTrace(pw); //NOSONAR pw.flush(); pw.close(); return sw.toString();