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

Unified Diff: adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/DefaultSubscriptionInfo.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/DefaultSubscriptionInfo.java
===================================================================
--- a/adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/DefaultSubscriptionInfo.java
+++ b/adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/DefaultSubscriptionInfo.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;
+
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
public final class DefaultSubscriptionInfo implements Comparable<DefaultSubscriptionInfo>
{
private final static String KEY_TITLE = "title";
private final static String KEY_URL = "url";
@@ -79,17 +81,17 @@ public final class DefaultSubscriptionIn
}
public boolean isComplete()
{
return this.getValue(KEY_COMPLETE).equals("true");
}
@Override
- public int compareTo(final DefaultSubscriptionInfo o)
+ public int compareTo(@NonNull final DefaultSubscriptionInfo o)
{
final int cmp = this.getTitle().compareTo(o.getTitle());
if (cmp != 0)
{
return cmp;
}
return this.getUrl().compareTo(o.getUrl());
}

Powered by Google App Engine
This is Rietveld