Launch loader directly without additional coroutine scope

mlykotom · mlykotom · commit 7c33946ce13c · 2024-02-08T15:06:55.000+01:00
Change-Id: Icbbdbcbcac1a6275857ebe998509f1e09109db7a
diff --git a/app/src/main/java/com/google/samples/apps/nowinandroid/util/ImageLoaderAsyncFactory.kt b/app/src/main/java/com/google/samples/apps/nowinandroid/util/ImageLoaderAsyncFactory.kt
@@ -44,9 +44,9 @@ class ImageLoaderAsyncFactory @Inject constructor(
     private lateinit var asyncNewImageLoader: Deferred<ImageLoader>
 
     init {
-        appScope.launch {
+        asyncNewImageLoader = appScope.async {
             // Initialize immediately, but need a Deferred for callers
-            asyncNewImageLoader = async { imageLoader.get() }
+            imageLoader.get()
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -44,9 +44,9 @@ class ImageLoaderAsyncFactory @Inject constructor(`
`44`	`44`	`private lateinit var asyncNewImageLoader: Deferred<ImageLoader>`
`45`	`45`
`46`	`46`	`init {`
`47`		`- appScope.launch {`
	`47`	`+ asyncNewImageLoader = appScope.async {`
`48`	`48`	`// Initialize immediately, but need a Deferred for callers`
`49`		`- asyncNewImageLoader = async { imageLoader.get() }`
	`49`	`+ imageLoader.get()`
`50`	`50`	`}`
`51`	`51`	`}`
`52`	`52`