addGeoRegion() actually doesn't serve a purpose, replace with add()

GeoRegionsEcosystem · Sep 7, 2024 · 2085361 · 2085361
1 parent f9cec08
commit 2085361
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 25 deletions.
diff --git a/src/GeoRegions.jl b/src/GeoRegions.jl
@@ -16,9 +16,7 @@ export
         RectRegion, PolyRegion, TiltRegion,
 
         ==, isGeoRegion, isID,
-        add, addGeoRegion,
-        rm, rmID,
-        overwrite,
+        add, rm, rmID, overwrite,
 
         templateGeoRegions, readGeoRegions, addGeoRegions, resetGeoRegions,
         tableGeoRegions, tableRectRegions, tableTiltRegions, tablePolyRegions,

diff --git a/src/georegions/add.jl b/src/georegions/add.jl
@@ -1,23 +1,4 @@
-add(
-    geo  :: GeoRegion;
-    path :: AbstractString = dirname(geo.path),
-    verbose :: Bool = true
-) = addGeoRegion(geo,path,verbose=verbose)
-
-function overwrite(
-    geo  :: GeoRegion;
-    path :: AbstractString,
-    verbose :: Bool = true
-)
-
-    rmID(geo.ID,path=path)
-    add(geo,path=path,verbose=verbose)
-
-    return nothing
-
-end
-
-function addGeoRegion(
+function add(
     geo  :: RectRegion,
     path :: AbstractString = dirname(geo.path);
     verbose :: Bool = true
@@ -33,7 +14,7 @@ function addGeoRegion(
 
 end
 
-function addGeoRegion(
+function add(
     geo  :: TiltRegion,
     path :: AbstractString = dirname(geo.path);
     verbose :: Bool = true
@@ -49,7 +30,7 @@ function addGeoRegion(
 
 end
 
-function addGeoRegion(
+function add(
     geo  :: PolyRegion,
     path :: AbstractString = dirname(geo.path);
     verbose :: Bool = true
@@ -65,4 +46,17 @@ function addGeoRegion(
 
     return nothing
 
+end
+
+function overwrite(
+    geo  :: GeoRegion;
+    path :: AbstractString,
+    verbose :: Bool = true
+)
+
+    rmID(geo.ID,path=path)
+    add(geo,path=path,verbose=verbose)
+
+    return nothing
+
 end