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 !!