Repository: ant Updated Branches: refs/heads/master 46b671665 -> 342bf0057
Review annotations Project: http://git-wip-us.apache.org/repos/asf/ant/repo Commit: http://git-wip-us.apache.org/repos/asf/ant/commit/342bf005 Tree: http://git-wip-us.apache.org/repos/asf/ant/tree/342bf005 Diff: http://git-wip-us.apache.org/repos/asf/ant/diff/342bf005 Branch: refs/heads/master Commit: 342bf00579318f00b373e86a148af984bc7d6acb Parents: 46b6716 Author: Gintas Grigelionis <[email protected]> Authored: Mon Apr 30 15:59:48 2018 +0200 Committer: Gintas Grigelionis <[email protected]> Committed: Mon Apr 30 17:29:44 2018 +0200 ---------------------------------------------------------------------- src/main/org/apache/tools/ant/DirectoryScanner.java | 1 - src/main/org/apache/tools/ant/RuntimeConfigurable.java | 3 ++- src/main/org/apache/tools/ant/input/SecureInputHandler.java | 1 + src/main/org/apache/tools/ant/launch/Locator.java | 1 - src/main/org/apache/tools/ant/listener/Log4jListener.java | 1 + src/main/org/apache/tools/ant/taskdefs/Concat.java | 1 - src/main/org/apache/tools/ant/taskdefs/CopyPath.java | 1 + .../tools/ant/taskdefs/compilers/DefaultCompilerAdapter.java | 1 - src/main/org/apache/tools/ant/taskdefs/email/Message.java | 1 + .../org/apache/tools/ant/taskdefs/optional/XMLValidateTask.java | 1 - .../apache/tools/ant/taskdefs/optional/jdepend/JDependTask.java | 2 -- .../tools/ant/taskdefs/optional/junit/JUnitTestRunner.java | 1 + .../ant/taskdefs/optional/junitlauncher/JUnitLauncherTask.java | 1 + .../optional/junitlauncher/LegacyPlainResultFormatter.java | 2 ++ .../optional/junitlauncher/LegacyXmlResultFormatter.java | 1 + src/main/org/apache/tools/ant/types/resources/TarResource.java | 1 - src/main/org/apache/tools/ant/util/JavaEnvUtils.java | 1 - src/main/org/apache/tools/bzip2/BlockSort.java | 1 + .../junit/org/apache/tools/ant/IntrospectionHelperTest.java | 2 +- .../junit/org/apache/tools/ant/taskdefs/XmlPropertyTest.java | 1 + src/tests/junit/org/apache/tools/ant/util/JavaEnvUtilsTest.java | 4 ++-- 21 files changed, 16 insertions(+), 13 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/DirectoryScanner.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/DirectoryScanner.java b/src/main/org/apache/tools/ant/DirectoryScanner.java index fd48d12..71465db 100644 --- a/src/main/org/apache/tools/ant/DirectoryScanner.java +++ b/src/main/org/apache/tools/ant/DirectoryScanner.java @@ -613,7 +613,6 @@ public class DirectoryScanner * * @since Ant 1.6 */ - @SuppressWarnings("deprecated") public static void resetDefaultExcludes() { synchronized (defaultExcludes) { defaultExcludes.clear(); http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/RuntimeConfigurable.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/RuntimeConfigurable.java b/src/main/org/apache/tools/ant/RuntimeConfigurable.java index 7afd82b..d996409 100644 --- a/src/main/org/apache/tools/ant/RuntimeConfigurable.java +++ b/src/main/org/apache/tools/ant/RuntimeConfigurable.java @@ -119,6 +119,7 @@ public class RuntimeConfigurable implements Serializable { } private static class EnableAttributeConsumer { + @SuppressWarnings("unused") public void add(EnableAttribute b) { // Ignore } @@ -176,7 +177,7 @@ public class RuntimeConfigurable implements Serializable { * false. * @since 1.9.1 */ - @SuppressWarnings("deprecated") + @SuppressWarnings("deprecation") public boolean isEnabled(UnknownElement owner) { if (!namespacedAttribute) { return true; http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/input/SecureInputHandler.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/input/SecureInputHandler.java b/src/main/org/apache/tools/ant/input/SecureInputHandler.java index 390a3cc..878a01d 100644 --- a/src/main/org/apache/tools/ant/input/SecureInputHandler.java +++ b/src/main/org/apache/tools/ant/input/SecureInputHandler.java @@ -38,6 +38,7 @@ public class SecureInputHandler extends DefaultInputHandler { * @param request the request to handle * @throws BuildException if not possible to read from console */ + @SuppressWarnings("unused") public void handleInput(InputRequest request) throws BuildException { String prompt = getPrompt(request); do { http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/launch/Locator.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/launch/Locator.java b/src/main/org/apache/tools/ant/launch/Locator.java index 0a644a8..9cd9626 100644 --- a/src/main/org/apache/tools/ant/launch/Locator.java +++ b/src/main/org/apache/tools/ant/launch/Locator.java @@ -470,7 +470,6 @@ public final class Locator { * @exception MalformedURLException if the URLs for the files cannot be * formed. */ - @SuppressWarnings("deprecated") public static URL[] getLocationURLs(File location, final String... extensions) throws MalformedURLException { http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/listener/Log4jListener.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/listener/Log4jListener.java b/src/main/org/apache/tools/ant/listener/Log4jListener.java index a116e3d..0000bc7 100644 --- a/src/main/org/apache/tools/ant/listener/Log4jListener.java +++ b/src/main/org/apache/tools/ant/listener/Log4jListener.java @@ -44,6 +44,7 @@ public class Log4jListener implements BuildListener { */ public Log4jListener() { // trigger the log4j initialization (if at all it's not yet initialized) + @SuppressWarnings("unused") final Logger log = Logger.getLogger(LOG_ANT); } http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/taskdefs/Concat.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/Concat.java b/src/main/org/apache/tools/ant/taskdefs/Concat.java index 8c52316..45b610d 100644 --- a/src/main/org/apache/tools/ant/taskdefs/Concat.java +++ b/src/main/org/apache/tools/ant/taskdefs/Concat.java @@ -601,7 +601,6 @@ public class Concat extends Task implements ResourceCollection { * input files. * @since Ant 1.8.2 */ - @SuppressWarnings("deprecated") public void setOverwrite(boolean forceOverwrite) { setForce(forceOverwrite); } http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/taskdefs/CopyPath.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/CopyPath.java b/src/main/org/apache/tools/ant/taskdefs/CopyPath.java index 9509423..54d2e32 100644 --- a/src/main/org/apache/tools/ant/taskdefs/CopyPath.java +++ b/src/main/org/apache/tools/ant/taskdefs/CopyPath.java @@ -63,6 +63,7 @@ public class CopyPath extends Task { private File destDir; // TODO not read, yet in a public setter + @SuppressWarnings("unused") private long granularity = FILE_UTILS.getFileTimestampGranularity(); private boolean preserveLastModified = false; http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/taskdefs/compilers/DefaultCompilerAdapter.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/compilers/DefaultCompilerAdapter.java b/src/main/org/apache/tools/ant/taskdefs/compilers/DefaultCompilerAdapter.java index f9b0d16..95eabbb 100644 --- a/src/main/org/apache/tools/ant/taskdefs/compilers/DefaultCompilerAdapter.java +++ b/src/main/org/apache/tools/ant/taskdefs/compilers/DefaultCompilerAdapter.java @@ -706,7 +706,6 @@ public abstract class DefaultCompilerAdapter * @return true if JDK 9 * @since Ant 1.9.8 */ - @SuppressWarnings("deprecated") protected boolean assumeJava9() { return assumeJava19(); } http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/taskdefs/email/Message.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/email/Message.java b/src/main/org/apache/tools/ant/taskdefs/email/Message.java index ec10e99..fc07396 100644 --- a/src/main/org/apache/tools/ant/taskdefs/email/Message.java +++ b/src/main/org/apache/tools/ant/taskdefs/email/Message.java @@ -109,6 +109,7 @@ public class Message extends ProjectComponent { * @param ps The print stream to write to * @throws IOException if an error occurs */ + @SuppressWarnings("resource") public void print(PrintStream ps) throws IOException { // We need character encoding aware printing here. // So, using BufferedWriter over OutputStreamWriter instead of PrintStream http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/taskdefs/optional/XMLValidateTask.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/XMLValidateTask.java b/src/main/org/apache/tools/ant/taskdefs/optional/XMLValidateTask.java index 9e3d808..c7000b3 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/XMLValidateTask.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/XMLValidateTask.java @@ -375,7 +375,6 @@ public class XMLValidateTask extends Task { * we have created and wrapped a SAX1 parser. * @return the new XMLReader. */ - @SuppressWarnings("deprecated") protected XMLReader createXmlReader() { Object reader = null; if (readerClassName == null) { http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/taskdefs/optional/jdepend/JDependTask.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/jdepend/JDependTask.java b/src/main/org/apache/tools/ant/taskdefs/optional/jdepend/JDependTask.java index 17fd3af..09e13f2 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/jdepend/JDependTask.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/jdepend/JDependTask.java @@ -389,7 +389,6 @@ public class JDependTask extends Task { * @exception BuildException if an error occurs */ @Override - @SuppressWarnings("deprecated") public void execute() throws BuildException { CommandlineJava commandline = new CommandlineJava(); @@ -628,7 +627,6 @@ public class JDependTask extends Task { return new ExecuteWatchdog(getTimeout()); } - @SuppressWarnings("deprecated") private Optional<Path> getWorkingPath() { Optional<Path> result = Optional.ofNullable(getClassespath()); if (result.isPresent()) { http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/taskdefs/optional/junit/JUnitTestRunner.java ---------------------------------------------------------------------- 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 393211b..7c0b0cd 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 @@ -1265,6 +1265,7 @@ public class JUnitTestRunner implements TestListener, JUnitTaskMirror.JUnitTestR public void addFailure(final Test test, final AssertionFailedError t) { testListener.addFailure(test, t); } + @SuppressWarnings("unused") public void addFailure(final Test test, final Throwable t) { // pre-3.4 if (t instanceof AssertionFailedError) { testListener.addFailure(test, (AssertionFailedError) t); http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/JUnitLauncherTask.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/JUnitLauncherTask.java b/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/JUnitLauncherTask.java index c505ee0..dc09a5e 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/JUnitLauncherTask.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/JUnitLauncherTask.java @@ -295,6 +295,7 @@ public class JUnitLauncherTask extends Task { return new AntClassLoader(this.getClass().getClassLoader(), getProject(), this.classPath, true); } + @SuppressWarnings("resource") private Optional<SwitchedStreamHandle> trySwitchSysOutErr(final TestRequest testRequest, final StreamType streamType) { switch (streamType) { case SYS_OUT: { http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LegacyPlainResultFormatter.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LegacyPlainResultFormatter.java b/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LegacyPlainResultFormatter.java index 15ae463..5b96b26 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LegacyPlainResultFormatter.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LegacyPlainResultFormatter.java @@ -174,6 +174,7 @@ class LegacyPlainResultFormatter extends AbstractJUnitResultFormatter implements } } + @SuppressWarnings("incomplete-switch") @Override public void executionFinished(final TestIdentifier testIdentifier, final TestExecutionResult testExecutionResult) { final long currentTime = System.currentTimeMillis(); @@ -272,6 +273,7 @@ class LegacyPlainResultFormatter extends AbstractJUnitResultFormatter implements } private final class Stats { + @SuppressWarnings("unused") private final TestIdentifier testIdentifier; private final AtomicLong numTestsRun = new AtomicLong(0); private final AtomicLong numTestsFailed = new AtomicLong(0); http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LegacyXmlResultFormatter.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LegacyXmlResultFormatter.java b/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LegacyXmlResultFormatter.java index 6ca2e51..e538fc9 100644 --- a/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LegacyXmlResultFormatter.java +++ b/src/main/org/apache/tools/ant/taskdefs/optional/junitlauncher/LegacyXmlResultFormatter.java @@ -124,6 +124,7 @@ class LegacyXmlResultFormatter extends AbstractJUnitResultFormatter implements T } private final class Stats { + @SuppressWarnings("unused") private final TestIdentifier testIdentifier; private final long startedAt; private long endedAt; http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/types/resources/TarResource.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/types/resources/TarResource.java b/src/main/org/apache/tools/ant/types/resources/TarResource.java index 1279bd8..9bed83c 100644 --- a/src/main/org/apache/tools/ant/types/resources/TarResource.java +++ b/src/main/org/apache/tools/ant/types/resources/TarResource.java @@ -198,7 +198,6 @@ public class TarResource extends ArchiveResource { return (TarResource) super.getCheckedRef(); } - @SuppressWarnings("deprecated") private void setEntry(TarEntry e) { if (e == null) { setExists(false); http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/ant/util/JavaEnvUtils.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/ant/util/JavaEnvUtils.java b/src/main/org/apache/tools/ant/util/JavaEnvUtils.java index fd00780..b8cb9e1 100644 --- a/src/main/org/apache/tools/ant/util/JavaEnvUtils.java +++ b/src/main/org/apache/tools/ant/util/JavaEnvUtils.java @@ -279,7 +279,6 @@ public final class JavaEnvUtils { * @return true if the version of Java is the same as the given version. * @since Ant 1.5 */ - @SuppressWarnings("deprecated") public static boolean isJavaVersion(String version) { return javaVersion.equals(version) || (javaVersion.equals(JAVA_9) && JAVA_1_9.equals(version)); http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/main/org/apache/tools/bzip2/BlockSort.java ---------------------------------------------------------------------- diff --git a/src/main/org/apache/tools/bzip2/BlockSort.java b/src/main/org/apache/tools/bzip2/BlockSort.java index f019da8..75c3ff2 100644 --- a/src/main/org/apache/tools/bzip2/BlockSort.java +++ b/src/main/org/apache/tools/bzip2/BlockSort.java @@ -117,6 +117,7 @@ class BlockSort { private static final int FALLBACK_QSORT_STACK_SIZE = 100; + @SuppressWarnings("unused") private static final int STACK_SIZE = QSORT_STACK_SIZE < FALLBACK_QSORT_STACK_SIZE ? FALLBACK_QSORT_STACK_SIZE : QSORT_STACK_SIZE; http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/tests/junit/org/apache/tools/ant/IntrospectionHelperTest.java ---------------------------------------------------------------------- diff --git a/src/tests/junit/org/apache/tools/ant/IntrospectionHelperTest.java b/src/tests/junit/org/apache/tools/ant/IntrospectionHelperTest.java index e512a3b..4318494 100644 --- a/src/tests/junit/org/apache/tools/ant/IntrospectionHelperTest.java +++ b/src/tests/junit/org/apache/tools/ant/IntrospectionHelperTest.java @@ -800,7 +800,7 @@ public class IntrospectionHelperTest { } } - public List<Object> add(List l) { + public List<Object> add(List<Object> l) { // INVALID extension point return null; } http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/tests/junit/org/apache/tools/ant/taskdefs/XmlPropertyTest.java ---------------------------------------------------------------------- diff --git a/src/tests/junit/org/apache/tools/ant/taskdefs/XmlPropertyTest.java b/src/tests/junit/org/apache/tools/ant/taskdefs/XmlPropertyTest.java index 25e83ac..9d03c5a 100644 --- a/src/tests/junit/org/apache/tools/ant/taskdefs/XmlPropertyTest.java +++ b/src/tests/junit/org/apache/tools/ant/taskdefs/XmlPropertyTest.java @@ -258,6 +258,7 @@ public class XmlPropertyTest { /** * Debugging method to print the properties in the given hashtable */ + @SuppressWarnings("unused") private static void printProperties(Hashtable<Object, Object> xmlproperties) { xmlproperties.forEach((key, value) -> System.out.println(key + " = " + value)); } http://git-wip-us.apache.org/repos/asf/ant/blob/342bf005/src/tests/junit/org/apache/tools/ant/util/JavaEnvUtilsTest.java ---------------------------------------------------------------------- diff --git a/src/tests/junit/org/apache/tools/ant/util/JavaEnvUtilsTest.java b/src/tests/junit/org/apache/tools/ant/util/JavaEnvUtilsTest.java index 2717746..f96e036 100644 --- a/src/tests/junit/org/apache/tools/ant/util/JavaEnvUtilsTest.java +++ b/src/tests/junit/org/apache/tools/ant/util/JavaEnvUtilsTest.java @@ -127,7 +127,7 @@ public class JavaEnvUtilsTest { } @Test - @SuppressWarnings("deprecated") + @SuppressWarnings("deprecation") public void isJavaVersionSupportsBothVersionsOfJava9() { assumeTrue(JavaEnvUtils.isJavaVersion(JavaEnvUtils.JAVA_9)); assertTrue("JAVA_1_9 is not considered equal to JAVA_9", @@ -135,7 +135,7 @@ public class JavaEnvUtilsTest { } @Test - @SuppressWarnings("deprecated") + @SuppressWarnings("deprecation") public void java10IsDetectedProperly() { assumeTrue("10".equals(System.getProperty("java.specification.version"))); assertEquals("10", JavaEnvUtils.getJavaVersion());
