perf: no redundant _addToIndex call or _size invalidation when adding…

… existing quad (#403)
rdfjs · Jul 30, 2024 · 5094b2d · 5094b2d
1 parent 426819c
commit 5094b2d
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/N3Store.js b/src/N3Store.js
@@ -306,13 +306,14 @@ export default class N3Store {
     predicate = this._termToNewNumericId(predicate);
     object    = this._termToNewNumericId(object);
 
-    const changed = this._addToIndex(graphItem.subjects,   subject,   predicate, object);
+    if (!this._addToIndex(graphItem.subjects,   subject,   predicate, object))
+      return false;
     this._addToIndex(graphItem.predicates, predicate, object,    subject);
     this._addToIndex(graphItem.objects,    object,    subject,   predicate);
 
     // The cached quad count is now invalid
     this._size = null;
-    return changed;
+    return true;
   }
 
   // ### `addQuads` adds multiple quads to the store