[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