Chore: change domain dedupe method

SukkaW · Nov 3, 2023 · 31d021d · 31d021d
1 parent 5701fc9
commit 31d021d
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 8 deletions.
diff --git a/Build/lib/domain-deduper.js b/Build/lib/domain-deduper.js
@@ -20,7 +20,8 @@ const domainDeduper = (inputDomains) => {
 
     // if `.example.com` exists, then `example.com` should also be removed
     const a = d.slice(1);
-    if (trie.has(a)) {
+
+    if (sets.has(a)) {
       sets.delete(a);
     }
   }

diff --git a/Build/lib/trie.js b/Build/lib/trie.js
@@ -84,9 +84,7 @@ const createTrie = (from) => {
       // eslint-disable-next-line guard-for-in -- plain object
       for (k in node) {
         if (k === SENTINEL) {
-          if (includeEqualWithSuffix) {
-            matches.push($suffix);
-          } else if ($suffix !== suffix) {
+          if (includeEqualWithSuffix || $suffix !== suffix) {
             matches.push($suffix);
           }
 
@@ -157,11 +155,9 @@ const createTrie = (from) => {
    */
   const has = (suffix) => {
     let node = root;
-    let token;
 
     for (let i = suffix.length - 1; i >= 0; i--) {
-      token = suffix[i];
-      node = node[token];
+      node = node[suffix[i]];
 
       if (typeof node === 'undefined') {
         return false;

diff --git a/Source/domainset/reject_sukka.conf b/Source/domainset/reject_sukka.conf
@@ -1450,7 +1450,7 @@ stat.letv.com
 static.app.m.letv.com
 
 # >> Xiaomi (MIUI)
-ad.intl.xiaomi.com
+.ad.intl.xiaomi.com
 .ad.mi.com
 .admob.xiaomi.com
 .ad1.xiaomi.com