refactor: expectedRecordLength normalization

Author: wdavidw

lib/browser/index.js

@@ -620,7 +620,8 @@ var Parser = /*#__PURE__*/function (_Transform) {
         escaping: false,
         // escapeIsQuote: options.escape === options.quote,
         escapeIsQuote: Buffer.isBuffer(options.escape) && Buffer.isBuffer(options.quote) && Buffer.compare(options.escape, options.quote) === 0,
-        expectedRecordLength: options.columns === null ? 0 : options.columns.length,
+        // columns can be `false`, `true`, `Array`
+        expectedRecordLength: Array.isArray(options.columns) ? options.columns.length : undefined,
         field: new ResizeableBuffer(20),
         firstLineToHeaders: fnFirstLineToHeaders,
         info: Object.assign({}, this.info),

lib/browser/sync.js

@@ -620,7 +620,8 @@ var Parser = /*#__PURE__*/function (_Transform) {
         escaping: false,
         // escapeIsQuote: options.escape === options.quote,
         escapeIsQuote: Buffer.isBuffer(options.escape) && Buffer.isBuffer(options.quote) && Buffer.compare(options.escape, options.quote) === 0,
-        expectedRecordLength: options.columns === null ? 0 : options.columns.length,
+        // columns can be `false`, `true`, `Array`
+        expectedRecordLength: Array.isArray(options.columns) ? options.columns.length : undefined,
         field: new ResizeableBuffer(20),
         firstLineToHeaders: fnFirstLineToHeaders,
         info: Object.assign({}, this.info),

lib/es5/index.js

@@ -512,7 +512,8 @@ var Parser = /*#__PURE__*/function (_Transform) {
         escaping: false,
         // escapeIsQuote: options.escape === options.quote,
         escapeIsQuote: Buffer.isBuffer(options.escape) && Buffer.isBuffer(options.quote) && Buffer.compare(options.escape, options.quote) === 0,
-        expectedRecordLength: options.columns === null ? 0 : options.columns.length,
+        // columns can be `false`, `true`, `Array`
+        expectedRecordLength: Array.isArray(options.columns) ? options.columns.length : undefined,
         field: new ResizeableBuffer(20),
         firstLineToHeaders: fnFirstLineToHeaders,
         info: Object.assign({}, this.info),

lib/index.js

@@ -439,7 +439,8 @@ class Parser extends Transform {
       escaping: false,
       // escapeIsQuote: options.escape === options.quote,
       escapeIsQuote: Buffer.isBuffer(options.escape) && Buffer.isBuffer(options.quote) && Buffer.compare(options.escape, options.quote) === 0,
-      expectedRecordLength: options.columns === null ? 0 : options.columns.length,
+      // columns can be `false`, `true`, `Array`
+      expectedRecordLength: Array.isArray(options.columns) ? options.columns.length : undefined,
       field: new ResizeableBuffer(20),
       firstLineToHeaders: fnFirstLineToHeaders,
       info: Object.assign({}, this.info),