Additional pure annotations for Rollup tree shaking #101

mozilla-rally · Jun 6, 2021 · f99f1d6 · f99f1d6
1 parent 9699c65
commit f99f1d6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/events.js b/src/events.js
@@ -192,14 +192,14 @@ class EventSingleton extends Event {
     singleton: false
 }) {
     if(singleton) {
-        return new EventSingleton({
+        return /*@__PURE__*/new EventSingleton({
             name,
             addListenerCallback,
             removeListenerCallback,
             notifyListenersCallback
         });
     }
-    return new Event({
+    return /*@__PURE__*/new Event({
         name,
         addListenerCallback,
         removeListenerCallback,

diff --git a/src/linkResolution.js b/src/linkResolution.js
@@ -36,21 +36,21 @@ import linkResolutionGoogleNewsContentScript from "include:./content-scripts/lin
  * @see {@link https://amp.dev/documentation/guides-and-tutorials/learn/amp-caches-and-cors/amp-cache-urls/}
  * @constant {RegExp}
  */
-export const ampRegExp = new RegExp(
+export const ampRegExp = /*@__PURE__*/(() => new RegExp(
     // AMP cache regular expression
     `(?:^https?://(?:(?<ampCacheSubdomain>[a-zA-Z0-9\\-\\.]*)\\.)?(?<ampCacheDomain>${ampCacheDomains.map(matching.escapeRegExpString).join("|")})/(?<ampCacheContentType>c|i|r)/(?<ampCacheIsSecure>s/)?(?<ampCacheUrl>.*)$)`
     + `|` +
     // AMP viewer regular expression
     `(?:^https?://(?<ampViewerDomainAndPath>${ampViewerDomainsAndPaths.map(matching.escapeRegExpString).join("|")})/(?<ampViewerUrl>.*)$)`
-    , "i");
+    , "i"))();
 
 /**
  * A MatchPatternSet for AMP caches and viewers.
  * @constant {matching.MatchPatternSet}
  */
-export const ampMatchPatternSet = matching.createMatchPatternSet(
+export const ampMatchPatternSet = /*@__PURE__*/(() => matching.createMatchPatternSet(
     matching.domainsToMatchPatterns(ampCacheDomains, false).concat(
-        ampViewerDomainsAndPaths.map(ampViewerDomainAndPath => `*://${ampViewerDomainAndPath}*`)));
+        ampViewerDomainsAndPaths.map(ampViewerDomainAndPath => `*://${ampViewerDomainAndPath}*`))))();
 
 /**
  * Parse the underlying URL from an AMP cache or viewer URL, if the URL is an AMP cache or viewer URL.