You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2013/05/07 00:39:57 UTC

svn commit: r1479706 - /commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/Lexer.java

Author: sebb
Date: Mon May  6 22:38:23 2013
New Revision: 1479706

URL: http://svn.apache.org/r1479706
Log:
Extract check for metaChar

Modified:
    commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/Lexer.java

Modified: commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/Lexer.java
URL: http://svn.apache.org/viewvc/commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/Lexer.java?rev=1479706&r1=1479705&r2=1479706&view=diff
==============================================================================
--- commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/Lexer.java (original)
+++ commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/Lexer.java Mon May  6 22:38:23 2013
@@ -108,7 +108,7 @@ abstract class Lexer {
             throw new IOException("EOF whilst processing escape sequence");
         default:
             // Now check for meta-characters
-            if (isDelimiter(c) || isEscape(c) || isQuoteChar(c) || isCommentStart(c)) {
+            if (isMetaChar(c)) {
                 return c;
             }
             // indicate unexpected char - available from in.getLastChar()
@@ -181,4 +181,12 @@ abstract class Lexer {
     boolean isCommentStart(final int c) {
         return c == commmentStart;
     }
+    
+    private boolean isMetaChar(final int c) {
+        return c == delimiter
+            || c == escape
+            || c == quoteChar
+            || c == commmentStart
+                ;
+    }
 }