Comment out empty if-block; reset checkLF

FasterXML · Feb 12, 2024 · ac02346 · ac02346
1 parent 3ab5ac5
commit ac02346
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/csv/src/main/java/com/fasterxml/jackson/dataformat/csv/impl/CsvDecoder.java b/csv/src/main/java/com/fasterxml/jackson/dataformat/csv/impl/CsvDecoder.java
@@ -832,13 +832,20 @@ protected String _nextQuotedString() throws IOException
                     _owner._reportParsingError("Missing closing quote for value"); // should indicate start position?
                 }
                 ptr = _inputPtr;
-                if (checkLF && inputBuffer[ptr] == '\n') {
-                    // undo earlier advancement, to keep line number correct
-                    --_currInputRow;
+                if (checkLF) {
+                    checkLF = false; // better reset
+                    if (inputBuffer[ptr] == '\n') {
+                        // undo earlier advancement, to keep line number correct
+                        --_currInputRow;
+                    }
                 }
             }
+            // 11-Feb-2024, tatu: Not quite sure what was supposed to happen here;
+            //   but nothing was done. Leaving for now, remove from 2.18 or later
+            /*
             if (checkLF) { // had a "hanging" CR in parse loop; check now
             }
+            */
             if (outPtr >= outBuf.length) {
                 outBuf = _textBuffer.finishCurrentSegment();
                 outPtr = 0;