Unified Diff: lib/filterClasses.js
Issue 29852555:
Issue 6559 - Convert subscriptions property of Filter to a set (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set:
Created Aug. 10, 2018, 2:12 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
lib/filterListener.js » ('j')
|
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 |
| @@ -35,31 +35,31 @@ |
| * Abstract base class for filters |
| * |
| * @param {string} text string representation of the filter |
| * @constructor |
| */ |
| function Filter(text) |
| { |
| this.text = text; |
| - this.subscriptions = []; |
| + this.subscriptions = new Set(); |
| } |
| exports.Filter = Filter; |
| Filter.prototype = |
| { |
| /** |
| * String representation of the filter |
| * @type {string} |
| */ |
| text: null, |
| /** |
| * Filter subscriptions the filter belongs to |
| - * @type {Subscription[]} |
| + * @type {Set.<Subscription>} |
| */ |
| subscriptions: null, |
| /** |
| * Filter type as a string, e.g. "blocking". |
| * @type {string} |
| */ |
| get type() |
« no previous file with comments
|
« no previous file
|
lib/filterListener.js » ('j')
|
no next file with comments »
