diff --git a/bugpatterns.md b/bugpatterns.md index d32f0a57694..8f7cba1cc55 100644 --- a/bugpatterns.md +++ b/bugpatterns.md @@ -590,6 +590,9 @@ __[AssistedInjectAndInjectOnSameConstructor](bugpattern/AssistedInjectAndInjectO __[AttemptedNegativeZero](bugpattern/AttemptedNegativeZero)__
-0 is the same as 0. For the floating-point negative zero, use -0.0. +__[AutoValueBoxedValues](bugpattern/AutoValueBoxedValues)__
+AutoValue instances should not usually contain boxed types that are not Nullable. We recommend removing the unnecessary boxing. + __[AutoValueFinalMethods](bugpattern/AutoValueFinalMethods)__
Make toString(), hashCode() and equals() final in AutoValue classes, so it is clear to readers that AutoValue is not overriding them @@ -1440,9 +1443,6 @@ Assertions may be disabled at runtime and do not guarantee that execution will h __[AssistedInjectAndInjectOnConstructors](bugpattern/AssistedInjectAndInjectOnConstructors)__
@AssistedInject and @Inject should not be used on different constructors in the same class. -__[AutoValueBoxedValues](bugpattern/AutoValueBoxedValues)__
-AutoValue instances should not usually contain boxed types that are not Nullable. We recommend removing the unnecessary boxing. - __[AvoidObjectArrays](bugpattern/AvoidObjectArrays)__
Object arrays are inferior to collections in almost every way. Prefer immutable collections (e.g., ImmutableSet, ImmutableList, etc.) over an object array whenever possible.