Noissue - Do not add subdomain wildcard if subdomain excluded

lib/abp2blocklist.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2017 eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 250 matching lines @@
     trigger["resource-type"] = getResourceTypes(filter);
 
     if (trigger["resource-type"].length == 0)
       return;
   }
 
   if (filter.thirdParty != null)
     trigger["load-type"] = [filter.thirdParty ? "third-party" : "first-party"];
 
   if (included.length > 0)
-    trigger["if-domain"] = included.map(name => "*" + name);
+  {
+    trigger["if-domain"] = included.map(name =>
+    {
+      if (filter instanceof filterClasses.WhitelistFilter)
+        return "*" + name;
+
+      // If this is not a whitelisting filter, add the subdomain wildcard only
+      // if no subdomains have been excluded.
+      let regExp = new RegExp("\." + escapeRegExp(name) + "$");
+      if (!excluded.some(name => regExp.test(name)))
+        return "*" + name;
+
+      return name;
+    });
+  }
   else if (excluded.length > 0)
+  {
     trigger["unless-domain"] = excluded.map(name => "*" + name);
+  }
 
   rules.push({trigger: trigger, action: {type: action}});
 }
 
 function hasNonASCI(obj)
 {
   if (typeof obj == "string")
   {
     if (/[^\x00-\x7F]/.test(obj))
       return true;
@@ skipping 167 matching lines @@
 
   for (let filter of this.elemhideExceptions)
     convertFilterAddRules(rules, filter, "ignore-previous-rules", false);
   for (let filter of this.requestFilters)
     convertFilterAddRules(rules, filter, "block", true);
   for (let filter of this.requestExceptions)
     convertFilterAddRules(rules, filter, "ignore-previous-rules", true);
 
   return rules.filter(rule => !hasNonASCI(rule));
 };