[CSV] deprecated setStrategy, made most private members final

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

Author: yonik

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

@@ -69,16 +69,17 @@ public class CSVParser {
   private static final String[] EMPTY_STRING_ARRAY = new String[0];
 
   // the input stream
-  private ExtendedBufferedReader in;
+  private final ExtendedBufferedReader in;
 
+  // TODO: this can be made final if setStrategy is removed
   private CSVStrategy strategy;
 
   // the following objects are shared to reduce garbage 
   /** A record buffer for getLine(). Grows as necessary and is reused. */
-  private ArrayList record = new ArrayList();
-  private Token reusableToken = new Token();
-  private CharBuffer wsBuf = new CharBuffer();
-  private CharBuffer code = new CharBuffer(4);
+  private final ArrayList record = new ArrayList();
+  private final Token reusableToken = new Token();
+  private final CharBuffer wsBuf = new CharBuffer();
+  private final CharBuffer code = new CharBuffer(4);
 
 
   /**
@@ -567,6 +568,7 @@ protected int unicodeEscapeLexer(int c) throws IOException {
    * Sets the specified CSV Strategy
    *
    * @return current instance of CSVParser to allow chained method calls
+   * @deprecated
    */
   public CSVParser setStrategy(CSVStrategy strategy) {
     this.strategy = strategy;