[PATCH 6 of 6 v6] strip: incrementally update the branchheads cache after a strip

Bryan O'Sullivan bos at serpentine.com
Fri Jun 15 18:11:18 UTC 2012


On Fri, Jun 15, 2012 at 10:35 AM, Joshua Redstone <joshua.redstone at fb.com>wrote:

>
>> +    newheadbranches = set([repo[rev].branch() for rev in newheadrevs])
>>
>
>  newheadbranches does not need to be a set.
>
>  I think it does. otherwise if there are duplicates then the equality
> check below fails..
>
>
>  +    if len(stripbranches) == 1 and len(newheadbranches) == 1 \
>> +            and stripbranches == newheadbranches:
>>
>
There cannot be duplicates if the collection contains only a single
element.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mercurial-scm.org/pipermail/mercurial-devel/attachments/20120615/e65361f0/attachment-0002.html>


More information about the Mercurial-devel mailing list