# HG changeset patch
# User Yuya Nishihara <yuya@tcha.org>
# Date 1492841166 -32400
# Node ID 11e667a8fcbaa590116e121338fa076773ebbed9
# Parent  de7dc25ecf2e823311fb46a8b0a474a58ededa6a
formatter: factor out function to create templater from literal or map file

(tmpl, mapfile) will be packed into a named tuple later.

diff -r de7dc25ecf2e -r 11e667a8fcba mercurial/cmdutil.py
--- a/mercurial/cmdutil.py	Sat Apr 22 14:53:05 2017 +0900
+++ b/mercurial/cmdutil.py	Sat Apr 22 15:06:06 2017 +0900
@@ -1580,15 +1580,8 @@
 
     def __init__(self, ui, repo, matchfn, diffopts, tmpl, mapfile, buffered):
         changeset_printer.__init__(self, ui, repo, matchfn, diffopts, buffered)
-        assert not (tmpl and mapfile)
-        defaulttempl = templatekw.defaulttempl
-        if mapfile:
-            self.t = templater.templater.frommapfile(mapfile,
-                                                     cache=defaulttempl)
-        else:
-            self.t = formatter.maketemplater(ui, 'changeset', tmpl,
-                                             cache=defaulttempl)
-
+        self.t = formatter.loadtemplater(ui, 'changeset', (tmpl, mapfile),
+                                         cache=templatekw.defaulttempl)
         self._counter = itertools.count()
         self.cache = {}
 
diff -r de7dc25ecf2e -r 11e667a8fcba mercurial/formatter.py
--- a/mercurial/formatter.py	Sat Apr 22 14:53:05 2017 +0900
+++ b/mercurial/formatter.py	Sat Apr 22 15:06:06 2017 +0900
@@ -408,6 +408,12 @@
 
 def gettemplater(ui, topic, spec, cache=None):
     tmpl, mapfile = lookuptemplate(ui, topic, spec)
+    return loadtemplater(ui, topic, (tmpl, mapfile), cache=cache)
+
+def loadtemplater(ui, topic, spec, cache=None):
+    """Create a templater from either a literal template or loading from
+    a map file"""
+    tmpl, mapfile = spec
     assert not (tmpl and mapfile)
     if mapfile:
         return templater.templater.frommapfile(mapfile, cache=cache)