Unified Diff: adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/SubscriptionInfo.java
Issue 29453722:
Noissue - Lint adjustments and optimizations (Closed)
Patch Set: Adjustments based on review comments
Created June 13, 2017, 9:23 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/Subscription.java ('K')
|
« adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/Subscription.java ('k')
|
adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/Subscriptions.java » ('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: adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/SubscriptionInfo.java |
| =================================================================== |
| --- a/adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/SubscriptionInfo.java |
| +++ b/adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/SubscriptionInfo.java |
| @@ -12,16 +12,18 @@ |
| * GNU General Public License for more details. |
| * |
| * You should have received a copy of the GNU General Public License |
| * along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>. |
| */ |
| package org.adblockplus.sbrowser.contentblocker.engine; |
| +import android.support.annotation.NonNull; |
| + |
| /** |
| * Class for querying subscription information from the engine. |
| */ |
| public class SubscriptionInfo implements Comparable<SubscriptionInfo> |
| { |
| public enum Type |
| { |
| ADS, |
| @@ -35,17 +37,17 @@ public class SubscriptionInfo implements |
| private final Type type; |
| private final String id; |
| private final String url; |
| private String title; |
| private boolean enabled = false; |
| private long lastUpdate; |
| - SubscriptionInfo(final Type type, final String id, final String url, final String title, final boolean enabled, final long lastUpdate) |
| + private SubscriptionInfo(final Type type, final String id, final String url, final String title, final boolean enabled, final long lastUpdate) |
| { |
| this.type = type; |
| this.id = id; |
| this.url = url; |
| this.title = title; |
| this.enabled = enabled; |
| this.lastUpdate = lastUpdate; |
| } |
| @@ -143,13 +145,13 @@ public class SubscriptionInfo implements |
| subscription.getId(), |
| url, |
| title, |
| subscription.isEnabled(), |
| subscription.getLastUpdateTimestamp()); |
| } |
| @Override |
| - public int compareTo(final SubscriptionInfo another) |
| + public int compareTo(@NonNull final SubscriptionInfo another) |
| { |
| return this.getTitle().compareTo(another.getTitle()); |
| } |
| } |
« adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/Subscription.java ('K')
|
« adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/Subscription.java ('k')
|
adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/Subscriptions.java » ('j')
|
no next file with comments »
