Make toString(), hashCode(), equals() and friends final in AutoValue classes

Developers usually don't look at the classes that AutoValue generates,
but at the hand-written classes. If the hand-written classes implement
toString(), hashCode(), equals() etc, AutoValue doesn't override them.
To make this more clear to readers of the hand-written classes
ErrorProne suggests to make these methods final.

See https://errorprone.info/bugpattern/AutoValueFinalMethods

Enable this check at ERROR severity to prevent future occurrences.

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: I972e8e8f9339b811c78157abef771b8c32be079c

java/com/google/gerrit/server/StarredChangesUtil.java

@@ -106,7 +106,7 @@ public class StarredChangesUtil {
     public abstract String label();
 
     @Override
-    public String toString() {
+    public final String toString() {
       return accountId() + SEPARATOR + label();
     }
   }

java/com/google/gerrit/server/account/ProjectWatches.java

@@ -263,7 +263,7 @@ public class ProjectWatches {
     public abstract ImmutableSet<NotifyType> notifyTypes();
 
     @Override
-    public String toString() {
+    public final String toString() {
       List<NotifyType> notifyTypes = new ArrayList<>(notifyTypes());
       StringBuilder notifyValue = new StringBuilder();
       notifyValue.append(firstNonNull(filter(), FILTER_ALL)).append(" [");

java/com/google/gerrit/server/account/externalids/ExternalId.java

@@ -196,7 +196,7 @@ public abstract class ExternalId implements Serializable {
     }
 
     @Override
-    public String toString() {
+    public final String toString() {
       return get();
     }
 
@@ -445,7 +445,7 @@ public abstract class ExternalId implements Serializable {
    * that was loaded from Git can be equal with an external ID that was created from code.
    */
   @Override
-  public boolean equals(Object obj) {
+  public final boolean equals(Object obj) {
     if (!(obj instanceof ExternalId)) {
       return false;
     }
@@ -457,7 +457,7 @@ public abstract class ExternalId implements Serializable {
   }
 
   @Override
-  public int hashCode() {
+  public final int hashCode() {
     return Objects.hash(key(), accountId(), email(), password());
   }
 
@@ -475,7 +475,7 @@ public abstract class ExternalId implements Serializable {
    * </pre>
    */
   @Override
-  public String toString() {
+  public final String toString() {
     Config c = new Config();
     writeToConfig(c);
     return c.toText();

java/com/google/gerrit/server/change/ChangeTriplet.java

@@ -65,7 +65,7 @@ public abstract class ChangeTriplet {
   public abstract Change.Key id();
 
   @Override
-  public String toString() {
+  public final String toString() {
     return format(branch(), id());
   }
 }

java/com/google/gerrit/server/config/ConfigKey.java

@@ -34,7 +34,7 @@ public abstract class ConfigKey {
   }
 
   @Override
-  public String toString() {
+  public final String toString() {
     StringBuilder sb = new StringBuilder();
     sb.append(section()).append(".");
     if (subsection() != null) {

java/com/google/gerrit/server/config/ScheduleConfig.java

@@ -194,7 +194,7 @@ public abstract class ScheduleConfig {
   }
 
   @Override
-  public String toString() {
+  public final String toString() {
     StringBuilder b = new StringBuilder();
     b.append(formatValue(keyInterval()));
     b.append(", ");

java/com/google/gerrit/server/notedb/NoteDbChangeState.java

@@ -143,7 +143,7 @@ public class NoteDbChangeState {
     abstract ImmutableMap<Account.Id, ObjectId> draftIds();
 
     @Override
-    public String toString() {
+    public final String toString() {
       return appendTo(new StringBuilder()).toString();
     }
 

java/com/google/gerrit/server/permissions/SectionSortCache.java

@@ -141,7 +141,7 @@ public class SectionSortCache {
     }
 
     @Override
-    public int hashCode() {
+    public final int hashCode() {
       return cachedHashCode();
     }
   }

java/com/google/gerrit/server/restapi/change/RelatedChangesSorter.java

@@ -267,12 +267,12 @@ class RelatedChangesSorter {
     }
 
     @Override
-    public int hashCode() {
+    public final int hashCode() {
       return Objects.hash(patchSet().getId(), commit());
     }
 
     @Override
-    public boolean equals(Object obj) {
+    public final boolean equals(Object obj) {
       if (!(obj instanceof PatchSetData)) {
         return false;
       }

java/com/google/gerrit/server/schema/GroupBundle.java

@@ -694,13 +694,13 @@ abstract class GroupBundle {
   }
 
   @Override
-  public int hashCode() {
+  public final int hashCode() {
     throw new UnsupportedOperationException(
         "hashCode is not supported because equals is not supported");
   }
 
   @Override
-  public boolean equals(Object o) {
+  public final boolean equals(Object o) {
     throw new UnsupportedOperationException("Use GroupBundle.compare(a, b) instead of equals");
   }
 

java/com/google/gerrit/server/util/LabelVote.java

@@ -85,7 +85,7 @@ public abstract class LabelVote {
   }
 
   @Override
-  public String toString() {
+  public final String toString() {
     return format();
   }
 }

tools/BUILD

@@ -26,6 +26,7 @@ java_package_configuration(
     javacopts = [
         "-XepDisableWarningsInGeneratedCode",
         "-Xep:AmbiguousMethodReference:WARN",
+        "-Xep:AutoValueFinalMethods:ERROR",
         "-Xep:BadAnnotationImplementation:WARN",
         "-Xep:BadComparable:WARN",
         "-Xep:BoxedPrimitiveConstructor:ERROR",