You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2012/10/13 19:45:06 UTC

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

Author: ggregory
Date: Sat Oct 13 17:45:05 2012
New Revision: 1397914

URL: http://svn.apache.org/viewvc?rev=1397914&view=rev
Log:
Rename method from "is" prefix to "read" prefix because it is not just a test method, it may actually consume input.

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=1397914&r1=1397913&r2=1397914&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 Sat Oct 13 17:45:05 2012
@@ -113,9 +113,9 @@ abstract class Lexer {
     /**
      * Greedily accepts \n, \r and \r\n This checker consumes silently the second control-character...
      *
-     * @return true if the given character is a line-terminator
+     * @return true if the given or next character is a line-terminator
      */
-    boolean isEndOfLine(int c) throws IOException {
+    boolean readEndOfLine(int c) throws IOException {
         // check if we have \r\n...
         if (c == CR && in.lookAhead() == LF) {
             // note: does not change c outside of this method !!