fix: revert PR #3379 (avoid useless spread) (#3382)

* Update result.js * Update build-result.js * fix: lint * fix: lint
brianc · Feb 13, 2025 · f6e829c · f6e829c
1 parent 7325807
commit f6e829c
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/packages/pg-native/lib/build-result.js b/packages/pg-native/lib/build-result.js
@@ -20,15 +20,16 @@ class Result {
   consumeFields(pq) {
     const nfields = pq.nfields()
     this.fields = new Array(nfields)
-    this._prebuiltEmptyResultObject = {}
+    var row = {}
     for (var x = 0; x < nfields; x++) {
       var name = pq.fname(x)
-      this._prebuiltEmptyResultObject[name] = null
+      row[name] = null
       this.fields[x] = {
         name: name,
         dataTypeID: pq.ftype(x),
       }
     }
+    this._prebuiltEmptyResultObject = { ...row }
   }
 
   consumeRows(pq) {

diff --git a/packages/pg/lib/result.js b/packages/pg/lib/result.js
@@ -88,18 +88,20 @@ class Result {
       this._parsers = new Array(fieldDescriptions.length)
     }
 
-    this._prebuiltEmptyResultObject = {}
+    var row = {}
 
     for (var i = 0; i < fieldDescriptions.length; i++) {
       var desc = fieldDescriptions[i]
-      this._prebuiltEmptyResultObject[desc.name] = null
+      row[desc.name] = null
 
       if (this._types) {
         this._parsers[i] = this._types.getTypeParser(desc.dataTypeID, desc.format || 'text')
       } else {
         this._parsers[i] = types.getTypeParser(desc.dataTypeID, desc.format || 'text')
       }
     }
+
+    this._prebuiltEmptyResultObject = { ...row }
   }
 }