Moving emptyLine state setting from processColumn to parseCycle. Lint…

…er fixes.
vslinko · Sep 21, 2023 · 5660dc2 · 5660dc2
1 parent a4357dd
commit 5660dc2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/reader.ts b/reader.ts
@@ -189,7 +189,6 @@ export class CSVReader {
   }
 
   private processColumn() {
-    this.emptyLine = false;
     const result = this.decoder.decode(
       this.columnBuffer.subarray(0, this.columnBufferIndex),
     );
@@ -360,6 +359,7 @@ export class CSVReader {
 
       if (!this.inColumn && this.hasNext(this.columnSeparator)) {
         this.debug("columnSeparator");
+        this.emptyLine = false;
         this.processColumn();
         this.skip(this.columnSeparator.length);
         continue;

diff --git a/reader_test.ts b/reader_test.ts
@@ -429,14 +429,14 @@ Deno.test({
       `col1,col2,col3
 a,b,c
 ,,
-d,e,f`
+d,e,f`,
     );
     const rows = await asyncArrayFrom(readCSVRows(reader));
     assertEquals(rows, [
       ["col1", "col2", "col3"],
       ["a", "b", "c"],
       ["", "", ""],
-      ["d", "e", "f"]
-    ])
-  }
-})
+      ["d", "e", "f"],
+    ]);
+  },
+});