From 4e90cdfdafa3ad4fa1946cd72c7cb968dad39279 Mon Sep 17 00:00:00 2001
From: Arturo Bernal <arturobernalg@gmail.com>
Date: Fri, 22 Oct 2021 16:32:22 +0200
Subject: [PATCH] * Replace For loop with map.forEach (#172)

* Use diamond type <>
* Extract commons expressions
---
 src/main/java/org/apache/commons/csv/CSVRecord.java     | 8 ++++----
 src/main/java/org/apache/commons/csv/Lexer.java         | 4 ++--
 src/test/java/org/apache/commons/csv/CSVRecordTest.java | 2 +-
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/main/java/org/apache/commons/csv/CSVRecord.java b/src/main/java/org/apache/commons/csv/CSVRecord.java
index a8aa082f2..e76bbc4c2 100644
--- a/src/main/java/org/apache/commons/csv/CSVRecord.java
+++ b/src/main/java/org/apache/commons/csv/CSVRecord.java
@@ -270,10 +270,10 @@ public <M extends Map<String, String>> M putIn(final M map) {
         if (getHeaderMapRaw() == null) {
             return map;
         }
-        getHeaderMapRaw().entrySet().forEach(entry -> {
-            final int col = entry.getValue().intValue();
+        getHeaderMapRaw().forEach((key, value) -> {
+            final int col = value;
             if (col < values.length) {
-                map.put(entry.getKey(), values[col]);
+                map.put(key, values[col]);
             }
         });
         return map;
@@ -314,7 +314,7 @@ public List<String> toList() {
      * @return A new Map. The map is empty if the record has no headers.
      */
     public Map<String, String> toMap() {
-        return putIn(new LinkedHashMap<String, String>(values.length));
+        return putIn(new LinkedHashMap<>(values.length));
     }
 
     /**
diff --git a/src/main/java/org/apache/commons/csv/Lexer.java b/src/main/java/org/apache/commons/csv/Lexer.java
index a8afc5016..edd9576ce 100644
--- a/src/main/java/org/apache/commons/csv/Lexer.java
+++ b/src/main/java/org/apache/commons/csv/Lexer.java
@@ -413,6 +413,7 @@ private Token parseSimpleToken(final Token token, int ch) throws IOException {
                 token.type = TOKEN;
                 break;
             }
+            // continue
             if (isEscape(ch)) {
                 if (isEscapeDelimiter()) {
                     token.content.append(delimiter);
@@ -424,11 +425,10 @@ private Token parseSimpleToken(final Token token, int ch) throws IOException {
                         token.content.append((char) unescaped);
                     }
                 }
-                ch = reader.read(); // continue
             } else {
                 token.content.append((char) ch);
-                ch = reader.read(); // continue
             }
+            ch = reader.read(); // continue
         }
 
         if (ignoreSurroundingSpaces) {
diff --git a/src/test/java/org/apache/commons/csv/CSVRecordTest.java b/src/test/java/org/apache/commons/csv/CSVRecordTest.java
index 39b02ba26..8a92a7e1e 100644
--- a/src/test/java/org/apache/commons/csv/CSVRecordTest.java
+++ b/src/test/java/org/apache/commons/csv/CSVRecordTest.java
@@ -207,7 +207,7 @@ public void testPutInMap() {
         this.recordWithHeader.putIn(map);
         this.validateMap(map, false);
         // Test that we can compile with assignment to the same map as the param.
-        final TreeMap<String, String> map2 = recordWithHeader.putIn(new TreeMap<String, String>());
+        final TreeMap<String, String> map2 = recordWithHeader.putIn(new TreeMap<>());
         this.validateMap(map2, false);
     }