From 661312a10515a7fb72bf3a0edbe42ecdf97850f9 Mon Sep 17 00:00:00 2001
From: Brian Osborn <bosborn@gmail.com>
Date: Thu, 24 Aug 2023 09:49:56 -0600
Subject: [PATCH] geopackage-ios 8.0.3 destroy and close calls for memory

---
 Mage/BaseMapOverlay.swift                           |  1 +
 Mage/GeoPackage/GeoPackage.m                        | 10 ++++++++++
 Mage/Map/Cache/GeoPackageFeatureTableCacheOverlay.m |  1 +
 Mage/Map/MapObservationManager.m                    |  1 +
 Mage/Mixins/FilteredObservationsMap.swift           |  1 +
 Mage/Mixins/SFGeometryMap.swift                     |  1 +
 6 files changed, 15 insertions(+)

diff --git a/Mage/BaseMapOverlay.swift b/Mage/BaseMapOverlay.swift
index 50a84e89..e9e88a7c 100644
--- a/Mage/BaseMapOverlay.swift
+++ b/Mage/BaseMapOverlay.swift
@@ -18,6 +18,7 @@ import geopackage_ios
     @objc public var darkTheme = false
     
     @objc public func cleanup() {
+        super.close()
         featureTiles = nil
     }
     
diff --git a/Mage/GeoPackage/GeoPackage.m b/Mage/GeoPackage/GeoPackage.m
index be20f38a..f181361d 100644
--- a/Mage/GeoPackage/GeoPackage.m
+++ b/Mage/GeoPackage/GeoPackage.m
@@ -246,6 +246,7 @@ -(void) addGeoPackageCacheOverlay: (NSMutableDictionary<NSString *, CacheOverlay
 
                 SFPGeometryTransform *transform = [[SFPGeometryTransform alloc] initWithFromProjection:projection andToEpsg:PROJ_EPSG_WORLD_GEODETIC_SYSTEM];
                 GPKGBoundingBox * boundingBox = [contentsBoundingBox transform:transform];
+                [transform destroy];
                 boundingBox = [GPKGTileBoundingBoxUtils boundWgs84BoundingBoxWithWebMercatorLimits:boundingBox];
                 
                 if(self.addedCacheBoundingBox == nil){
@@ -281,6 +282,9 @@ -(void) addGeoPackageTileCacheOverlay: (NSMutableDictionary<NSString *, CacheOve
                 cacheOverlay = nil;
             } else {
                 // remove the old one and it will be re-added to preserve layer order
+                if ([tileTableCacheOverlay.tileOverlay isKindOfClass:[GPKGBoundedOverlay class]]){
+                    [((GPKGBoundedOverlay *)tileTableCacheOverlay.tileOverlay) close];
+                }
                 [self.mapView removeOverlay:tileTableCacheOverlay.tileOverlay];
                 cacheOverlay = nil;
             }
@@ -293,6 +297,9 @@ -(void) addGeoPackageTileCacheOverlay: (NSMutableDictionary<NSString *, CacheOve
             [tileTableCacheOverlay setTileOverlay:geoPackageTileOverlay];
             
             // Check for linked feature tables
+            for (GPKGFeatureOverlayQuery *query in tileTableCacheOverlay.featureOverlayQueries) {
+                [query close];
+            }
             [tileTableCacheOverlay.featureOverlayQueries removeAllObjects];
             GPKGFeatureTileTableLinker * linker = [[GPKGFeatureTileTableLinker alloc] initWithGeoPackage:geoPackage];
             NSArray<GPKGFeatureDao *> * featureDaos = [linker featureDaosForTileTable:tileDao.tableName];
@@ -333,6 +340,7 @@ -(void) addGeoPackageTileCacheOverlay: (NSMutableDictionary<NSString *, CacheOve
                 [tileTableCacheOverlay removeFromMap:weakSelf.mapView];
             }
             if (geoPackageTileOverlay != nil) {
+                [geoPackageTileOverlay close];
                 [weakSelf.mapView removeOverlay:geoPackageTileOverlay];
             }
         });
@@ -468,6 +476,7 @@ -(void) addGeoPackageFeatureCacheOverlay: (NSMutableDictionary<NSString *, Cache
                 }
                 @finally {
                     [resultSet close];
+                    [shapeConverter close];
                 }
             }
 
@@ -494,6 +503,7 @@ -(void) addGeoPackageFeatureCacheOverlay: (NSMutableDictionary<NSString *, Cache
                 [featureTableCacheOverlay removeFromMap:weakSelf.mapView];
             }
             if (featureOverlay != nil) {
+                [featureOverlay close];
                 [self.mapView removeOverlay:featureOverlay];
             }
         });
diff --git a/Mage/Map/Cache/GeoPackageFeatureTableCacheOverlay.m b/Mage/Map/Cache/GeoPackageFeatureTableCacheOverlay.m
index 678a0b58..9cd5f3fa 100644
--- a/Mage/Map/Cache/GeoPackageFeatureTableCacheOverlay.m
+++ b/Mage/Map/Cache/GeoPackageFeatureTableCacheOverlay.m
@@ -189,6 +189,7 @@ -(GPKGFeatureTableData *) getFeatureTableDataWithLocationCoordinate: (CLLocation
                                 SFPGeometryTransform *transform = [[SFPGeometryTransform alloc] initWithFromProjection:self.featureOverlayQuery.featureTiles.featureDao.projection andToEpsg:4326];
                                 
                                 centroid = [transform transformPoint:centroid];
+                                [transform destroy];
                                 coordinate = CLLocationCoordinate2DMake([centroid.y doubleValue], [centroid.x doubleValue]);
                             }
                             [featureDataTypes setValue:[GPKGDataTypes name:featureRow.featureColumns.columns[i].dataType] forKey:columnName];
diff --git a/Mage/Map/MapObservationManager.m b/Mage/Map/MapObservationManager.m
index 2e88f08d..15b03c00 100644
--- a/Mage/Map/MapObservationManager.m
+++ b/Mage/Map/MapObservationManager.m
@@ -65,6 +65,7 @@ -(MapObservation *) addToMapWithObservation: (Observation *) observation withGeo
         
         GPKGMapShapeConverter *shapeConverter = [[GPKGMapShapeConverter alloc] init];
         GPKGMapShape *shape = [shapeConverter toShapeWithGeometry:geometry];
+        [shapeConverter close];
         switch(shape.shapeType){
             case GPKG_MST_POLYLINE:
                 {
diff --git a/Mage/Mixins/FilteredObservationsMap.swift b/Mage/Mixins/FilteredObservationsMap.swift
index 032cb4df..5962c1d2 100644
--- a/Mage/Mixins/FilteredObservationsMap.swift
+++ b/Mage/Mixins/FilteredObservationsMap.swift
@@ -155,6 +155,7 @@ class FilteredObservationsMapMixin: NSObject, MapMixin {
                 let style = ObservationShapeStyleParser.style(of: observation)
                 let shapeConverter = GPKGMapShapeConverter()
                 let shape = shapeConverter?.toShape(with: geometry)
+                shapeConverter?.close()
                 
                 if let mkpolyline = shape?.shape as? MKPolyline {
                     let styledPolyline = StyledPolyline.create(polyline: mkpolyline)
diff --git a/Mage/Mixins/SFGeometryMap.swift b/Mage/Mixins/SFGeometryMap.swift
index bd649fac..514301c2 100644
--- a/Mage/Mixins/SFGeometryMap.swift
+++ b/Mage/Mixins/SFGeometryMap.swift
@@ -46,6 +46,7 @@ class SFGeometryMapMixin: NSObject, MapMixin {
         }
         let shapeConverter: GPKGMapShapeConverter = GPKGMapShapeConverter()
         let shape = shapeConverter.add(sfGeometry, to: mapView)
+        shapeConverter.close()
         geometryToShape[sfGeometry] = shape
     }