diff options
4 files changed, 12 insertions, 12 deletions
diff --git a/java/com/google/gerrit/entities/Comment.java b/java/com/google/gerrit/entities/Comment.java index 37b8620d82..92bcaf66aa 100644 --- a/java/com/google/gerrit/entities/Comment.java +++ b/java/com/google/gerrit/entities/Comment.java @@ -88,7 +88,7 @@ public abstract class Comment { Key k = (Key) o; return Objects.equals(uuid, k.uuid) && Objects.equals(filename, k.filename) - && Objects.equals(patchSetId, k.patchSetId); + && patchSetId == k.patchSetId; } return false; } @@ -113,7 +113,7 @@ public abstract class Comment { @Override public boolean equals(Object o) { if (o instanceof Identity) { - return Objects.equals(id, ((Identity) o).id); + return id == ((Identity) o).id; } return false; } @@ -180,10 +180,10 @@ public abstract class Comment { public boolean equals(Object o) { if (o instanceof Range) { Range r = (Range) o; - return Objects.equals(startLine, r.startLine) - && Objects.equals(startChar, r.startChar) - && Objects.equals(endLine, r.endLine) - && Objects.equals(endChar, r.endChar); + return startLine == r.startLine + && startChar == r.startChar + && endLine == r.endLine + && endChar == r.endChar; } return false; } diff --git a/java/com/google/gerrit/entities/EmailHeader.java b/java/com/google/gerrit/entities/EmailHeader.java index bf5a644a16..d2be65e781 100644 --- a/java/com/google/gerrit/entities/EmailHeader.java +++ b/java/com/google/gerrit/entities/EmailHeader.java @@ -157,7 +157,7 @@ public abstract class EmailHeader { @Override public boolean equals(Object o) { - return (o instanceof Date) && Objects.equals(value, ((Date) o).value); + return (o instanceof Date) && value.getTime() == ((Date) o).value.getTime(); } @Override diff --git a/java/com/google/gerrit/extensions/client/Comment.java b/java/com/google/gerrit/extensions/client/Comment.java index 634992e9cb..5cd13dbf3c 100644 --- a/java/com/google/gerrit/extensions/client/Comment.java +++ b/java/com/google/gerrit/extensions/client/Comment.java @@ -70,10 +70,10 @@ public abstract class Comment { public boolean equals(Object o) { if (o instanceof Range) { Range r = (Range) o; - return Objects.equals(startLine, r.startLine) - && Objects.equals(startCharacter, r.startCharacter) - && Objects.equals(endLine, r.endLine) - && Objects.equals(endCharacter, r.endCharacter); + return startLine == r.startLine + && startCharacter == r.startCharacter + && endLine == r.endLine + && endCharacter == r.endCharacter; } return false; } diff --git a/java/com/google/gerrit/server/account/externalids/ExternalId.java b/java/com/google/gerrit/server/account/externalids/ExternalId.java index 0a23a10f00..0a51171a3a 100644 --- a/java/com/google/gerrit/server/account/externalids/ExternalId.java +++ b/java/com/google/gerrit/server/account/externalids/ExternalId.java @@ -310,7 +310,7 @@ public abstract class ExternalId implements Serializable { ExternalId o = (ExternalId) obj; return Objects.equals(key(), o.key()) && Objects.equals(accountId(), o.accountId()) - && Objects.equals(isCaseInsensitive(), o.isCaseInsensitive()) + && isCaseInsensitive() == o.isCaseInsensitive() && Objects.equals(email(), o.email()) && Objects.equals(password(), o.password()); } |