# HG changeset patch # User Yuya Nishihara <y...@tcha.org> # Date 1491395264 -32400 # Wed Apr 05 21:27:44 2017 +0900 # Node ID ab39d72f0fd0e26f8803482d337adb7528f25a96 # Parent 4c2c30bc38b4f84ce8f215146bbf158e299065b3 templatekw: eliminate unnecessary temporary variable 'names' from _showlist()
Replace 'names' with the optional argument 'plural'. diff --git a/mercurial/templatekw.py b/mercurial/templatekw.py --- a/mercurial/templatekw.py +++ b/mercurial/templatekw.py @@ -106,11 +106,10 @@ def _showlist(name, values, plural=None, expand 'end_foos'. ''' templ = args['templ'] - if plural: - names = plural - else: names = name + 's' + if not plural: + plural = name + 's' if not values: - noname = 'no_' + names + noname = 'no_' + plural if noname in templ: yield templ(noname, **args) return @@ -121,7 +120,7 @@ def _showlist(name, values, plural=None, for v in values: yield dict(v, **args) return - startname = 'start_' + names + startname = 'start_' + plural if startname in templ: yield templ(startname, **args) vargs = args.copy() @@ -144,7 +143,7 @@ def _showlist(name, values, plural=None, yield one(v) if last is not None: yield one(last, tag=lastname) - endname = 'end_' + names + endname = 'end_' + plural if endname in templ: yield templ(endname, **args) _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel