[PATCH 2 of 2] revset: added extend and append methods to lazy structures
Lucas Moscovicz
lmoscovicz at fb.com
Fri Feb 21 17:57:33 CST 2014
On 2/21/14, 3:55 PM, "Matt Mackall" <mpm at selenic.com> wrote:
>On Fri, 2014-02-21 at 23:53 +0000, Lucas Moscovicz wrote:
>>
>> On 2/21/14, 3:52 PM, "Matt Mackall" <mpm at selenic.com> wrote:
>>
>> >Ok, that answers half of my question?
>>
>> I¹m not sure if append is used, I implemented it just in case I need it
>>in
>> another patch but I think I can just remove it for now.
>
>Let me try again: can we nuke all the code that attempts to use
>list.extend on revsets rather than add an extend method to a bunch of
>classes?
I think we actually may be able to do that. I’ll see if I can get it
working without the extend method.
>
>--
>Mathematics is the supreme nostalgia of our time.
>
>
More information about the Mercurial-devel
mailing list