comparison mercurial/templateutil.py @ 48913:f254fc73d956

global: bulk replace simple pycompat.iteritems(x) with x.items() pycompat.iteritems() just calls .items(). This commit applies a regular expression search and replace to convert simple instances of pycompat.iteritems() with .items(). There are still a handful of calls to pycompat.iteritems() remaining. But these all have more complicated expressions that I wasn't comfortable performing an automated replace on. In addition, some simple replacements were withheld because they broke pytype. These will be handled by their own changesets. Differential Revision: https://phab.mercurial-scm.org/D12318
author Gregory Szorc <gregory.szorc@gmail.com>
date Thu, 03 Mar 2022 18:28:30 -0800
parents 6000f5b25c9b
children 642e31cb55f0
comparison
equal deleted inserted replaced
48912:a0674e916fb6 48913:f254fc73d956
308 308
309 def filter(self, context, mapping, select): 309 def filter(self, context, mapping, select):
310 if util.safehasattr(self._values, b'get'): 310 if util.safehasattr(self._values, b'get'):
311 values = { 311 values = {
312 k: v 312 k: v
313 for k, v in pycompat.iteritems(self._values) 313 for k, v in self._values.items()
314 if select(self._wrapvalue(k, v)) 314 if select(self._wrapvalue(k, v))
315 } 315 }
316 else: 316 else:
317 values = [v for v in self._values if select(self._wrapvalue(v, v))] 317 values = [v for v in self._values if select(self._wrapvalue(v, v))]
318 return hybrid(None, values, self._makemap, self._joinfmt, self._keytype) 318 return hybrid(None, values, self._makemap, self._joinfmt, self._keytype)
340 340
341 def tovalue(self, context, mapping): 341 def tovalue(self, context, mapping):
342 # TODO: make it non-recursive for trivial lists/dicts 342 # TODO: make it non-recursive for trivial lists/dicts
343 xs = self._values 343 xs = self._values
344 if util.safehasattr(xs, b'get'): 344 if util.safehasattr(xs, b'get'):
345 return { 345 return {k: unwrapvalue(context, mapping, v) for k, v in xs.items()}
346 k: unwrapvalue(context, mapping, v)
347 for k, v in pycompat.iteritems(xs)
348 }
349 return [unwrapvalue(context, mapping, x) for x in xs] 346 return [unwrapvalue(context, mapping, x) for x in xs]
350 347
351 348
352 class hybriditem(mappable, wrapped): 349 class hybriditem(mappable, wrapped):
353 """Wrapper for non-list/dict object to support map operation 350 """Wrapper for non-list/dict object to support map operation
535 # drop internal resources (recursively) which shouldn't be displayed 532 # drop internal resources (recursively) which shouldn't be displayed
536 lm = context.overlaymap(mapping, nm) 533 lm = context.overlaymap(mapping, nm)
537 items.append( 534 items.append(
538 { 535 {
539 k: unwrapvalue(context, lm, v) 536 k: unwrapvalue(context, lm, v)
540 for k, v in pycompat.iteritems(nm) 537 for k, v in nm.items()
541 if k not in knownres 538 if k not in knownres
542 } 539 }
543 ) 540 )
544 return items 541 return items
545 542
713 """Wrap data like hybriddict(), but also supports old-style list template 710 """Wrap data like hybriddict(), but also supports old-style list template
714 711
715 This exists for backward compatibility with the old-style template. Use 712 This exists for backward compatibility with the old-style template. Use
716 hybriddict() for new template keywords. 713 hybriddict() for new template keywords.
717 """ 714 """
718 c = [{key: k, value: v} for k, v in pycompat.iteritems(data)] 715 c = [{key: k, value: v} for k, v in data.items()]
719 f = _showcompatlist(context, mapping, name, c, plural, separator) 716 f = _showcompatlist(context, mapping, name, c, plural, separator)
720 return hybriddict(data, key=key, value=value, fmt=fmt, gen=f) 717 return hybriddict(data, key=key, value=value, fmt=fmt, gen=f)
721 718
722 719
723 def compatlist( 720 def compatlist(