Unified Diff: lib/filterClasses.js
Issue 29790626:
Noissue - Avoid delete operator on RegExpFilter.regexpSource (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set:
Created May 25, 2018, 10:20 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/filterClasses.js |
| =================================================================== |
| --- a/lib/filterClasses.js |
| +++ b/lib/filterClasses.js |
| @@ -664,17 +664,17 @@ |
| // several times on Safari, due to WebKit bug 132872 |
| let prop = Object.getOwnPropertyDescriptor(this, "regexp"); |
| if (prop) |
| return prop.value; |
| let source = Filter.toRegExp(this.regexpSource); |
| let regexp = new RegExp(source, this.matchCase ? "" : "i"); |
| Object.defineProperty(this, "regexp", {value: regexp}); |
| - delete this.regexpSource; |
| + this.regexpSource = null; |
| return regexp; |
| }, |
| /** |
| * Content types the filter applies to, combination of values from |
| * RegExpFilter.typeMap |
| * @type {number} |
| */ |
| contentType: 0x7FFFFFFF, |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
