Deprecate CSVParser constructors

Author: garydgregory

src/changes/changes.xml

@@ -44,7 +44,7 @@
       <!-- FIX -->
       <!-- ADD -->
       <action type="add" issue="CSV-313" dev="ggregory" due-to="Gary Gregory">Add CSVPrinter.getRecordCount().</action>
-      <action type="add" dev="ggregory" due-to="Gary Gregory">Add and use CSVParser.Builder and builder().</action>
+      <action type="add" dev="ggregory" due-to="Gary Gregory">Add and use CSVParser.Builder and builder() and deprecate CSVParser constructors.</action>
       <!-- UPDATE -->
       <action type="update" dev="ggregory" due-to="Gary Gregory, Dependabot">Bump org.apache.commons:commons-parent from 76 to 78 #486, #495.</action>
       <action type="update" dev="ggregory" due-to="Gary Gregory, Dependabot">Bump org.codehaus.mojo:taglist-maven-plugin from 3.1.0 to 3.2.1 #493.</action> 

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

@@ -476,7 +476,9 @@ public static CSVParser parse(final URL url, final Charset charset, final CSVFor
      * @throws IOException
      *             If there is a problem reading the header or skipping the first record
      * @throws CSVException Thrown on invalid input.
+     * @deprecated Will be removed in the next major version, use {@link Builder#get()}.
      */
+    @Deprecated
     public CSVParser(final Reader reader, final CSVFormat format) throws IOException {
         this(reader, format, 0, 1);
     }
@@ -503,7 +505,9 @@ public CSVParser(final Reader reader, final CSVFormat format) throws IOException
      *             If there is a problem reading the header or skipping the first record
      * @throws CSVException Thrown on invalid input.
      * @since 1.1
+     * @deprecated Will be private in the next major version, use {@link Builder#get()}.
      */
+    @Deprecated
     @SuppressWarnings("resource")
     public CSVParser(final Reader reader, final CSVFormat format, final long characterOffset, final long recordNumber)
         throws IOException {