Final fields

2 files changed, 4 insertions, 4 deletions

diff --git a/library/src/main/java/com/davemorrissey/labs/subscaleview/decoder/CompatDecoderFactory.java b/library/src/main/java/com/davemorrissey/labs/subscaleview/decoder/CompatDecoderFactory.java
index d4a54ce..870489f 100644
--- a/library/src/main/java/com/davemorrissey/labs/subscaleview/decoder/CompatDecoderFactory.java
+++ b/library/src/main/java/com/davemorrissey/labs/subscaleview/decoder/CompatDecoderFactory.java
@@ -13,8 +13,8 @@ import java.lang.reflect.InvocationTargetException;
 @SuppressWarnings("WeakerAccess")
 public class CompatDecoderFactory<T> implements DecoderFactory<T> {
 
-    private Class<? extends T> clazz;
-    private Bitmap.Config bitmapConfig;
+    private final Class<? extends T> clazz;
+    private final Bitmap.Config bitmapConfig;
 
     /**
      * Construct a factory for the given class. This must have a default constructor.
diff --git a/library/src/main/java/com/davemorrissey/labs/subscaleview/decoder/SkiaPooledImageRegionDecoder.java b/library/src/main/java/com/davemorrissey/labs/subscaleview/decoder/SkiaPooledImageRegionDecoder.java
index 0dea670..80e415b 100644
--- a/library/src/main/java/com/davemorrissey/labs/subscaleview/decoder/SkiaPooledImageRegionDecoder.java
+++ b/library/src/main/java/com/davemorrissey/labs/subscaleview/decoder/SkiaPooledImageRegionDecoder.java
@@ -72,7 +72,7 @@ public class SkiaPooledImageRegionDecoder implements ImageRegionDecoder {
     private Uri uri;
 
     private long fileLength = Long.MAX_VALUE;
-    private Point imageDimensions = new Point(0, 0);
+    private final Point imageDimensions = new Point(0, 0);
     private final AtomicBoolean lazyInited = new AtomicBoolean(false);
 
     @Keep
@@ -339,7 +339,7 @@ public class SkiaPooledImageRegionDecoder implements ImageRegionDecoder {
      */
     private static class DecoderPool {
         private final Semaphore available = new Semaphore(0, true);
-        private Map<BitmapRegionDecoder, Boolean> decoders = new ConcurrentHashMap<>();
+        private final Map<BitmapRegionDecoder, Boolean> decoders = new ConcurrentHashMap<>();
 
         /**
          * Returns false if there is at least one decoder in the pool.