[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