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

Delta Between Two Patch Sets: adblockplussbrowser/src/org/adblockplus/sbrowser/contentblocker/engine/SubscriptionInfo.java

Issue 29453722: Noissue - Lint adjustments and optimizations (Closed)
Left Patch Set: Created June 1, 2017, 9:30 p.m.
Right Patch Set: Adjusting HashSet initialization in Subscription Created July 19, 2017, 4:40 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
LEFTRIGHT
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-2016 Eyeo GmbH 3 * Copyright (C) 2006-2016 Eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 24 matching lines...) Expand all
35 ACCEPTABLE_ADS 35 ACCEPTABLE_ADS
36 } 36 }
37 37
38 private final Type type; 38 private final Type type;
39 private final String id; 39 private final String id;
40 private final String url; 40 private final String url;
41 private String title; 41 private String title;
42 private boolean enabled = false; 42 private boolean enabled = false;
43 private long lastUpdate; 43 private long lastUpdate;
44 44
45 SubscriptionInfo(final Type type, final String id, final String url, final Str ing title, final boolean enabled, final long lastUpdate) 45 private SubscriptionInfo(final Type type, final String id, final String url, f inal String title, final boolean enabled, final long lastUpdate)
jens 2017/06/02 10:50:01 I think this constructor can be private, as we onl
diegocarloslima 2017/06/02 21:03:42 Acknowledged.
46 { 46 {
47 this.type = type; 47 this.type = type;
48 this.id = id; 48 this.id = id;
49 this.url = url; 49 this.url = url;
50 this.title = title; 50 this.title = title;
51 this.enabled = enabled; 51 this.enabled = enabled;
52 this.lastUpdate = lastUpdate; 52 this.lastUpdate = lastUpdate;
53 } 53 }
54 54
55 public Type getType() 55 public Type getType()
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
148 subscription.isEnabled(), 148 subscription.isEnabled(),
149 subscription.getLastUpdateTimestamp()); 149 subscription.getLastUpdateTimestamp());
150 } 150 }
151 151
152 @Override 152 @Override
153 public int compareTo(@NonNull final SubscriptionInfo another) 153 public int compareTo(@NonNull final SubscriptionInfo another)
154 { 154 {
155 return this.getTitle().compareTo(another.getTitle()); 155 return this.getTitle().compareTo(another.getTitle());
156 } 156 }
157 } 157 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld