Merge pull request #1212 from MariusVolkhart/imagePipelineConfig

[IMPROVEMENT] Change ImagePipelineConfig initialization in Dagger graph

Merge pull request #1212 from MariusVolkhart/imagePipelineConfig
[IMPROVEMENT] Change ImagePipelineConfig initialization in Dagger graph
c176e9fe · Filipe de Lima Brito · GitHub · 607378a7 · 186f131e · c176e9fe
Unverified Commit c176e9fe authored Apr 30, 2018 by Filipe de Lima Brito Committed by GitHub Apr 30, 2018
Show whitespace changes
Inline Side-by-side

Showing with 1 addition and 3 deletions

AppModule.kt .../main/java/chat/rocket/android/dagger/module/AppModule.kt +1 -3

No files found.
--- a/app/src/main/java/chat/rocket/android/dagger/module/AppModule.kt
+++ b/app/src/main/java/chat/rocket/android/dagger/module/AppModule.kt
@@ -38,7 +38,6 @@ import chat.rocket.core.internal.ReactionsAdapter
 import com.facebook.drawee.backends.pipeline.DraweeConfig
 import com.facebook.imagepipeline.backends.okhttp3.OkHttpImagePipelineConfigFactory
 import com.facebook.imagepipeline.core.ImagePipelineConfig
-import com.facebook.imagepipeline.listener.RequestListener
 import com.facebook.imagepipeline.listener.RequestLoggingListener
 import com.squareup.moshi.Moshi
 import dagger.Module
@@ -138,8 +137,7 @@ class AppModule {
    @Provides
    @Singleton
    fun provideImagePipelineConfig(context: Context, @ForFresco okHttpClient: OkHttpClient): ImagePipelineConfig {
-        val listeners = HashSet<RequestListener>()
+        val listeners = setOf(RequestLoggingListener())
-        listeners.add(RequestLoggingListener())
        return OkHttpImagePipelineConfigFactory.newBuilder(context, okHttpClient)
            .setRequestListeners(listeners)