Clean up Lint warnings. (#1269)

coil-kt · May 12, 2022 · 1f1772f · 1f1772f
1 parent f5f9faf
commit 1f1772f
Show file tree

Hide file tree

Showing 7 changed files with 8 additions and 8 deletions.
diff --git a/coil-base/src/androidTest/java/coil/request/DisposableTest.kt b/coil-base/src/androidTest/java/coil/request/DisposableTest.kt
@@ -17,7 +17,6 @@ import coil.util.TestActivity
 import coil.util.activity
 import coil.util.requestManager
 import coil.util.runTestMain
-import kotlinx.coroutines.ExperimentalCoroutinesApi
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.first
 import org.junit.After
@@ -29,7 +28,6 @@ import kotlin.test.assertNotNull
 import kotlin.test.assertNull
 import kotlin.test.assertTrue
 
-@OptIn(ExperimentalCoroutinesApi::class)
 class DisposableTest {
 
     private lateinit var context: Context

diff --git a/coil-base/src/main/java/coil/drawable/CrossfadeDrawable.kt b/coil-base/src/main/java/coil/drawable/CrossfadeDrawable.kt
@@ -115,6 +115,7 @@ class CrossfadeDrawable @JvmOverloads constructor(
         maxAlpha = alpha
     }
 
+    @Deprecated("Deprecated in Java")
     @Suppress("DEPRECATION")
     override fun getOpacity(): Int {
         val start = start

diff --git a/coil-gif/src/main/java/coil/decode/ImageDecoderDecoder.kt b/coil-gif/src/main/java/coil/decode/ImageDecoderDecoder.kt
@@ -164,7 +164,6 @@ class ImageDecoderDecoder @JvmOverloads constructor(
         return ScaleDrawable(baseDrawable, options.scale)
     }
 
-    @RequiresApi(28)
     class Factory @JvmOverloads constructor(
         private val enforceMinimumFrameDelay: Boolean = true
     ) : Decoder.Factory {

diff --git a/coil-gif/src/main/java/coil/drawable/MovieDrawable.kt b/coil-gif/src/main/java/coil/drawable/MovieDrawable.kt
@@ -192,8 +192,11 @@ class MovieDrawable @JvmOverloads constructor(
         paint.alpha = alpha
     }
 
+    @Deprecated("Deprecated in Java")
+    @Suppress("DEPRECATION")
     override fun getOpacity(): Int {
-        return if (paint.alpha == 255 && (pixelOpacity == OPAQUE || (pixelOpacity == UNCHANGED && movie.isOpaque))) {
+        return if (paint.alpha == 255 &&
+            (pixelOpacity == OPAQUE || (pixelOpacity == UNCHANGED && movie.isOpaque))) {
             PixelFormat.OPAQUE
         } else {
             PixelFormat.TRANSLUCENT

diff --git a/coil-gif/src/main/java/coil/drawable/ScaleDrawable.kt b/coil-gif/src/main/java/coil/drawable/ScaleDrawable.kt
@@ -48,6 +48,7 @@ class ScaleDrawable @JvmOverloads constructor(
         child.alpha = alpha
     }
 
+    @Deprecated("Deprecated in Java")
     @Suppress("DEPRECATION")
     override fun getOpacity() = child.opacity
 

diff --git a/coil-singleton/src/test/java/coil/ImageLoaderFactoryTest.kt b/coil-singleton/src/test/java/coil/ImageLoaderFactoryTest.kt
@@ -3,7 +3,6 @@ package coil
 import android.app.Application
 import android.content.Context
 import androidx.test.core.app.ApplicationProvider
-import kotlinx.coroutines.ExperimentalCoroutinesApi
 import org.junit.After
 import org.junit.Before
 import org.junit.Test
@@ -15,7 +14,6 @@ import kotlin.test.assertFalse
 import kotlin.test.assertSame
 import kotlin.test.assertTrue
 
-@OptIn(ExperimentalCoroutinesApi::class)
 @RunWith(RobolectricTestRunner::class)
 class ImageLoaderFactoryTest {
 

diff --git a/docs/image_loaders.md b/docs/image_loaders.md
@@ -56,8 +56,8 @@ class FakeImageLoader(private val context: Context) : ImageLoader {
 
     override val defaults = DefaultRequestOptions()
     override val components = ComponentRegistry()
-    override val memoryCache get() = null
-    override val diskCache get() = null
+    override val memoryCache: MemoryCache? get() = null
+    override val diskCache: DiskCache? get() = null
 
     override fun enqueue(request: ImageRequest): Disposable {
         // Always call onStart before onSuccess.