From ada4189aea2ef994f4c1aa5f09ce3ec3e5c34e68 Mon Sep 17 00:00:00 2001 From: Arkadiusz Fal Date: Sat, 15 Nov 2025 15:44:29 +0100 Subject: [PATCH] Refactor dirty region handling in MPVOGLView Simplify the conditional logic for marking dirty regions by using optional binding instead of force unwrapping. --- Shared/Player/MPV/MPVOGLView.swift | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/Shared/Player/MPV/MPVOGLView.swift b/Shared/Player/MPV/MPVOGLView.swift index bf71d7ec..b83887f1 100644 --- a/Shared/Player/MPV/MPVOGLView.swift +++ b/Shared/Player/MPV/MPVOGLView.swift @@ -104,11 +104,10 @@ final class MPVOGLView: GLKView { // Function to set a dirty region when a part of the screen changes func markRegionAsDirty(_ region: CGRect) { - if dirtyRegion == nil { - dirtyRegion = region + if var dirtyRegion { + self.dirtyRegion = dirtyRegion.union(region) } else { - // Expand the dirty region to include the new region - dirtyRegion = dirtyRegion!.union(region) + dirtyRegion = region } }