This is an automated email from the ASF dual-hosted git repository. ggregory pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-csv.git
commit b7ed400303a3b36c61683a578d8a671378db28af Author: Gary Gregory <[email protected]> AuthorDate: Mon Mar 11 16:31:20 2024 -0400 No need to store in local variable --- src/main/java/org/apache/commons/csv/CSVFormat.java | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/src/main/java/org/apache/commons/csv/CSVFormat.java b/src/main/java/org/apache/commons/csv/CSVFormat.java index 731fb2d7..c81e9428 100644 --- a/src/main/java/org/apache/commons/csv/CSVFormat.java +++ b/src/main/java/org/apache/commons/csv/CSVFormat.java @@ -2149,12 +2149,12 @@ public final class CSVFormat implements Serializable { int start = 0; int pos = 0; final int end = charSeq.length(); - final char[] delim = getDelimiterCharArray(); - final int delimLength = delim.length; + final char[] delimArray = getDelimiterCharArray(); + final int delimLength = delimArray.length; final char escape = getEscapeChar(); while (pos < end) { char c = charSeq.charAt(pos); - final boolean isDelimiterStart = isDelimiter(c, charSeq, pos, delim, delimLength); + final boolean isDelimiterStart = isDelimiter(c, charSeq, pos, delimArray, delimLength); final boolean isCr = c == CR; final boolean isLf = c == LF; if (isCr || isLf || c == escape || isDelimiterStart) { @@ -2171,8 +2171,7 @@ public final class CSVFormat implements Serializable { if (isDelimiterStart) { for (int i = 1; i < delimLength; i++) { pos++; - c = charSeq.charAt(pos); - escape(c, appendable); + escape(charSeq.charAt(pos), appendable); } } @@ -2223,8 +2222,7 @@ public final class CSVFormat implements Serializable { escape((char) c, appendable); if (isDelimiterStart) { for (int i = 1; i < delimLength; i++) { - c = bufferedReader.read(); - escape((char) c, appendable); + escape((char) bufferedReader.read(), appendable); } } start = pos + 1; // start on the current char after this one
