Simplify tile consumption logic

p-lr · Jul 3, 2020 · 3884fbb · 3884fbb
1 parent 7c1e6bc
commit 3884fbb
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/mapview/src/main/java/com/peterlaurence/mapview/viewmodel/TileCanvasViewModel.kt b/mapview/src/main/java/com/peterlaurence/mapview/viewmodel/TileCanvasViewModel.kt
@@ -151,14 +151,10 @@ internal class TileCanvasViewModel(private val scope: CoroutineScope, tileSize:
      */
     private fun CoroutineScope.consumeTiles(tileChannel: ReceiveChannel<Tile>) = launch {
         for (tile in tileChannel) {
-            if (lastVisible.contains(tile)) {
-                if (!tilesToRender.contains(tile)) {
-                    tile.setPaint()
-                    tilesToRender.add(tile)
-                    idleDebounced.offer(Unit)
-                } else {
-                    tile.recycle()
-                }
+            if (lastVisible.contains(tile) && !tilesToRender.contains(tile)) {
+                tile.setPaint()
+                tilesToRender.add(tile)
+                idleDebounced.offer(Unit)
                 renderThrottled()
             } else {
                 tile.recycle()