diff options
author | cyBerta <cyberta@riseup.net> | 2018-01-11 14:59:56 +0100 |
---|---|---|
committer | cyBerta <cyberta@riseup.net> | 2018-01-11 14:59:56 +0100 |
commit | 1e94e6e1403d97e47119318bd43b173ef20658b1 (patch) | |
tree | 2f37b74f259915d1b04facd0d0f59856f112f8b8 /app/src/main/java/se/leap/bitmaskclient/Provider.java | |
parent | 0105cb76e1fc6b59d42dff3059d5bb9548dab372 (diff) |
#8773 code review fixes
Diffstat (limited to 'app/src/main/java/se/leap/bitmaskclient/Provider.java')
-rw-r--r-- | app/src/main/java/se/leap/bitmaskclient/Provider.java | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/app/src/main/java/se/leap/bitmaskclient/Provider.java b/app/src/main/java/se/leap/bitmaskclient/Provider.java index ae07bc25..60b1b93c 100644 --- a/app/src/main/java/se/leap/bitmaskclient/Provider.java +++ b/app/src/main/java/se/leap/bitmaskclient/Provider.java @@ -234,7 +234,7 @@ public final class Provider implements Parcelable { public boolean equals(Object o) { if (o instanceof Provider) { Provider p = (Provider) o; - return p.getMainUrl().getDomain().equals(getMainUrl().getDomain()); + return p.getDomain().equals(getDomain()); } else return false; } @@ -253,7 +253,7 @@ public final class Provider implements Parcelable { @Override public int hashCode() { - return getMainUrl().getDomain().hashCode(); + return getDomain().hashCode(); } @Override @@ -261,6 +261,7 @@ public final class Provider implements Parcelable { return new Gson().toJson(this); } + //TODO: write a test for marshalling! private Provider(Parcel in) { try { mainUrl.setUrl(new URL(in.readString())); |