fix: long operation fails on orientation changes in concurrency demo #2 · dev-tim/Android-RxJava@f0a73ec · GitHub

dev-tim / Android-RxJava Public

forked from kaushikgopal/RxJava-Android-Samples

• Notifications You must be signed in to change notification settings
• Fork 0
• Star 0

• Code
• Pull requests
• Actions
• Projects
• Wiki
• Security and quality
• Insights

Additional navigation options

Commit f0a73ec

Browse filesBrowse the repository at this point in the historyBrowse files

Kaushik Gopal

committed

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

1 parent 52bc3be commit f0a73ec
Copy full SHA for f0a73ec

1 file changed

+4-2Lines changed: 4 additions & 2 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• app/src/main/java/com/morihacky/android/rxjava
  • ConcurrencyWithSchedulersDemoFragment.java

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎app/src/main/java/com/morihacky/android/rxjava/ConcurrencyWithSchedulersDemoFragment.java‎

Copy file name to clipboardExpand all lines: app/src/main/java/com/morihacky/android/rxjava/ConcurrencyWithSchedulersDemoFragment.java

+4-2Lines changed: 4 additions & 2 deletions

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

0 commit comments

Comments

0 (0)