You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by br...@apache.org on 2016/05/05 20:02:36 UTC
svn commit: r1742468 - in
/commons/proper/csv/trunk/src/main/java/org/apache/commons/csv:
ExtendedBufferedReader.java Lexer.java
Author: britter
Date: Thu May 5 20:02:35 2016
New Revision: 1742468
URL: http://svn.apache.org/viewvc?rev=1742468&view=rev
Log:
PMD: Remove useless parentheses
Modified:
commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/ExtendedBufferedReader.java
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/ExtendedBufferedReader.java
URL: http://svn.apache.org/viewvc/commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/ExtendedBufferedReader.java?rev=1742468&r1=1742467&r2=1742468&view=diff
==============================================================================
--- commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/ExtendedBufferedReader.java (original)
+++ commons/proper/csv/trunk/src/main/java/org/apache/commons/csv/ExtendedBufferedReader.java Thu May 5 20:02:35 2016
@@ -58,7 +58,7 @@ final class ExtendedBufferedReader exten
@Override
public int read() throws IOException {
final int current = super.read();
- if (current == CR || (current == LF && lastChar != CR)) {
+ if (current == CR || current == LF && lastChar != CR) {
eolCounter++;
}
lastChar = current;
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=1742468&r1=1742467&r2=1742468&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 Thu May 5 20:02:35 2016
@@ -110,7 +110,7 @@ final class Lexer implements Closeable {
}
// did we reach eof during the last iteration already ? EOF
- if (isEndOfFile(lastChar) || (!isDelimiter(lastChar) && isEndOfFile(c))) {
+ if (isEndOfFile(lastChar) || !isDelimiter(lastChar) && isEndOfFile(c)) {
token.type = EOF;
// don't set token.isReady here because no content
return token;