From 48ec7d90681c60dee50c0f4e46f18513a5c05166 Mon Sep 17 00:00:00 2001
From: ghm <ghm@google.com>
Date: Thu, 28 Mar 2024 11:28:37 -0700
Subject: [PATCH] Prepare RedundantSetterCall for becoming an ERROR.

PiperOrigin-RevId: 620003152
---
 .../google/errorprone/bugpatterns/RedundantSetterCall.java    | 4 ++--
 .../google/errorprone/scanner/BuiltInCheckerSuppliers.java    | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/core/src/main/java/com/google/errorprone/bugpatterns/RedundantSetterCall.java b/core/src/main/java/com/google/errorprone/bugpatterns/RedundantSetterCall.java
index 14f9dc05758..aea5199421a 100644
--- a/core/src/main/java/com/google/errorprone/bugpatterns/RedundantSetterCall.java
+++ b/core/src/main/java/com/google/errorprone/bugpatterns/RedundantSetterCall.java
@@ -19,7 +19,7 @@
 import static com.google.common.base.CaseFormat.UPPER_CAMEL;
 import static com.google.common.base.CaseFormat.UPPER_UNDERSCORE;
 import static com.google.common.collect.Streams.stream;
-import static com.google.errorprone.BugPattern.SeverityLevel.WARNING;
+import static com.google.errorprone.BugPattern.SeverityLevel.ERROR;
 import static com.google.errorprone.VisitorState.memoize;
 import static com.google.errorprone.matchers.Matchers.allOf;
 import static com.google.errorprone.matchers.Matchers.anyOf;
@@ -66,7 +66,7 @@
 /** A BugPattern; see the summary. */
 @BugPattern(
     summary = "A field was set twice in the same chained expression.",
-    severity = WARNING,
+    severity = ERROR,
     altNames = "ProtoRedundantSet",
     tags = StandardTags.FRAGILE_CODE)
 public final class RedundantSetterCall extends BugChecker implements MethodInvocationTreeMatcher {
diff --git a/core/src/main/java/com/google/errorprone/scanner/BuiltInCheckerSuppliers.java b/core/src/main/java/com/google/errorprone/scanner/BuiltInCheckerSuppliers.java
index c8dab81d18f..2506e23075c 100644
--- a/core/src/main/java/com/google/errorprone/scanner/BuiltInCheckerSuppliers.java
+++ b/core/src/main/java/com/google/errorprone/scanner/BuiltInCheckerSuppliers.java
@@ -794,6 +794,7 @@ public static ScannerSupplier warningChecks() {
           RandomCast.class,
           RandomModInteger.class,
           RectIntersectReturnValueIgnored.class,
+          RedundantSetterCall.class,
           RequiredModifiersChecker.class,
           RestrictedApiChecker.class,
           ReturnValueIgnored.class,
@@ -1032,7 +1033,6 @@ public static ScannerSupplier warningChecks() {
           ProtoTimestampGetSecondsGetNano.class,
           QualifierOrScopeOnInjectMethod.class,
           ReachabilityFenceUsage.class,
-          RedundantSetterCall.class,
           ReferenceEquality.class,
           RethrowReflectiveOperationExceptionAsLinkageError.class,
           ReturnAtTheEndOfVoidFunction.class,