style: Avoid double calls of os.path.split (#3811)

Shorter, nicer and also performs little better.
OSGeo · Jun 15, 2024 · b3f9812 · b3f9812
1 parent ca553a5
commit b3f9812
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/display/d.mon/render_cmd.py b/display/d.mon/render_cmd.py
@@ -27,8 +27,7 @@
 # remove empty mapfile from non-rendering modules
 def remove_mapfile(mapfile):
     # adopted from Map.DeleteLayer() in gui/wxpython/core/render.py
-    base = os.path.split(mapfile)[0]
-    mapfile = os.path.split(mapfile)[1]
+    base, mapfile = os.path.split(mapfile)
     tempbase = mapfile.split(".")[0]
     if base == "" or tempbase == "":
         return

diff --git a/gui/wxpython/core/render.py b/gui/wxpython/core/render.py
@@ -1441,8 +1441,7 @@ def DeleteLayer(self, layer, overlay=False):
 
         if layer in list:
             if layer.mapfile:
-                base = os.path.split(layer.mapfile)[0]
-                mapfile = os.path.split(layer.mapfile)[1]
+                base, mapfile = os.path.split(layer.mapfile)
                 tempbase = mapfile.split(".")[0]
                 if base == "" or tempbase == "":
                     return None