Revert "[IO-414] don't write a BOM on every (or any) line (#492)"

This reverts commit 469ef76.

Author: garydgregory

src/main/java/org/apache/commons/io/IOUtils.java

@@ -45,7 +45,6 @@
 import java.nio.channels.ReadableByteChannel;
 import java.nio.channels.Selector;
 import java.nio.charset.Charset;
-import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.util.Arrays;
 import java.util.Collection;
@@ -3829,15 +3828,11 @@ public static void writeLines(final Collection<?> lines, final String lineEnding
      * an {@link OutputStream} line by line, using the specified character
      * encoding and the specified line ending.
      *
-     * UTF-16 is written big-endian with no byte order mark.
-     * For little endian, use UTF-16LE. For a BOM, write it to the stream
-     * before calling this method.
-     *
      * @param lines the lines to write, null entries produce blank lines
      * @param lineEnding the line separator to use, null is system default
      * @param output the {@link OutputStream} to write to, not null, not closed
      * @param charset the charset to use, null means platform default
-     * @throws NullPointerException if output is null
+     * @throws NullPointerException if the output is null
      * @throws IOException          if an I/O error occurs
      * @since 2.3
      */
@@ -3849,11 +3844,7 @@ public static void writeLines(final Collection<?> lines, String lineEnding, fina
         if (lineEnding == null) {
             lineEnding = System.lineSeparator();
         }
-        Charset cs = Charsets.toCharset(charset);
-        // don't write a BOM
-        if (cs == StandardCharsets.UTF_16) {
-            cs = StandardCharsets.UTF_16BE;
-        }
+        final Charset cs = Charsets.toCharset(charset);
         final byte[] eolBytes = lineEnding.getBytes(cs);
         for (final Object line : lines) {
             if (line != null) {

src/test/java/org/apache/commons/io/IOUtilsTest.java

@@ -1761,13 +1761,4 @@ public void testWriteLittleString() throws IOException {
         }
     }
 
-    @Test
-    public void testWriteLines() throws IOException {
-        final String[] data = {"The", "quick"};
-        final ByteArrayOutputStream out = new ByteArrayOutputStream();
-        IOUtils.writeLines(Arrays.asList(data), "\n", out, "UTF-16");
-        final String result = new String(out.toByteArray(), StandardCharsets.UTF_16);
-        assertEquals("The\nquick\n", result);
-    }
-
 }