Skip to content

Commit 135b6c1

Browse files
committed
Some cleaning up
1 parent d7037b5 commit 135b6c1

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

rxjava-contrib/rxjava-android/src/main/java/rx/operators/OperatorWeakBinding.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ private WeakSubscriber(Subscriber<? super T> source) {
5252

5353
@Override
5454
public void onCompleted() {
55-
Subscriber<? super T> sub = subscriberRef.get();
55+
final Subscriber<? super T> sub = subscriberRef.get();
5656
if (shouldForwardNotification(sub)) {
5757
sub.onCompleted();
5858
} else {
@@ -62,7 +62,7 @@ public void onCompleted() {
6262

6363
@Override
6464
public void onError(Throwable e) {
65-
Subscriber<? super T> sub = subscriberRef.get();
65+
final Subscriber<? super T> sub = subscriberRef.get();
6666
if (shouldForwardNotification(sub)) {
6767
sub.onError(e);
6868
} else {
@@ -72,7 +72,7 @@ public void onError(Throwable e) {
7272

7373
@Override
7474
public void onNext(T t) {
75-
Subscriber<? super T> sub = subscriberRef.get();
75+
final Subscriber<? super T> sub = subscriberRef.get();
7676
if (shouldForwardNotification(sub)) {
7777
sub.onNext(t);
7878
} else {
@@ -90,7 +90,8 @@ private void handleLostBinding(Subscriber<? super T> sub, String context) {
9090
Log.d(LOG_TAG, "subscriber gone; skipping " + context);
9191
} else {
9292
final R r = boundRef.get();
93-
if (r != null) { // the predicate failed to validate
93+
if (r != null) {
94+
// the predicate failed to validate
9495
Log.d(LOG_TAG, "bound component has become invalid; skipping " + context);
9596
} else {
9697
Log.d(LOG_TAG, "bound component gone; skipping " + context);

0 commit comments

Comments
 (0)