Use {} for hardlink tracking instead of []

isaacs · May 15, 2019 · 9fc84b9 · 9fc84b9
1 parent 15e59f1
commit 9fc84b9
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/lib/parse.js b/lib/parse.js
@@ -38,7 +38,7 @@ function Parse () {
   me._stream = new BlockStream(512)
   me.position = 0
   me._ended = false
-  me._hardLinks = []
+  me._hardLinks = {}
 
   me._stream.on("error", function (e) {
     me.emit("error", e)
@@ -251,18 +251,14 @@ Parse.prototype._startEntry = function (c) {
 
   if (onend) entry.on("end", onend)
 
-  if (entry.type === "File") {
-    this._hardLinks.forEach(function(link) {
-      if (link.path === entry.path) {
-        ev = "ignoredEntry"
-      }
-    })
+  if (entry.type === "File" && this._hardLinks[entry.path]) {
+    ev = "ignoredEntry"
   }
 
   this._entry = entry
 
   if (entry.type === "Link") {
-    this._hardLinks.push(entry)
+    this._hardLinks[entry.path] = entry
   }
 
   var me = this