diff options
author | Cole Faust <colefaust@google.com> | 2022-10-15 21:33:31 -0700 |
---|---|---|
committer | Cole Faust <colefaust@google.com> | 2022-10-15 21:33:31 -0700 |
commit | dcc6d81c40310bd07b911c3506d7ae096b2cb42d (patch) | |
tree | 7ecd1e8f7156bed41017897a983906f5ab482fe0 /rcs/rcsservice/src/com/android/service | |
parent | a1dfd835f7c99a46b1180805fa190b1c918caa79 (diff) | |
download | ims-dcc6d81c40310bd07b911c3506d7ae096b2cb42d.tar.gz |
Fix errorprone warnings that should be errors
This commit is part of a large scale change to fix errorprone
errors that have been downgraded to warnings in the android
source tree, so that they can be promoted to errors again.
The full list of changes include the following, but not all
will be present in any one individual commit:
BadAnnotationImplementation
BadShiftAmount
BanJNDI
BoxedPrimitiveEquality
ComparableType
ComplexBooleanConstant
CollectionToArraySafeParameter
ConditionalExpressionNumericPromotion
DangerousLiteralNull
DoubleBraceInitialization
DurationFrom
DurationTemporalUnit
EmptyTopLevelDeclaration
EqualsNull
EqualsReference
FormatString
FromTemporalAccessor
GetClassOnAnnotation
GetClassOnClass
HashtableContains
IdentityBinaryExpression
IdentityHashMapBoxing
InstantTemporalUnit
InvalidTimeZoneID
InvalidZoneId
IsInstanceIncompatibleType
JUnitParameterMethodNotFound
LockOnBoxedPrimitive
MathRoundIntLong
MislabeledAndroidString
MisusedDayOfYear
MissingSuperCall
MisusedWeekYear
ModifyingCollectionWithItself
NoCanIgnoreReturnValueOnClasses
NonRuntimeAnnotation
NullableOnContainingClass
NullTernary
OverridesJavaxInjectableMethod
ParcelableCreator
PeriodFrom
PreconditionsInvalidPlaceholder
ProtoBuilderReturnValueIgnored
ProtoFieldNullComparison
RandomModInteger
RectIntersectReturnValueIgnored
ReturnValueIgnored
SelfAssignment
SelfComparison
SelfEquals
SizeGreaterThanOrEqualsZero
StringBuilderInitWithChar
TreeToString
TryFailThrowable
UnnecessaryCheckNotNull
UnusedCollectionModifiedInPlace
XorPower
See https://errorprone.info/bugpatterns for more
information on the checks.
Bug: 253827323
Test: m RUN_ERROR_PRONE=true javac-check
Change-Id: Ia3faefdefc558f005de72bc59260eccbc121674e
Diffstat (limited to 'rcs/rcsservice/src/com/android/service')
7 files changed, 8 insertions, 13 deletions
diff --git a/rcs/rcsservice/src/com/android/service/ims/AlarmBroadcastReceiver.java b/rcs/rcsservice/src/com/android/service/ims/AlarmBroadcastReceiver.java index e06b4a4..9c06a5e 100644 --- a/rcs/rcsservice/src/com/android/service/ims/AlarmBroadcastReceiver.java +++ b/rcs/rcsservice/src/com/android/service/ims/AlarmBroadcastReceiver.java @@ -68,5 +68,4 @@ public class AlarmBroadcastReceiver extends BroadcastReceiver{ logger.debug("not interest in intent=" + intent); } } -}; - +} diff --git a/rcs/rcsservice/src/com/android/service/ims/StackListener.java b/rcs/rcsservice/src/com/android/service/ims/StackListener.java index 649a66b..a89f459 100644 --- a/rcs/rcsservice/src/com/android/service/ims/StackListener.java +++ b/rcs/rcsservice/src/com/android/service/ims/StackListener.java @@ -508,8 +508,8 @@ public class StackListener extends Handler{ public void capInfoReceived(String presentityURI, PresTupleInfo[] pTupleInfo) throws RemoteException { logger.debug("ListenerHandler : CapInfoReceived"); - if(presentityURI == null || presentityURI == null){ - logger.error("presentityURI=null or presentityURI=null"); + if(presentityURI == null) { + logger.error("presentityURI=null"); return; } diff --git a/rcs/rcsservice/src/com/android/service/ims/Task.java b/rcs/rcsservice/src/com/android/service/ims/Task.java index 61e85b4..ce4ef34 100644 --- a/rcs/rcsservice/src/com/android/service/ims/Task.java +++ b/rcs/rcsservice/src/com/android/service/ims/Task.java @@ -78,5 +78,4 @@ public class Task{ " mSipResponseCode=" + mSipResponseCode + " mSipReasonPhrase=" + mSipReasonPhrase; } -}; - +} diff --git a/rcs/rcsservice/src/com/android/service/ims/presence/PresenceAvailabilityTask.java b/rcs/rcsservice/src/com/android/service/ims/presence/PresenceAvailabilityTask.java index 08a5b32..6f53fa6 100644 --- a/rcs/rcsservice/src/com/android/service/ims/presence/PresenceAvailabilityTask.java +++ b/rcs/rcsservice/src/com/android/service/ims/presence/PresenceAvailabilityTask.java @@ -69,5 +69,4 @@ public class PresenceAvailabilityTask extends PresenceTask{ return super.toString() + " mNotifyTimeStamp=" + mNotifyTimeStamp; } -}; - +} diff --git a/rcs/rcsservice/src/com/android/service/ims/presence/PresenceCapabilityTask.java b/rcs/rcsservice/src/com/android/service/ims/presence/PresenceCapabilityTask.java index fb22b5e..f93b317 100644 --- a/rcs/rcsservice/src/com/android/service/ims/presence/PresenceCapabilityTask.java +++ b/rcs/rcsservice/src/com/android/service/ims/presence/PresenceCapabilityTask.java @@ -161,4 +161,4 @@ public class PresenceCapabilityTask extends PresenceTask{ TaskManager.getDefault().removeTask(mTaskId); } -}; +} diff --git a/rcs/rcsservice/src/com/android/service/ims/presence/PresencePublishTask.java b/rcs/rcsservice/src/com/android/service/ims/presence/PresencePublishTask.java index 24d1ada..49bd1e2 100644 --- a/rcs/rcsservice/src/com/android/service/ims/presence/PresencePublishTask.java +++ b/rcs/rcsservice/src/com/android/service/ims/presence/PresencePublishTask.java @@ -61,5 +61,4 @@ public class PresencePublishTask extends PresenceTask{ " mCreateTimestamp=" + mCreateTimestamp + " mRetryCount=" + mRetryCount; } -}; - +} diff --git a/rcs/rcsservice/src/com/android/service/ims/presence/PresenceTask.java b/rcs/rcsservice/src/com/android/service/ims/presence/PresenceTask.java index 5465e10..1f24507 100644 --- a/rcs/rcsservice/src/com/android/service/ims/presence/PresenceTask.java +++ b/rcs/rcsservice/src/com/android/service/ims/presence/PresenceTask.java @@ -55,5 +55,4 @@ public class PresenceTask extends Task { " mSipResponseCode=" + mSipResponseCode + " mSipReasonPhrase=" + mSipReasonPhrase; } -}; - +} |