fix: long operation fails on orientation changes in concurrency demo kaushikgopal#2

Kaushik Gopal · Kaushik Gopal · commit f0a73ece56c8 · 2014-09-11T08:06:14.000-07:00
diff --git a/app/src/main/java/com/morihacky/android/rxjava/ConcurrencyWithSchedulersDemoFragment.java b/app/src/main/java/com/morihacky/android/rxjava/ConcurrencyWithSchedulersDemoFragment.java
@@ -107,7 +107,9 @@ public void onNext(Boolean aBoolean) {
     @Override
     public void onDestroy() {
         super.onDestroy();
-        _subscription.unsubscribe();
+        if (_subscription != null) {
+          _subscription.unsubscribe();
+        }
     }
 
     private void _log(String logMsg) {
@@ -142,7 +144,7 @@ private void _doSomeLongOperation_thatBlocksCurrentThread() {
         try {
             Thread.sleep(3000);
         } catch (InterruptedException e) {
-            e.printStackTrace();
+          Timber.d("Operation was interrupted");
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,9 @@ public void onNext(Boolean aBoolean) {`
`107`	`107`	`@Override`
`108`	`108`	`public void onDestroy() {`
`109`	`109`	`super.onDestroy();`
`110`		`- _subscription.unsubscribe();`
	`110`	`+ if (_subscription != null) {`
	`111`	`+ _subscription.unsubscribe();`
	`112`	`+ }`
`111`	`113`	`}`
`112`	`114`
`113`	`115`	`private void _log(String logMsg) {`
`@@ -142,7 +144,7 @@ private void _doSomeLongOperation_thatBlocksCurrentThread() {`
`142`	`144`	`try {`
`143`	`145`	`Thread.sleep(3000);`
`144`	`146`	`} catch (InterruptedException e) {`
`145`		`- e.printStackTrace();`
	`147`	`+ Timber.d("Operation was interrupted");`
`146`	`148`	`}`
`147`	`149`	`}`
`148`	`150`