[MERGE] add stop_rule to Branch.iter_merge_sorted_revisions()

Martin Albisetti argentina at gmail.com
Tue Jan 27 00:43:38 GMT 2009


On Mon, Jan 26, 2009 at 10:18 PM, Ian Clatworthy
<ian.clatworthy at internode.on.net> wrote:
>> Any thoughts on the best default? It's currently "include" but
>> the API has only existed for 24 hours so I'll change it to
>> "exclude" (say) if we think that's the most common case.
>
> I've made exclude the default. I don't think it matters much and
> I'm happy to choose a different one if anyone feels strongly.

I don't want to make any noise, but for Loggerhead's use case, it will
generally by "include".
Again, if there's any benefit in making it exclude, it's not really
much of a hassle to pass in another parameter  :)


Thanks for the work you've done on this!

-- 
Martin



More information about the bazaar mailing list