Remove fallback logic for charset parameter

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/csv/trunk@1519269 13f79535-47bb-0310-9956-ffa450edef68

Author: britter

src/main/java/org/apache/commons/csv/CSVParser.java

@@ -143,18 +143,18 @@ public static CSVParser parse(String string, final CSVFormat format) throws IOEx
      * @param url
      *            a URL. Must not be null.
      * @param charset
-     *            the charset for the resource, if {@code null}, uses {@code UTF-8}. UTF-8 is one of the encodings
-     *            required by the Java specification.
+     *            the charset for the resource. Must not be null.
      * @param format
      *            the CSVFormat used for CSV parsing. Must not be null.
      * @return a new parser
      * @throws IllegalArgumentException
-     *             If the parameters of the format are inconsistent or if either url or format are null.
+     *             If the parameters of the format are inconsistent or if either url, charset or format are null.
      * @throws IOException
      *             If an I/O error occurs
      */
     public static CSVParser parse(URL url, Charset charset, final CSVFormat format) throws IOException {
         Assertions.notNull(url, "url");
+        Assertions.notNull(charset, "charset");
         Assertions.notNull(format, "format");
 
         return new CSVParser(new InputStreamReader(url.openStream(),

src/test/java/org/apache/commons/csv/CSVParserTest.java

@@ -749,6 +749,11 @@ public void testParseNullUrlCharsetFormat() throws Exception {
         CSVParser.parse(null, Charset.defaultCharset(), CSVFormat.DEFAULT);
     }
 
+    @Test(expected = IllegalArgumentException.class)
+    public void testParserUrlNullCharsetFormat() throws Exception {
+        CSVParser.parse(new URL("http://commons.apache.org"), null, CSVFormat.DEFAULT);
+    }
+
     @Test(expected = IllegalArgumentException.class)
     public void testParseUrlCharsetNullFormat() throws Exception {
         CSVParser.parse(new URL("http://commons.apache.org"), Charset.defaultCharset(), null);