[PATCH default] formatter: improve implementation of data method

David Carr davidmc24 at gmail.com
Fri Sep 21 07:20:04 CDT 2012


# HG changeset patch
# User David M. Carr  <david at carrclan.us>
# Date 1348198259 14400
# Node ID 9d390bd2fc4e00ce330a520db6634040099edd9e
# Parent  ccd28eca37f67bcf534b016c0fd6b99140bc0632
formatter: improve implementation of data method

This alternate syntax was proposed by Bryan O'Sullivan in a review of
772b3764d3e8.  I haven't been able to measure any particular performance
difference, but the new syntax is more concise and easier to read.

diff --git a/mercurial/formatter.py b/mercurial/formatter.py
--- a/mercurial/formatter.py
+++ b/mercurial/formatter.py
@@ -26,8 +26,7 @@
         self._item = {}
     def data(self, **data):
         '''insert data into item that's not shown in default output'''
-        for k, v in data.iteritems():
-            self._item[k] = v
+        self._item.update(data)
     def write(self, fields, deftext, *fielddata, **opts):
         '''do default text output while assigning data to item'''
         for k, v in zip(fields.split(), fielddata):


More information about the Mercurial-devel mailing list