You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ant.apache.org by gi...@apache.org on 2018/08/20 05:16:06 UTC

ant git commit: Use StandardCharsets

Repository: ant
Updated Branches:
  refs/heads/master 80a0e76ff -> 16d0f4d81


Use StandardCharsets

Project: http://git-wip-us.apache.org/repos/asf/ant/repo
Commit: http://git-wip-us.apache.org/repos/asf/ant/commit/16d0f4d8
Tree: http://git-wip-us.apache.org/repos/asf/ant/tree/16d0f4d8
Diff: http://git-wip-us.apache.org/repos/asf/ant/diff/16d0f4d8

Branch: refs/heads/master
Commit: 16d0f4d811220bf60b27d6402a7732bbaec81186
Parents: 80a0e76
Author: Gintas Grigelionis <gi...@apache.org>
Authored: Mon Aug 20 07:17:03 2018 +0200
Committer: Gintas Grigelionis <gi...@apache.org>
Committed: Mon Aug 20 07:17:03 2018 +0200

----------------------------------------------------------------------
 .../org/apache/tools/ant/ArgumentProcessorRegistry.java   |  8 ++------
 .../org/apache/tools/ant/ProjectHelperRepository.java     |  7 ++-----
 src/main/org/apache/tools/ant/XmlLogger.java              |  3 ++-
 src/main/org/apache/tools/ant/filters/ClassConstants.java |  4 ++--
 src/main/org/apache/tools/ant/taskdefs/AntStructure.java  |  3 ++-
 src/main/org/apache/tools/ant/taskdefs/Jar.java           |  7 ++++---
 src/main/org/apache/tools/ant/taskdefs/Javac.java         |  3 ++-
 src/main/org/apache/tools/ant/taskdefs/Javadoc.java       |  3 ++-
 .../org/apache/tools/ant/taskdefs/LoadProperties.java     |  4 ++--
 src/main/org/apache/tools/ant/taskdefs/Manifest.java      |  3 ++-
 .../apache/tools/ant/taskdefs/cvslib/ChangeLogTask.java   |  3 ++-
 .../org/apache/tools/ant/taskdefs/cvslib/CvsTagDiff.java  |  3 ++-
 .../tools/ant/taskdefs/optional/EchoProperties.java       |  3 ++-
 .../taskdefs/optional/junit/XMLJUnitResultFormatter.java  |  3 ++-
 .../ant/taskdefs/optional/junit/XMLResultAggregator.java  |  7 ++++---
 .../junitlauncher/LegacyPlainResultFormatter.java         |  8 ++------
 .../tools/ant/types/resources/JavaConstantResource.java   |  3 ++-
 src/main/org/apache/tools/ant/types/spi/Service.java      |  3 ++-
 src/main/org/apache/tools/ant/util/DOMElementWriter.java  |  3 ++-
 .../apache/tools/ant/util/LayoutPreservingProperties.java |  7 ++++---
 src/main/org/apache/tools/ant/util/ResourceUtils.java     |  2 ++
 src/main/org/apache/tools/tar/TarEntry.java               | 10 ++--------
 src/main/org/apache/tools/tar/TarInputStream.java         |  3 ++-
 src/main/org/apache/tools/tar/TarOutputStream.java        |  7 ++++---
 .../org/apache/tools/zip/AbstractUnicodeExtraField.java   |  9 ++-------
 .../junit/org/apache/tools/ant/taskdefs/EchoTest.java     |  3 ++-
 .../junit/org/apache/tools/ant/taskdefs/JarTest.java      |  3 ++-
 .../tools/ant/taskdefs/cvslib/ChangeLogWriterTest.java    |  3 ++-
 .../junit/org/apache/tools/ant/types/PatternSetTest.java  |  3 ++-
 .../org/apache/tools/ant/util/ReaderInputStreamTest.java  |  7 ++++---
 .../junit/org/apache/tools/zip/UTF8ZipFilesTest.java      | 10 +++++-----
 src/tests/junit/org/apache/tools/zip/ZipEncodingTest.java |  5 +++--
 32 files changed, 78 insertions(+), 75 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/ArgumentProcessorRegistry.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/ArgumentProcessorRegistry.java b/src/main/org/apache/tools/ant/ArgumentProcessorRegistry.java
index 9119191..c2e0bbc 100644
--- a/src/main/org/apache/tools/ant/ArgumentProcessorRegistry.java
+++ b/src/main/org/apache/tools/ant/ArgumentProcessorRegistry.java
@@ -21,9 +21,9 @@ import java.io.BufferedReader;
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.InputStreamReader;
-import java.io.UnsupportedEncodingException;
 import java.net.URL;
 import java.net.URLConnection;
+import java.nio.charset.StandardCharsets;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
@@ -148,11 +148,7 @@ public class ArgumentProcessorRegistry {
             throws IOException {
         InputStreamReader isr = null;
         try {
-            try {
-                isr = new InputStreamReader(is, "UTF-8");
-            } catch (UnsupportedEncodingException e) {
-                isr = new InputStreamReader(is);
-            }
+            isr = new InputStreamReader(is, StandardCharsets.UTF_8);
             BufferedReader rd = new BufferedReader(isr);
             String processorClassName = rd.readLine();
             if (processorClassName != null && !processorClassName.isEmpty()) {

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/ProjectHelperRepository.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/ProjectHelperRepository.java b/src/main/org/apache/tools/ant/ProjectHelperRepository.java
index b1c48be..532374b 100644
--- a/src/main/org/apache/tools/ant/ProjectHelperRepository.java
+++ b/src/main/org/apache/tools/ant/ProjectHelperRepository.java
@@ -23,6 +23,7 @@ import java.io.InputStreamReader;
 import java.lang.reflect.Constructor;
 import java.net.URL;
 import java.net.URLConnection;
+import java.nio.charset.StandardCharsets;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.Iterator;
@@ -181,11 +182,7 @@ public class ProjectHelperRepository {
             // This code is needed by EBCDIC and other strange systems.
             // It's a fix for bugs reported in xerces
             InputStreamReader isr;
-            try {
-                isr = new InputStreamReader(is, "UTF-8");
-            } catch (java.io.UnsupportedEncodingException e) {
-                isr = new InputStreamReader(is);
-            }
+            isr = new InputStreamReader(is, StandardCharsets.UTF_8);
             BufferedReader rd = new BufferedReader(isr);
 
             String helperClassName = rd.readLine();

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/XmlLogger.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/XmlLogger.java b/src/main/org/apache/tools/ant/XmlLogger.java
index 5adce14..84a1ba9 100644
--- a/src/main/org/apache/tools/ant/XmlLogger.java
+++ b/src/main/org/apache/tools/ant/XmlLogger.java
@@ -22,6 +22,7 @@ import java.io.OutputStream;
 import java.io.OutputStreamWriter;
 import java.io.PrintStream;
 import java.io.Writer;
+import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.nio.file.Paths;
 import java.util.Hashtable;
@@ -180,7 +181,7 @@ public class XmlLogger implements BuildLogger {
 
         try (OutputStream stream =
             outStream == null ? Files.newOutputStream(Paths.get(outFilename)) : outStream;
-                Writer out = new OutputStreamWriter(stream, "UTF8")) {
+                Writer out = new OutputStreamWriter(stream, StandardCharsets.UTF_8)) {
             out.write("<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n");
             if (!xslUri.isEmpty()) {
                 out.write("<?xml-stylesheet type=\"text/xsl\" href=\"" + xslUri

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/filters/ClassConstants.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/filters/ClassConstants.java b/src/main/org/apache/tools/ant/filters/ClassConstants.java
index 70dd02a..58af620 100644
--- a/src/main/org/apache/tools/ant/filters/ClassConstants.java
+++ b/src/main/org/apache/tools/ant/filters/ClassConstants.java
@@ -21,9 +21,9 @@ import java.io.IOException;
 import java.io.Reader;
 import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
+import java.nio.charset.StandardCharsets;
 
 import org.apache.tools.ant.BuildException;
-import org.apache.tools.ant.util.ResourceUtils;
 
 /**
  * Assembles the constants declared in a Java class in
@@ -97,7 +97,7 @@ public final class ClassConstants
             if (clazz == null || clazz.isEmpty()) {
                 ch = -1;
             } else {
-                final byte[] bytes = clazz.getBytes(ResourceUtils.ISO_8859_1);
+                final byte[] bytes = clazz.getBytes(StandardCharsets.ISO_8859_1);
                 try {
                     final Class<?> javaClassHelper = Class.forName(JAVA_CLASS_HELPER);
                     if (javaClassHelper != null) {

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/taskdefs/AntStructure.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/AntStructure.java b/src/main/org/apache/tools/ant/taskdefs/AntStructure.java
index 1770774..11529a6 100644
--- a/src/main/org/apache/tools/ant/taskdefs/AntStructure.java
+++ b/src/main/org/apache/tools/ant/taskdefs/AntStructure.java
@@ -25,6 +25,7 @@ import java.io.OutputStream;
 import java.io.OutputStreamWriter;
 import java.io.PrintWriter;
 import java.io.UnsupportedEncodingException;
+import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.util.ArrayList;
 import java.util.Collections;
@@ -91,7 +92,7 @@ public class AntStructure extends Task {
             OutputStream fos = null;
             try {
                 fos = Files.newOutputStream(output.toPath());
-                out = new PrintWriter(new OutputStreamWriter(fos, "UTF8"));
+                out = new PrintWriter(new OutputStreamWriter(fos, StandardCharsets.UTF_8));
             } catch (final UnsupportedEncodingException ue) {
                 FileUtils.close(fos);
                 /*

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/taskdefs/Jar.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/Jar.java b/src/main/org/apache/tools/ant/taskdefs/Jar.java
index f83532d..53e333b 100644
--- a/src/main/org/apache/tools/ant/taskdefs/Jar.java
+++ b/src/main/org/apache/tools/ant/taskdefs/Jar.java
@@ -29,6 +29,7 @@ import java.io.PrintWriter;
 import java.io.Reader;
 import java.io.UnsupportedEncodingException;
 import java.nio.charset.Charset;
+import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.util.ArrayList;
 import java.util.Collections;
@@ -330,7 +331,8 @@ public class Jar extends Zip {
             if (ze == null) {
                 return null;
             }
-            try (InputStreamReader isr = new InputStreamReader(zf.getInputStream(ze), "UTF-8")) {
+            try (InputStreamReader isr = new InputStreamReader(zf.getInputStream(ze),
+                    StandardCharsets.UTF_8)) {
                 return getManifest(isr);
             }
         }
@@ -581,8 +583,7 @@ public class Jar extends Zip {
     private void createIndexList(ZipOutputStream zOut) throws IOException {
         ByteArrayOutputStream baos = new ByteArrayOutputStream();
         // encoding must be UTF8 as specified in the specs.
-        PrintWriter writer = new PrintWriter(new OutputStreamWriter(baos,
-                                                                    "UTF8"));
+        PrintWriter writer = new PrintWriter(new OutputStreamWriter(baos, StandardCharsets.UTF_8));
 
         // version-info blankline
         writer.println("JarIndex-Version: 1.0");

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/taskdefs/Javac.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/Javac.java b/src/main/org/apache/tools/ant/taskdefs/Javac.java
index 1cf9964..7184d81 100644
--- a/src/main/org/apache/tools/ant/taskdefs/Javac.java
+++ b/src/main/org/apache/tools/ant/taskdefs/Javac.java
@@ -21,6 +21,7 @@ package org.apache.tools.ant.taskdefs;
 import java.io.File;
 import java.io.IOException;
 import java.io.OutputStream;
+import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.util.ArrayList;
 import java.util.Collection;
@@ -1480,7 +1481,7 @@ public class Javac extends MatchingTask {
             log("Creating empty " + pkgInfoClass);
             try (OutputStream os = Files.newOutputStream(pkgInfoClass.toPath())) {
                 os.write(PACKAGE_INFO_CLASS_HEADER);
-                final byte[] name = pkg.getBytes("UTF-8");
+                final byte[] name = pkg.getBytes(StandardCharsets.UTF_8);
                 final int length = name.length + /* "/package-info" */ 13;
                 os.write((byte) length / 256);
                 os.write((byte) length % 256);

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/taskdefs/Javadoc.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/Javadoc.java b/src/main/org/apache/tools/ant/taskdefs/Javadoc.java
index 1ca9dad..dc08876 100644
--- a/src/main/org/apache/tools/ant/taskdefs/Javadoc.java
+++ b/src/main/org/apache/tools/ant/taskdefs/Javadoc.java
@@ -29,6 +29,7 @@ import java.io.InputStreamReader;
 import java.io.OutputStreamWriter;
 import java.net.MalformedURLException;
 import java.net.URL;
+import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.util.ArrayList;
 import java.util.Collections;
@@ -2411,7 +2412,7 @@ public class Javadoc extends Task {
         try {
             fixData =
                 fixLineFeeds(FileUtils
-                             .readFully(new InputStreamReader(in, "US-ASCII")))
+                             .readFully(new InputStreamReader(in, StandardCharsets.US_ASCII)))
                 .trim();
         } finally {
             FileUtils.close(in);

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/taskdefs/LoadProperties.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/LoadProperties.java b/src/main/org/apache/tools/ant/taskdefs/LoadProperties.java
index 52e1d5e..582b241 100644
--- a/src/main/org/apache/tools/ant/taskdefs/LoadProperties.java
+++ b/src/main/org/apache/tools/ant/taskdefs/LoadProperties.java
@@ -23,6 +23,7 @@ import java.io.File;
 import java.io.IOException;
 import java.io.InputStreamReader;
 import java.nio.charset.Charset;
+import java.nio.charset.StandardCharsets;
 import java.util.List;
 import java.util.Properties;
 import java.util.Vector;
@@ -39,7 +40,6 @@ import org.apache.tools.ant.types.Resource;
 import org.apache.tools.ant.types.ResourceCollection;
 import org.apache.tools.ant.types.resources.FileResource;
 import org.apache.tools.ant.types.resources.JavaResource;
-import org.apache.tools.ant.util.ResourceUtils;
 
 /**
  * Load a file's contents as Ant properties.
@@ -188,7 +188,7 @@ public class LoadProperties extends Task {
                     text += "\n";
                 }
                 ByteArrayInputStream tis = new ByteArrayInputStream(
-                    text.getBytes(ResourceUtils.ISO_8859_1));
+                    text.getBytes(StandardCharsets.ISO_8859_1));
                 final Properties props = new Properties();
                 props.load(tis);
 

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/taskdefs/Manifest.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/Manifest.java b/src/main/org/apache/tools/ant/taskdefs/Manifest.java
index ef6dc88..141bcdd 100644
--- a/src/main/org/apache/tools/ant/taskdefs/Manifest.java
+++ b/src/main/org/apache/tools/ant/taskdefs/Manifest.java
@@ -26,6 +26,7 @@ import java.io.PrintWriter;
 import java.io.Reader;
 import java.io.StringWriter;
 import java.io.UnsupportedEncodingException;
+import java.nio.charset.StandardCharsets;
 import java.util.Collections;
 import java.util.Enumeration;
 import java.util.LinkedHashMap;
@@ -759,7 +760,7 @@ public class Manifest {
                     defManifest);
             }
             try {
-                insr = new InputStreamReader(in, "UTF-8");
+                insr = new InputStreamReader(in, StandardCharsets.UTF_8);
                 Manifest defaultManifest = new Manifest(insr);
                 String version = System.getProperty("java.runtime.version");
                 if (version == null) {

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/taskdefs/cvslib/ChangeLogTask.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/cvslib/ChangeLogTask.java b/src/main/org/apache/tools/ant/taskdefs/cvslib/ChangeLogTask.java
index 8182749..292c8c1 100644
--- a/src/main/org/apache/tools/ant/taskdefs/cvslib/ChangeLogTask.java
+++ b/src/main/org/apache/tools/ant/taskdefs/cvslib/ChangeLogTask.java
@@ -22,6 +22,7 @@ import java.io.IOException;
 import java.io.OutputStreamWriter;
 import java.io.PrintWriter;
 import java.io.UnsupportedEncodingException;
+import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.text.SimpleDateFormat;
 import java.util.ArrayList;
@@ -421,7 +422,7 @@ public class ChangeLogTask extends AbstractCvsTask {
          throws BuildException {
 
         try (final PrintWriter writer = new PrintWriter(
-            new OutputStreamWriter(Files.newOutputStream(destFile.toPath()), "UTF-8"))) {
+            new OutputStreamWriter(Files.newOutputStream(destFile.toPath()), StandardCharsets.UTF_8))) {
 
             new ChangeLogWriter().printChangeLog(writer, entrySet);
 

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/taskdefs/cvslib/CvsTagDiff.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/cvslib/CvsTagDiff.java b/src/main/org/apache/tools/ant/taskdefs/cvslib/CvsTagDiff.java
index 3dc3a6f..12cd81c 100644
--- a/src/main/org/apache/tools/ant/taskdefs/cvslib/CvsTagDiff.java
+++ b/src/main/org/apache/tools/ant/taskdefs/cvslib/CvsTagDiff.java
@@ -24,6 +24,7 @@ import java.io.IOException;
 import java.io.OutputStreamWriter;
 import java.io.PrintWriter;
 import java.io.UnsupportedEncodingException;
+import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.util.ArrayList;
 import java.util.List;
@@ -404,7 +405,7 @@ public class CvsTagDiff extends AbstractCvsTask {
      */
     private void writeTagDiff(CvsTagEntry[] entries) throws BuildException {
         try (PrintWriter writer = new PrintWriter(new OutputStreamWriter(
-            Files.newOutputStream(mydestfile.toPath()), "UTF-8"))) {
+            Files.newOutputStream(mydestfile.toPath()), StandardCharsets.UTF_8))) {
             writer.println("<?xml version=\"1.0\" encoding=\"UTF-8\"?>");
             Document doc = DOMUtils.newDocument();
             Element root = doc.createElement("tagdiff");

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/taskdefs/optional/EchoProperties.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/EchoProperties.java b/src/main/org/apache/tools/ant/taskdefs/optional/EchoProperties.java
index 007597d..256e4db 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/EchoProperties.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/EchoProperties.java
@@ -24,6 +24,7 @@ import java.io.InputStream;
 import java.io.OutputStream;
 import java.io.OutputStreamWriter;
 import java.io.Writer;
+import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.util.ArrayList;
 import java.util.Collections;
@@ -452,7 +453,7 @@ public class EchoProperties extends Task {
             rootElement.appendChild(propElement);
         }
 
-        try (Writer wri = new OutputStreamWriter(os, "UTF8")) {
+        try (Writer wri = new OutputStreamWriter(os, StandardCharsets.UTF_8)) {
             wri.write("<?xml version=\"1.0\" encoding=\"UTF-8\"?>");
             new DOMElementWriter().write(rootElement, wri, 0, "\t");
             wri.flush();

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLJUnitResultFormatter.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLJUnitResultFormatter.java b/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLJUnitResultFormatter.java
index da56d22..544233b 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLJUnitResultFormatter.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLJUnitResultFormatter.java
@@ -25,6 +25,7 @@ import java.io.OutputStreamWriter;
 import java.io.Writer;
 import java.net.InetAddress;
 import java.net.UnknownHostException;
+import java.nio.charset.StandardCharsets;
 import java.util.Date;
 import java.util.Hashtable;
 import java.util.Map;
@@ -190,7 +191,7 @@ public class XMLJUnitResultFormatter implements JUnitResultFormatter, XMLConstan
         if (out != null) {
             Writer wri = null;
             try {
-                wri = new BufferedWriter(new OutputStreamWriter(out, "UTF8"));
+                wri = new BufferedWriter(new OutputStreamWriter(out, StandardCharsets.UTF_8));
                 wri.write("<?xml version=\"1.0\" encoding=\"UTF-8\" ?>\n");
                 new DOMElementWriter().write(rootElement, wri, 0, "  ");
             } catch (final IOException exc) {

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLResultAggregator.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLResultAggregator.java b/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLResultAggregator.java
index df54e92..c188ad9 100644
--- a/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLResultAggregator.java
+++ b/src/main/org/apache/tools/ant/taskdefs/optional/junit/XMLResultAggregator.java
@@ -20,9 +20,9 @@ package org.apache.tools.ant.taskdefs.optional.junit;
 import java.io.BufferedOutputStream;
 import java.io.File;
 import java.io.IOException;
-import java.io.OutputStream;
 import java.io.OutputStreamWriter;
 import java.io.PrintWriter;
+import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.util.Vector;
 import java.util.stream.Stream;
@@ -198,8 +198,9 @@ public class XMLResultAggregator extends Task implements XMLConstants {
      * @throws IOException thrown if there is an error while writing the content.
      */
     protected void writeDOMTree(Document doc, File file) throws IOException {
-        try (OutputStream os = Files.newOutputStream(file.toPath());
-             PrintWriter wri = new PrintWriter(new OutputStreamWriter(new BufferedOutputStream(os), "UTF8"))) {
+        try (PrintWriter wri = new PrintWriter(new OutputStreamWriter(
+                new BufferedOutputStream(Files.newOutputStream(file.toPath())),
+                StandardCharsets.UTF_8))) {
             wri.write("<?xml version=\"1.0\" encoding=\"UTF-8\" ?>\n");
             new DOMElementWriter().write(doc.getDocumentElement(), wri, 0,
                 "  ");

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/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 5b96b26..56b28ec 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
@@ -30,7 +30,7 @@ import java.io.OutputStream;
 import java.io.OutputStreamWriter;
 import java.io.PrintWriter;
 import java.io.StringWriter;
-import java.io.UnsupportedEncodingException;
+import java.nio.charset.StandardCharsets;
 import java.util.Map;
 import java.util.Optional;
 import java.util.concurrent.ConcurrentHashMap;
@@ -242,11 +242,7 @@ class LegacyPlainResultFormatter extends AbstractJUnitResultFormatter implements
     @Override
     public void setDestination(final OutputStream os) {
         this.outputStream = os;
-        try {
-            this.writer = new BufferedWriter(new OutputStreamWriter(this.outputStream, "UTF-8"));
-        } catch (UnsupportedEncodingException e) {
-            throw new RuntimeException("Failed to create a writer", e);
-        }
+        this.writer = new BufferedWriter(new OutputStreamWriter(this.outputStream, StandardCharsets.UTF_8));
     }
 
     protected boolean shouldReportExecutionFinished(final TestIdentifier testIdentifier, final TestExecutionResult testExecutionResult) {

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/types/resources/JavaConstantResource.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/types/resources/JavaConstantResource.java b/src/main/org/apache/tools/ant/types/resources/JavaConstantResource.java
index 04554c6b8f..a223c60 100644
--- a/src/main/org/apache/tools/ant/types/resources/JavaConstantResource.java
+++ b/src/main/org/apache/tools/ant/types/resources/JavaConstantResource.java
@@ -21,6 +21,7 @@ import java.io.ByteArrayInputStream;
 import java.io.IOException;
 import java.io.InputStream;
 import java.lang.reflect.Field;
+import java.nio.charset.StandardCharsets;
 
 /**
  * A resource that is a java constant.
@@ -55,7 +56,7 @@ public class JavaConstantResource extends AbstractClasspathResource {
                 : Class.forName(classname);
             Field field = clazz.getField(fieldname);
             String value = field.get(null).toString();
-            return new ByteArrayInputStream(value.getBytes("UTF-8"));
+            return new ByteArrayInputStream(value.getBytes(StandardCharsets.UTF_8));
         } catch (ClassNotFoundException e) {
             throw new IOException("Class not found:" + classname);
         } catch (NoSuchFieldException e) {

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/types/spi/Service.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/types/spi/Service.java b/src/main/org/apache/tools/ant/types/spi/Service.java
index 7d7c8a2..4448a58 100644
--- a/src/main/org/apache/tools/ant/types/spi/Service.java
+++ b/src/main/org/apache/tools/ant/types/spi/Service.java
@@ -20,6 +20,7 @@ package org.apache.tools.ant.types.spi;
 import java.io.ByteArrayInputStream;
 import java.io.IOException;
 import java.io.InputStream;
+import java.nio.charset.StandardCharsets;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.stream.Collectors;
@@ -83,7 +84,7 @@ public class Service extends ProjectComponent {
     public InputStream getAsStream() throws IOException {
         return new ByteArrayInputStream(
             providerList.stream().map(Provider::getClassName)
-                .collect(Collectors.joining("\n")).getBytes("UTF-8"));
+                .collect(Collectors.joining("\n")).getBytes(StandardCharsets.UTF_8));
     }
 
     /**

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/util/DOMElementWriter.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/util/DOMElementWriter.java b/src/main/org/apache/tools/ant/util/DOMElementWriter.java
index 2827467..ebef59b 100644
--- a/src/main/org/apache/tools/ant/util/DOMElementWriter.java
+++ b/src/main/org/apache/tools/ant/util/DOMElementWriter.java
@@ -23,6 +23,7 @@ import java.io.OutputStream;
 import java.io.OutputStreamWriter;
 import java.io.StringWriter;
 import java.io.Writer;
+import java.nio.charset.StandardCharsets;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
@@ -170,7 +171,7 @@ public class DOMElementWriter {
      * @throws IOException if an error happens while writing to the stream.
      */
     public void write(Element root, OutputStream out) throws IOException {
-        Writer wri = new OutputStreamWriter(out, "UTF8");
+        Writer wri = new OutputStreamWriter(out, StandardCharsets.UTF_8);
         writeXMLDeclaration(wri);
         write(root, wri, 0, "  ");
         wri.flush();

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/util/LayoutPreservingProperties.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/util/LayoutPreservingProperties.java b/src/main/org/apache/tools/ant/util/LayoutPreservingProperties.java
index 2f2397a..27a2205 100644
--- a/src/main/org/apache/tools/ant/util/LayoutPreservingProperties.java
+++ b/src/main/org/apache/tools/ant/util/LayoutPreservingProperties.java
@@ -28,6 +28,7 @@ import java.io.OutputStreamWriter;
 import java.io.PrintStream;
 import java.io.PushbackReader;
 import java.io.Serializable;
+import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.util.ArrayList;
 import java.util.HashMap;
@@ -145,7 +146,7 @@ public class LayoutPreservingProperties extends Properties {
     @Override
     public void load(final InputStream inStream) throws IOException {
         final String s = readLines(inStream);
-        final byte[] ba = s.getBytes(ResourceUtils.ISO_8859_1);
+        final byte[] ba = s.getBytes(StandardCharsets.ISO_8859_1);
         final ByteArrayInputStream bais = new ByteArrayInputStream(ba);
         super.load(bais);
     }
@@ -260,7 +261,7 @@ public class LayoutPreservingProperties extends Properties {
 
     @Override
     public void store(final OutputStream out, final String header) throws IOException {
-        final OutputStreamWriter osw = new OutputStreamWriter(out, ResourceUtils.ISO_8859_1);
+        final OutputStreamWriter osw = new OutputStreamWriter(out, StandardCharsets.ISO_8859_1);
 
         int skipLines = 0;
         final int totalLines = logicalLines.size();
@@ -315,7 +316,7 @@ public class LayoutPreservingProperties extends Properties {
      * @param is the stream from which to read the data
      */
     private String readLines(final InputStream is) throws IOException {
-        final InputStreamReader isr = new InputStreamReader(is, ResourceUtils.ISO_8859_1);
+        final InputStreamReader isr = new InputStreamReader(is, StandardCharsets.ISO_8859_1);
         final PushbackReader pbr = new PushbackReader(isr, 1);
 
         if (!logicalLines.isEmpty()) {

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/ant/util/ResourceUtils.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/ant/util/ResourceUtils.java b/src/main/org/apache/tools/ant/util/ResourceUtils.java
index f1114e7..5fd856d 100644
--- a/src/main/org/apache/tools/ant/util/ResourceUtils.java
+++ b/src/main/org/apache/tools/ant/util/ResourceUtils.java
@@ -69,8 +69,10 @@ public class ResourceUtils {
     /**
      * Name of charset "ISO Latin Alphabet No. 1, a.k.a. ISO-LATIN-1".
      *
+     * @deprecated use StandardCharsets.ISO_8859_1
      * @since Ant 1.8.1
      */
+    @Deprecated
     public static final String ISO_8859_1 = "ISO-8859-1";
 
     private static final long MAX_IO_CHUNK_SIZE = 16 * 1024 * 1024L; // 16 MB

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/tar/TarEntry.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/tar/TarEntry.java b/src/main/org/apache/tools/tar/TarEntry.java
index f706a64..77b43e4 100644
--- a/src/main/org/apache/tools/tar/TarEntry.java
+++ b/src/main/org/apache/tools/tar/TarEntry.java
@@ -25,7 +25,7 @@ package org.apache.tools.tar;
 
 import java.io.File;
 import java.io.IOException;
-import java.io.UnsupportedEncodingException;
+import java.nio.charset.StandardCharsets;
 import java.util.Date;
 import java.util.Locale;
 
@@ -1072,13 +1072,7 @@ public class TarEntry implements TarConstants {
      */
     private static boolean matchAsciiBuffer(String expected, byte[] buffer,
                                             int offset, int length) {
-        byte[] buffer1;
-        try {
-            buffer1 = expected.getBytes("ASCII");
-        } catch (UnsupportedEncodingException e) {
-            // Should not happen
-            throw new RuntimeException(e); //NOSONAR
-        }
+        byte[] buffer1 = expected.getBytes(StandardCharsets.US_ASCII);
         return isEqual(buffer1, 0, buffer1.length, buffer, offset, length,
                        false);
     }

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/tar/TarInputStream.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/tar/TarInputStream.java b/src/main/org/apache/tools/tar/TarInputStream.java
index e4bd53b..151461e 100644
--- a/src/main/org/apache/tools/tar/TarInputStream.java
+++ b/src/main/org/apache/tools/tar/TarInputStream.java
@@ -28,6 +28,7 @@ import java.io.FilterInputStream;
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
+import java.nio.charset.StandardCharsets;
 import java.util.HashMap;
 import java.util.Map;
 
@@ -449,7 +450,7 @@ public class TarInputStream extends FilterInputStream {
                             }
                             // Drop trailing NL
                             String value = new String(rest, 0,
-                                                      restLen - 1, "UTF-8");
+                                                      restLen - 1, StandardCharsets.UTF_8);
                             headers.put(keyword, value);
                             break;
                         }

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/tar/TarOutputStream.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/tar/TarOutputStream.java b/src/main/org/apache/tools/tar/TarOutputStream.java
index 0414e12..50cd9da 100644
--- a/src/main/org/apache/tools/tar/TarOutputStream.java
+++ b/src/main/org/apache/tools/tar/TarOutputStream.java
@@ -28,6 +28,7 @@ import java.io.IOException;
 import java.io.OutputStream;
 import java.io.StringWriter;
 import java.nio.ByteBuffer;
+import java.nio.charset.StandardCharsets;
 import java.util.Date;
 import java.util.HashMap;
 import java.util.Map;
@@ -500,7 +501,7 @@ public class TarOutputStream extends FilterOutputStream {
                 + 3 /* blank, equals and newline */
                 + 2 /* guess 9 < actual length < 100 */;
             String line = len + " " + key + "=" + value + "\n";
-            int actualLength = line.getBytes("UTF-8").length;
+            int actualLength = line.getBytes(StandardCharsets.UTF_8).length;
             while (len != actualLength) {
                 // Adjust for cases where length < 10 or > 100
                 // or where UTF-8 encoding isn't a single octet
@@ -509,11 +510,11 @@ public class TarOutputStream extends FilterOutputStream {
                 // first pass so we'd need a second.
                 len = actualLength;
                 line = len + " " + key + "=" + value + "\n";
-                actualLength = line.getBytes("UTF-8").length;
+                actualLength = line.getBytes(StandardCharsets.UTF_8).length;
             }
             w.write(line);
         }
-        byte[] data = w.toString().getBytes("UTF-8");
+        byte[] data = w.toString().getBytes(StandardCharsets.UTF_8);
         pex.setSize(data.length);
         putNextEntry(pex);
         write(data);

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/main/org/apache/tools/zip/AbstractUnicodeExtraField.java
----------------------------------------------------------------------
diff --git a/src/main/org/apache/tools/zip/AbstractUnicodeExtraField.java b/src/main/org/apache/tools/zip/AbstractUnicodeExtraField.java
index 0f236d4..b4b1713 100644
--- a/src/main/org/apache/tools/zip/AbstractUnicodeExtraField.java
+++ b/src/main/org/apache/tools/zip/AbstractUnicodeExtraField.java
@@ -18,7 +18,7 @@
 
 package org.apache.tools.zip;
 
-import java.io.UnsupportedEncodingException;
+import java.nio.charset.StandardCharsets;
 import java.util.zip.CRC32;
 import java.util.zip.ZipException;
 
@@ -51,12 +51,7 @@ public abstract class AbstractUnicodeExtraField implements ZipExtraField {
         crc32.update(bytes, off, len);
         nameCRC32 = crc32.getValue();
 
-        try {
-            unicodeName = text.getBytes("UTF-8");
-        } catch (final UnsupportedEncodingException e) {
-            throw new RuntimeException("FATAL: UTF-8 encoding not supported.", //NOSONAR
-                                       e);
-        }
+        unicodeName = text.getBytes(StandardCharsets.UTF_8);
     }
 
     /**

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/tests/junit/org/apache/tools/ant/taskdefs/EchoTest.java
----------------------------------------------------------------------
diff --git a/src/tests/junit/org/apache/tools/ant/taskdefs/EchoTest.java b/src/tests/junit/org/apache/tools/ant/taskdefs/EchoTest.java
index 1076697..d703687 100644
--- a/src/tests/junit/org/apache/tools/ant/taskdefs/EchoTest.java
+++ b/src/tests/junit/org/apache/tools/ant/taskdefs/EchoTest.java
@@ -24,6 +24,7 @@ import java.io.FileInputStream;
 import java.io.IOException;
 import java.io.InputStreamReader;
 import java.io.PrintStream;
+import java.nio.charset.StandardCharsets;
 
 import org.apache.tools.ant.DefaultLogger;
 import org.apache.tools.ant.Project;
@@ -77,7 +78,7 @@ public class EchoTest {
         echo.setFile(removeThis);
         echo.setEncoding("UTF-8");
         echo.execute();
-        String x = FileUtils.readFully(new InputStreamReader(new FileInputStream(removeThis), "UTF-8"));
+        String x = FileUtils.readFully(new InputStreamReader(new FileInputStream(removeThis), StandardCharsets.UTF_8));
         assertEquals(x, "\u00e4\u00a9");
     }
 

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/tests/junit/org/apache/tools/ant/taskdefs/JarTest.java
----------------------------------------------------------------------
diff --git a/src/tests/junit/org/apache/tools/ant/taskdefs/JarTest.java b/src/tests/junit/org/apache/tools/ant/taskdefs/JarTest.java
index 12072d7..bd490e8 100644
--- a/src/tests/junit/org/apache/tools/ant/taskdefs/JarTest.java
+++ b/src/tests/junit/org/apache/tools/ant/taskdefs/JarTest.java
@@ -24,6 +24,7 @@ import java.io.FileReader;
 import java.io.InputStream;
 import java.io.InputStreamReader;
 import java.io.IOException;
+import java.nio.charset.StandardCharsets;
 import java.util.zip.ZipEntry;
 import java.util.zip.ZipFile;
 
@@ -251,7 +252,7 @@ public class JarTest {
         try (ZipFile archive = new ZipFile(new File(getOutputDir(), tempJar))) {
             ZipEntry ze = archive.getEntry("META-INF/INDEX.LIST");
             InputStream is = archive.getInputStream(ze);
-            BufferedReader r = new BufferedReader(new InputStreamReader(is, "UTF8"));
+            BufferedReader r = new BufferedReader(new InputStreamReader(is, StandardCharsets.UTF_8));
             boolean foundSub = false;
             boolean foundSubFoo = false;
             boolean foundFoo = false;

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/tests/junit/org/apache/tools/ant/taskdefs/cvslib/ChangeLogWriterTest.java
----------------------------------------------------------------------
diff --git a/src/tests/junit/org/apache/tools/ant/taskdefs/cvslib/ChangeLogWriterTest.java b/src/tests/junit/org/apache/tools/ant/taskdefs/cvslib/ChangeLogWriterTest.java
index 19b37a4..24b35c7 100644
--- a/src/tests/junit/org/apache/tools/ant/taskdefs/cvslib/ChangeLogWriterTest.java
+++ b/src/tests/junit/org/apache/tools/ant/taskdefs/cvslib/ChangeLogWriterTest.java
@@ -22,6 +22,7 @@ import java.io.ByteArrayOutputStream;
 import java.io.InputStream;
 import java.io.OutputStreamWriter;
 import java.io.PrintWriter;
+import java.nio.charset.StandardCharsets;
 import java.util.Date;
 
 import org.apache.tools.ant.util.JAXPUtils;
@@ -48,7 +49,7 @@ public class ChangeLogWriterTest {
         CVSEntry[] entries = {entry};
 
         ByteArrayOutputStream output = new ByteArrayOutputStream();
-        PrintWriter pwriter = new PrintWriter(new OutputStreamWriter(output, "UTF-8"));
+        PrintWriter pwriter = new PrintWriter(new OutputStreamWriter(output, StandardCharsets.UTF_8));
         writer.printChangeLog(pwriter, entries);
 
         // make sure that the parsing does not break

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/tests/junit/org/apache/tools/ant/types/PatternSetTest.java
----------------------------------------------------------------------
diff --git a/src/tests/junit/org/apache/tools/ant/types/PatternSetTest.java b/src/tests/junit/org/apache/tools/ant/types/PatternSetTest.java
index 031c7ab..5873f5c 100644
--- a/src/tests/junit/org/apache/tools/ant/types/PatternSetTest.java
+++ b/src/tests/junit/org/apache/tools/ant/types/PatternSetTest.java
@@ -32,6 +32,7 @@ import java.io.IOException;
 import java.io.OutputStream;
 import java.io.OutputStreamWriter;
 import java.io.Writer;
+import java.nio.charset.StandardCharsets;
 
 import static org.junit.Assert.assertArrayEquals;
 import static org.junit.Assert.assertEquals;
@@ -227,7 +228,7 @@ public class PatternSetTest {
         Writer w = null;
         try {
             o = new FileOutputStream(testFile);
-            w = new OutputStreamWriter(o, "UTF-16LE");
+            w = new OutputStreamWriter(o, StandardCharsets.UTF_16LE);
             w.write("\u00e4\n");
         } finally {
             FileUtils.close(w);

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/tests/junit/org/apache/tools/ant/util/ReaderInputStreamTest.java
----------------------------------------------------------------------
diff --git a/src/tests/junit/org/apache/tools/ant/util/ReaderInputStreamTest.java b/src/tests/junit/org/apache/tools/ant/util/ReaderInputStreamTest.java
index dcb3391..a82189e 100644
--- a/src/tests/junit/org/apache/tools/ant/util/ReaderInputStreamTest.java
+++ b/src/tests/junit/org/apache/tools/ant/util/ReaderInputStreamTest.java
@@ -24,6 +24,7 @@ import java.io.File;
 import java.io.FileInputStream;
 import java.io.InputStreamReader;
 import java.io.StringReader;
+import java.nio.charset.StandardCharsets;
 
 import static org.junit.Assert.assertEquals;
 
@@ -60,9 +61,9 @@ public class ReaderInputStreamTest {
             }
             bytes[pos++] = (byte) res;
         }
-        bytes = "abc".getBytes("utf-16");
+        bytes = "abc".getBytes(StandardCharsets.UTF_16);
         //        String n = new String(bytes, 0, pos, "utf-16");
-        new String(bytes, 0, bytes.length, "utf-16");
+        new String(bytes, 0, bytes.length, StandardCharsets.UTF_16);
     }
 
     @SuppressWarnings("resource")
@@ -81,7 +82,7 @@ public class ReaderInputStreamTest {
 
     @Test
     public void testPreample() throws Exception {
-        byte[] bytes = "".getBytes("utf-16");
+        byte[] bytes = "".getBytes(StandardCharsets.UTF_16);
         System.out.println("Preample len is " + bytes.length);
     }
 

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/tests/junit/org/apache/tools/zip/UTF8ZipFilesTest.java
----------------------------------------------------------------------
diff --git a/src/tests/junit/org/apache/tools/zip/UTF8ZipFilesTest.java b/src/tests/junit/org/apache/tools/zip/UTF8ZipFilesTest.java
index f58b775..70bd749 100644
--- a/src/tests/junit/org/apache/tools/zip/UTF8ZipFilesTest.java
+++ b/src/tests/junit/org/apache/tools/zip/UTF8ZipFilesTest.java
@@ -22,6 +22,7 @@ import java.io.File;
 import java.io.IOException;
 import java.io.InputStream;
 import java.nio.ByteBuffer;
+import java.nio.charset.StandardCharsets;
 import java.util.Collections;
 import java.util.zip.CRC32;
 import org.junit.Test;
@@ -146,7 +147,7 @@ public class UTF8ZipFilesTest {
             }
 
             zos.putNextEntry(ze);
-            zos.write("Hello, world!".getBytes("US-ASCII"));
+            zos.write("Hello, world!".getBytes(StandardCharsets.US_ASCII));
             zos.closeEntry();
 
             ze = new ZipEntry(EURO_FOR_DOLLAR_TXT);
@@ -162,7 +163,7 @@ public class UTF8ZipFilesTest {
             }
 
             zos.putNextEntry(ze);
-            zos.write("Give me your money!".getBytes("US-ASCII"));
+            zos.write("Give me your money!".getBytes(StandardCharsets.US_ASCII));
             zos.closeEntry();
 
             ze = new ZipEntry(ASCII_TXT);
@@ -179,7 +180,7 @@ public class UTF8ZipFilesTest {
             }
 
             zos.putNextEntry(ze);
-            zos.write("ascii".getBytes("US-ASCII"));
+            zos.write("ascii".getBytes(StandardCharsets.US_ASCII));
             zos.closeEntry();
         }
     }
@@ -225,8 +226,7 @@ public class UTF8ZipFilesTest {
             crc.update(ne.array(), ne.arrayOffset(), ne.limit());
 
             assertEquals(crc.getValue(), ucpf.getNameCRC32());
-            assertEquals(expectedName, new String(ucpf.getUnicodeName(),
-                                                  UTF_8));
+            assertEquals(expectedName, new String(ucpf.getUnicodeName(), StandardCharsets.UTF_8));
         }
     }
 

http://git-wip-us.apache.org/repos/asf/ant/blob/16d0f4d8/src/tests/junit/org/apache/tools/zip/ZipEncodingTest.java
----------------------------------------------------------------------
diff --git a/src/tests/junit/org/apache/tools/zip/ZipEncodingTest.java b/src/tests/junit/org/apache/tools/zip/ZipEncodingTest.java
index 66418ce..72bcb32 100644
--- a/src/tests/junit/org/apache/tools/zip/ZipEncodingTest.java
+++ b/src/tests/junit/org/apache/tools/zip/ZipEncodingTest.java
@@ -21,6 +21,7 @@ package org.apache.tools.zip;
 
 import java.io.IOException;
 import java.nio.ByteBuffer;
+import java.nio.charset.StandardCharsets;
 
 import org.junit.Test;
 
@@ -142,9 +143,9 @@ public class ZipEncodingTest {
         assertByteEquals(testBytes, encoded);
 
         assertFalse(enc.canEncode(UNENC_STRING));
-        assertByteEquals("%U2016".getBytes("US-ASCII"), enc.encode(UNENC_STRING));
+        assertByteEquals("%U2016".getBytes(StandardCharsets.US_ASCII), enc.encode(UNENC_STRING));
         assertFalse(enc.canEncode(BAD_STRING));
-        assertByteEquals(BAD_STRING_ENC.getBytes("US-ASCII"),
+        assertByteEquals(BAD_STRING_ENC.getBytes(StandardCharsets.US_ASCII),
                 enc.encode(BAD_STRING));
     }