Truncate old value when replacing an element

mmomtchev · Aug 22, 2020 · d56af30 · d56af30
1 parent 01244ba
commit d56af30
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/index.es.js b/index.es.js
@@ -378,8 +378,10 @@ class SharedMap {
                 this.stats.collisions++;
                 /* Replacing existing key */
                 if (this._match(key, pos)) {
-                    for (let i = 0; i < value.length; i++)
+                    let i;
+                    for (i = 0; i < value.length; i++)
                         this.valuesData[pos * this.meta[META.objSize] + i] = value.charCodeAt(i);
+                    this.valuesData[pos * this.meta[META.objSize] + i] = 0;
                     exclusive || this._unlockLine(slidingLock);
                     return;
                 }

diff --git a/index.js b/index.js
@@ -379,8 +379,10 @@ export default class SharedMap {
                 this.stats.collisions++;
                 /* Replacing existing key */
                 if (this._match(key, pos)) {
-                    for (let i = 0; i < value.length; i++)
+                    let i;
+                    for (i = 0; i < value.length; i++)
                         this.valuesData[pos * this.meta[META.objSize] + i] = value.charCodeAt(i);
+                    this.valuesData[pos * this.meta[META.objSize] + i] = 0;
                     exclusive || this._unlockLine(slidingLock);
                     return;
                 }

diff --git a/index.umd.js b/index.umd.js
@@ -384,8 +384,10 @@
                    this.stats.collisions++;
                    /* Replacing existing key */
                    if (this._match(key, pos)) {
-                       for (let i = 0; i < value.length; i++)
+                       let i;
+                       for (i = 0; i < value.length; i++)
                            this.valuesData[pos * this.meta[META.objSize] + i] = value.charCodeAt(i);
+                       this.valuesData[pos * this.meta[META.objSize] + i] = 0;
                        exclusive || this._unlockLine(slidingLock);
                        return;
                    }