src: cache length in loops

Author: wdavidw

CHANGELOG.md

@@ -10,6 +10,10 @@
 * promise: new API module
 * errors: finish normalisation of all errors
 
+## Trunk
+
+* src: cache length in loops
+
 ## Version 4.8.8
 
 * api: fix regression in browser environments

lib/es5/index.js

@@ -858,7 +858,7 @@ var Parser = /*#__PURE__*/function (_Transform) {
         if (columns !== false) {
           var obj = {}; // Transform record array to an object
 
-          for (var i in record) {
+          for (var i = 0, l = record.length; i < l; i++) {
             if (columns[i] === undefined || columns[i].disabled) continue;
             obj[columns[i].name] = record[i];
           }
@@ -1326,7 +1326,7 @@ var isRecordEmpty = function isRecordEmpty(record) {
 var normalizeColumnsArray = function normalizeColumnsArray(columns) {
   var normalizedColumns = [];
 
-  for (var i = 0; i < columns.length; i++) {
+  for (var i = 0, l = columns.length; i < l; i++) {
     var column = columns[i];
 
     if (column === undefined || column === null || column === false) {

lib/index.js

@@ -726,7 +726,7 @@ class Parser extends Transform {
       if(columns !== false){
         const obj = {}
         // Transform record array to an object
-        for(let i in record){
+        for(let i = 0, l = record.length; i < l; i++){
           if(columns[i] === undefined || columns[i].disabled) continue
           obj[columns[i].name] = record[i]
         }
@@ -1090,8 +1090,7 @@ const isRecordEmpty = function(record){
 
 const normalizeColumnsArray = function(columns){
   const normalizedColumns = [];
-
-  for(let i=0; i< columns.length; i++){
+  for(let i = 0, l = columns.length; i < l; i++){
     const column = columns[i]
     if(column === undefined || column === null || column === false){
       normalizedColumns[i] = { disabled: true }