From 7df132a848b9740c0e7c97339e8989b724a4c082 Mon Sep 17 00:00:00 2001 From: Gary Gregory Date: Fri, 5 Mar 2021 13:54:34 -0500 Subject: [PATCH] No need to nest in else. --- .../java/org/apache/commons/csv/Lexer.java | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/src/main/java/org/apache/commons/csv/Lexer.java b/src/main/java/org/apache/commons/csv/Lexer.java index 2795ca29..46e97917 100644 --- a/src/main/java/org/apache/commons/csv/Lexer.java +++ b/src/main/java/org/apache/commons/csv/Lexer.java @@ -300,14 +300,17 @@ final class Lexer implements Closeable { if (isDelimiter(c)) { token.type = TOKEN; return token; - } else if (isEndOfFile(c)) { + } + if (isEndOfFile(c)) { token.type = EOF; token.isReady = true; // There is data at EOF return token; - } else if (readEndOfLine(c)) { + } + if (readEndOfLine(c)) { token.type = EORECORD; return token; - } else if (!isWhitespace(c)) { + } + if (!isWhitespace(c)) { // error invalid char between token and next delimiter throw new IOException("(line " + getCurrentLineNumber() + ") invalid char between encapsulated token and delimiter"); @@ -350,14 +353,17 @@ final class Lexer implements Closeable { if (readEndOfLine(ch)) { token.type = EORECORD; break; - } else if (isEndOfFile(ch)) { + } + if (isEndOfFile(ch)) { token.type = EOF; token.isReady = true; // There is data at EOF break; - } else if (isDelimiter(ch)) { + } + if (isDelimiter(ch)) { token.type = TOKEN; break; - } else if (isEscape(ch)) { + } + if (isEscape(ch)) { final int unescaped = readEscape(); if (unescaped == END_OF_STREAM) { // unexpected char after escape token.content.append((char) ch).append((char) reader.getLastChar());