Log critical failures

Author: afarber

openmapview/src/main/kotlin/de/afarber/openmapview/DiskTileCache.kt

@@ -10,6 +10,7 @@ package de.afarber.openmapview
 import android.content.Context
 import android.graphics.Bitmap
 import android.graphics.BitmapFactory
+import android.util.Log
 import com.jakewharton.disklrucache.DiskLruCache
 import java.io.File
 import java.io.IOException
@@ -28,6 +29,7 @@ class DiskTileCache(
     private val diskCache: DiskLruCache?
 
     companion object {
+        private val TAG = DiskTileCache::class.java.simpleName
         private const val DISK_CACHE_VERSION = 1
         private const val VALUE_COUNT = 1
     }
@@ -38,6 +40,7 @@ class DiskTileCache(
             try {
                 DiskLruCache.open(cacheDir, DISK_CACHE_VERSION, VALUE_COUNT, maxSize)
             } catch (e: IOException) {
+                Log.e(TAG, "Failed to initialize disk cache at $cacheDir", e)
                 null
             }
     }
@@ -50,6 +53,7 @@ class DiskTileCache(
                 BitmapFactory.decodeStream(inputStream)
             }
         } catch (e: IOException) {
+            Log.w(TAG, "Failed to read tile $key from disk cache", e)
             null
         }
     }
@@ -66,27 +70,28 @@ class DiskTileCache(
                     bitmap.compress(Bitmap.CompressFormat.PNG, 100, outputStream)
                     editor.commit()
                 } catch (e: IOException) {
+                    Log.w(TAG, "Failed to write tile $key to disk cache", e)
                     editor.abort()
                 }
             }
         } catch (e: IOException) {
-            // Ignore disk cache errors
+            Log.w(TAG, "Failed to edit tile $key in disk cache", e)
         }
     }
 
     fun clear() {
         try {
             diskCache?.delete()
         } catch (e: IOException) {
-            // Ignore
+            Log.w(TAG, "Failed to clear disk cache", e)
         }
     }
 
     fun close() {
         try {
             diskCache?.close()
         } catch (e: IOException) {
-            // Ignore
+            Log.w(TAG, "Failed to close disk cache", e)
         }
     }
 

openmapview/src/main/kotlin/de/afarber/openmapview/TileCache.kt

@@ -9,6 +9,7 @@ package de.afarber.openmapview
 
 import android.content.Context
 import android.graphics.Bitmap
+import android.util.Log
 import android.util.LruCache
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
@@ -24,6 +25,10 @@ import kotlinx.coroutines.launch
 class TileCache(
     context: Context? = null,
 ) {
+    companion object {
+        private val TAG = TileCache::class.java.simpleName
+    }
+
     // Use 1/8 of available heap memory for tile cache
     private val maxMemory = (Runtime.getRuntime().maxMemory() / 1024).toInt()
     private val cacheSize = maxMemory / 8
@@ -50,7 +55,7 @@ class TileCache(
                         try {
                             diskCache?.put(key, oldValue)
                         } catch (e: Exception) {
-                            // Ignore errors when disk cache is closed
+                            Log.w(TAG, "Failed to write evicted tile to disk cache", e)
                         }
                     }
                 }
@@ -87,7 +92,7 @@ class TileCache(
         try {
             diskCache?.clear()
         } catch (e: Exception) {
-            // Ignore errors during clear
+            Log.w(TAG, "Failed to clear disk cache", e)
         }
     }
 
@@ -96,7 +101,7 @@ class TileCache(
             diskCache?.close()
             diskCache = null
         } catch (e: Exception) {
-            // Ignore errors during close
+            Log.w(TAG, "Failed to close disk cache", e)
         }
     }
 }

openmapview/src/main/kotlin/de/afarber/openmapview/TileDownloader.kt

@@ -9,13 +9,18 @@ package de.afarber.openmapview
 
 import android.graphics.Bitmap
 import android.graphics.BitmapFactory
+import android.util.Log
 import io.ktor.client.HttpClient
 import io.ktor.client.engine.android.Android
 import io.ktor.client.request.get
 import io.ktor.client.request.header
 import io.ktor.client.statement.readBytes
 
 class TileDownloader {
+    companion object {
+        private val TAG = TileDownloader::class.java.simpleName
+    }
+
     private val client =
         HttpClient(Android) {
             engine {
@@ -41,6 +46,7 @@ class TileDownloader {
                 }
             BitmapFactory.decodeByteArray(bytes, 0, bytes.size, options)
         } catch (e: Exception) {
+            Log.w(TAG, "Failed to download tile from $url", e)
             null
         }