Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

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.
Jump to:
View side-by-side diff with in-line comments
Download patch
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());
}
}

Powered by Google App Engine
This is Rietveld