diff --git a/services/Android.bp b/services/Android.bp index b5cae92bd436c..01893039e7729 100644 --- a/services/Android.bp +++ b/services/Android.bp @@ -21,6 +21,8 @@ java_defaults { "-Xep:TryFailThrowable:ERROR", "-Xep:HashtableContains:ERROR", "-Xep:FormatString:ERROR", + "-Xep:ArrayHashCode:ERROR", + "-Xep:SelfAssignment:ERROR", // NOTE: only enable to generate local patchfiles // "-XepPatchChecks:refaster:frameworks/base/errorprone/refaster/EfficientXml.java.refaster", // "-XepPatchLocation:/tmp/refaster/", diff --git a/services/autofill/java/com/android/server/autofill/AutofillManagerService.java b/services/autofill/java/com/android/server/autofill/AutofillManagerService.java index 051281c5dd08f..fc95cdd53f294 100644 --- a/services/autofill/java/com/android/server/autofill/AutofillManagerService.java +++ b/services/autofill/java/com/android/server/autofill/AutofillManagerService.java @@ -1614,8 +1614,8 @@ public final class AutofillManagerService @NonNull IBinder appCallback, @NonNull IResultReceiver receiver) throws RemoteException { final int userId = UserHandle.getCallingUserId(); - activityToken = Objects.requireNonNull(activityToken, "activityToken"); - appCallback = Objects.requireNonNull(appCallback, "appCallback"); + Objects.requireNonNull(activityToken, "activityToken"); + Objects.requireNonNull(appCallback, "appCallback"); boolean restored = false; synchronized (mLock) { diff --git a/services/people/java/com/android/server/people/data/EventIndex.java b/services/people/java/com/android/server/people/data/EventIndex.java index 6a13b0e6dd6ee..02afb8d1cc183 100644 --- a/services/people/java/com/android/server/people/data/EventIndex.java +++ b/services/people/java/com/android/server/people/data/EventIndex.java @@ -254,7 +254,7 @@ public class EventIndex { @Override public int hashCode() { - return Objects.hash(mLastUpdatedTime, mEventBitmaps); + return Objects.hash(mLastUpdatedTime, Arrays.hashCode(mEventBitmaps)); } synchronized void writeToProto(@NonNull ProtoOutputStream protoOutputStream) {