Merge pull request #41 from UrbanCompass/mobile-4335/observable-refle…

…ction-crash-bug-fix Fix for Kotlin Reflection Exception in Observable
UrbanCompass · Aug 7, 2019 · 3db4ae2 · 3db4ae2
2 parents 42e1fe1 + de1bf37
commit 3db4ae2
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 3 deletions.
diff --git a/snail-kotlin/src/main/java/com/compass/snail/Observable.kt b/snail-kotlin/src/main/java/com/compass/snail/Observable.kt
@@ -2,7 +2,6 @@
 
 package com.compass.snail
 
-import android.util.Log
 import com.compass.snail.disposer.Disposable
 import kotlinx.coroutines.CoroutineDispatcher
 import kotlinx.coroutines.GlobalScope
@@ -93,7 +92,6 @@ open class Observable<T> : IObservable<T> {
             subscriber.eventHandler(event)
         } catch (e: Exception) {
             e.printStackTrace()
-            Log.e("Snail Observable", "Removing subscriber $subscriber")
             subscribers.remove(subscriber)
         }
     }

diff --git a/snail-kotlin/src/test/java/com/compass/snail/LifecycleTests.kt b/snail-kotlin/src/test/java/com/compass/snail/LifecycleTests.kt
@@ -8,7 +8,6 @@ import androidx.lifecycle.LifecycleRegistry
 import com.compass.snail.disposer.disposeBy
 import com.compass.snail.disposer.onDestroy
 import com.compass.snail.disposer.onPause
-import com.compass.snail.disposer.onStart
 import com.compass.snail.disposer.onStop
 import org.junit.Assert
 import org.junit.Before