Unified Diff: lib/iniParser.js
Issue 29934588:
Issue 7094 - Encapsulate management of subscription filters (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: Lazy initialize and clean up filter text lookup
Created Nov. 18, 2018, 2:20 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« lib/filterStorage.js ('K')
|
« lib/filterStorage.js ('k')
|
lib/subscriptionClasses.js » ('j')
|
lib/subscriptionClasses.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/iniParser.js |
| =================================================================== |
| --- a/lib/iniParser.js |
| +++ b/lib/iniParser.js |
| @@ -104,17 +104,17 @@ |
| if (this.subscriptions.length) |
| { |
| let currentSubscription = this.subscriptions[ |
| this.subscriptions.length - 1 |
| ]; |
| for (let text of this._curObj) |
| { |
| let filter = Filter.fromText(text); |
| - currentSubscription.filters.push(filter); |
| + currentSubscription.addFilter(filter); |
| filter.addSubscription(currentSubscription); |
| } |
| } |
| break; |
| } |
| } |
| if (line === null) |
« lib/filterStorage.js ('K')
|
« lib/filterStorage.js ('k')
|
lib/subscriptionClasses.js » ('j')
|
lib/subscriptionClasses.js » ('J')
