From 14109b33d60a2a79be2912c06be5d11cdca5b2d1 Mon Sep 17 00:00:00 2001 From: gorhill Date: Sun, 22 Oct 2017 14:20:33 -0400 Subject: [PATCH] minor code review --- src/js/reverselookup-worker.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/js/reverselookup-worker.js b/src/js/reverselookup-worker.js index 71c272014..569212868 100644 --- a/src/js/reverselookup-worker.js +++ b/src/js/reverselookup-worker.js @@ -98,20 +98,20 @@ var fromNetFilter = function(details) { var fromCosmeticFilter = function(details) { var match = /^#@?#/.exec(details.rawFilter), prefix = match[0], - filter = details.rawFilter.slice(prefix.length), - selector = filter; + needle = details.rawFilter.slice(prefix.length), + selector = needle; // With low generic simple cosmetic filters, the class or id prefix // character is not part of the compiled data. So we must be ready to // look-up version of the selector without the prefix character. - var idOrClassPrefix = filter.charAt(0), + var idOrClassPrefix = needle.charAt(0), cssPrefixMatcher; if ( idOrClassPrefix === '#' ) { cssPrefixMatcher = '#?'; - filter = filter.slice(1); + needle = needle.slice(1); } else if ( idOrClassPrefix === '.' ) { cssPrefixMatcher = '\\.?'; - filter = filter.slice(1); + needle = needle.slice(1); } else { idOrClassPrefix = ''; cssPrefixMatcher = ''; @@ -122,7 +122,7 @@ var fromCosmeticFilter = function(details) { var reFilter = new RegExp( '^\\[\\d,[^\\n]*\\\\*"' + cssPrefixMatcher + - reEscapeCosmetic(filter) + + reEscapeCosmetic(needle) + '\\\\*"[^\\n]*\\]$', 'gm' );