templater: simplify templatepaths() to avoid iterating a singleton list
authorMartin von Zweigbergk <martinvonz@google.com>
Tue, 21 Jul 2020 13:05:37 -0700
changeset 45198 e2ec36167151
parent 45197 55464c0b3a89
child 45199 91aa9bba3dc9
templater: simplify templatepaths() to avoid iterating a singleton list The function iterates over a hard-coded list of one element since d844e220792a (templater: don't search randomly for templates - trust util.datapath, 2014-09-28). Differential Revision: https://phab.mercurial-scm.org/D8785
mercurial/templater.py
--- a/mercurial/templater.py	Mon Jul 06 22:46:49 2020 +0200
+++ b/mercurial/templater.py	Tue Jul 21 13:05:37 2020 -0700
@@ -1047,12 +1047,8 @@
 
 def templatepaths():
     '''return locations used for template files.'''
-    pathsrel = [b'templates']
-    paths = [
-        os.path.normpath(os.path.join(resourceutil.datapath, f))
-        for f in pathsrel
-    ]
-    return [p for p in paths if os.path.isdir(p)]
+    path = os.path.normpath(os.path.join(resourceutil.datapath, b'templates'))
+    return [path] if os.path.isdir(path) else []
 
 
 def templatepath(name):