taler-ios

commit c2df690ff07f79aa595786730aa9edc91e2d5ffe
parent 09437d5cb7981ffee91926632556a4fb04a58e2d
Author: Marc Stibane <marc@taler.net>
Date:   Sat, 22 Aug 2026 10:01:29 +0200

AI: potential crash

Diffstat:
M
TalerWallet1/Views/OIM/ArrowHistoryView.swift
 | 
19
++++++++++++-------
M
TalerWallet1/Views/OIM/RiverHistoryView.swift
 | 
19
++++++++++++-------

2 files changed, 24 insertions(+), 14 deletions(-)
diff --git a/TalerWallet1/Views/OIM/ArrowHistoryView.swift b/TalerWallet1/Views/OIM/ArrowHistoryView.swift
@@ -163,13 +163,18 @@ struct ArrowTileView: View {
                         let height = geo.size.height
                         let width = geo.size.width
 //                        let _ = print("width = \(width), height = \(height)")
-                        let treeCount = Int.random(in: 23...57)
-                        ForEach(0..<treeCount, id: \.self) { treeIndex in
-                            let xOffset = Double.random(in: 3...width-treeSpace)
-                            let yOffset = Double.random(in: 3...height-treeSpace)
-                            let treeSize = Double.random(in: 10...20)
-                            tree.frame(width: treeSize, height: treeSize)
-                                .offset(x: xOffset, y: yOffset)
+                        // width/height can be 0 (or smaller than treeSpace) during the first
+                        // layout pass or when the tile is zoomed very small - skip the trees
+                        // then instead of handing Double.random an inverted range (crash).
+                        if width > treeSpace + 3 && height > treeSpace + 3 {
+                            let treeCount = Int.random(in: 23...57)
+                            ForEach(0..<treeCount, id: \.self) { treeIndex in
+                                let xOffset = Double.random(in: 3...width-treeSpace)
+                                let yOffset = Double.random(in: 3...height-treeSpace)
+                                let treeSize = Double.random(in: 10...20)
+                                tree.frame(width: treeSize, height: treeSize)
+                                    .offset(x: xOffset, y: yOffset)
+                            }
                         }
                     }
                 }
diff --git a/TalerWallet1/Views/OIM/RiverHistoryView.swift b/TalerWallet1/Views/OIM/RiverHistoryView.swift
@@ -171,13 +171,18 @@ struct RiverTileView: View {
                         let height = geo.size.height
                         let width = geo.size.width
 //                        let _ = print("width = \(width), height = \(height)")
-                        let treeCount = Int.random(in: 23...57)
-                        ForEach(0..<treeCount, id: \.self) { treeIndex in
-                            let xOffset = Double.random(in: 3...width-treeSpace)
-                            let yOffset = Double.random(in: 3...height-treeSpace)
-                            let treeSize = Double.random(in: 10...20)
-                            tree.frame(width: treeSize, height: treeSize)
-                                .offset(x: xOffset, y: yOffset)
+                        // width/height can be 0 (or smaller than treeSpace) during the first
+                        // layout pass or when the tile is zoomed very small - skip the trees
+                        // then instead of handing Double.random an inverted range (crash).
+                        if width > treeSpace + 3 && height > treeSpace + 3 {
+                            let treeCount = Int.random(in: 23...57)
+                            ForEach(0..<treeCount, id: \.self) { treeIndex in
+                                let xOffset = Double.random(in: 3...width-treeSpace)
+                                let yOffset = Double.random(in: 3...height-treeSpace)
+                                let treeSize = Double.random(in: 10...20)
+                                tree.frame(width: treeSize, height: treeSize)
+                                    .offset(x: xOffset, y: yOffset)
+                            }
                         }
                     }
                 }