[PATCH 4 of 4] ui: avoid mutable default arguments

Kevin Bullock kbullock+mercurial at ringworld.org
Thu Sep 24 10:51:00 CDT 2015


> On Sep 24, 2015, at 08:05, Pierre-Yves David <pierre-yves.david at ens-lyon.org> wrote:
> 
> On 09/23/2015 09:16 AM, Matt Mackall wrote:
>> On Wed, 2015-09-23 at 02:05 -0700, Pierre-Yves David wrote:
>>> 
>>> On 09/22/2015 05:15 PM, Siddharth Agarwal wrote:
>>>> # HG changeset patch
>>>> # User Siddharth Agarwal <sid0 at fb.com>
>>>> # Date 1442966194 25200
>>>> #      Tue Sep 22 16:56:34 2015 -0700
>>>> # Node ID 75950e78a828c5994bd36974bdfbc5f62a0009a3
>>>> # Parent  2e9d6b73a2a88c044175a05d32626b18afc0e889
>>>> ui: avoid mutable default arguments
>>> 
>>> eeeeeeeerk, patches 3 and 4 pushed to the clowncopter.
>> 
>> We've got a bunch more {} and [].
> 
> I've cleaned these up, this is an about 27 patches series that can be pulled here:
> 
> hg pull -r 266a98ba3e42 http://hg.netv6.net/marmoute-wip/mercurial/ <http://hg.netv6.net/marmoute-wip/mercurial/>


Series LGTM.

pacem in terris / мир / शान्ति / ‎‫سَلاَم‬ / 平和
Kevin R. Bullock

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20150924/eb86d3d5/attachment.html>


More information about the Mercurial-devel mailing list