Skip to content

Commit 6121baf

Browse files
committed
Merge pull request ReactiveX#3369 from KevinTCoughlin/remove-unnecessary-unboxing
Lint fixes for unnecessary unboxing
2 parents 84622bb + ab02902 commit 6121baf

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

src/main/java/rx/internal/operators/OnSubscribeRedo.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ public Observable<? extends Notification<?>> call(Observable<? extends Notificat
111111
@Override
112112
public Notification<Integer> call(Notification<Integer> n, Notification<?> term) {
113113
final int value = n.getValue();
114-
if (predicate.call(value, term.getThrowable()).booleanValue())
114+
if (predicate.call(value, term.getThrowable()))
115115
return Notification.createOnNext(value + 1);
116116
else
117117
return (Notification<Integer>) term;

src/main/java/rx/internal/operators/OperatorReplay.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -805,7 +805,7 @@ public void replay(InnerProducer<T> output) {
805805
int sourceIndex = size;
806806

807807
Integer destIndexObject = output.index();
808-
int destIndex = destIndexObject != null ? destIndexObject.intValue() : 0;
808+
int destIndex = destIndexObject != null ? destIndexObject : 0;
809809

810810
long r = output.get();
811811
long r0 = r;

src/main/java/rx/schedulers/TestScheduler.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,9 +57,9 @@ private static class CompareActionsByTime implements Comparator<TimedAction> {
5757
@Override
5858
public int compare(TimedAction action1, TimedAction action2) {
5959
if (action1.time == action2.time) {
60-
return Long.valueOf(action1.count).compareTo(Long.valueOf(action2.count));
60+
return action1.count < action2.count ? -1 : ((action1.count > action2.count) ? 1 : 0);
6161
} else {
62-
return Long.valueOf(action1.time).compareTo(Long.valueOf(action2.time));
62+
return action1.time < action2.time ? -1 : ((action1.time > action2.time) ? 1 : 0);
6363
}
6464
}
6565
}

0 commit comments

Comments
 (0)