[MERGE] Branch.iter_merge_sorted_revisions
Martin Pool
mbp at canonical.com
Fri Jan 23 23:49:15 GMT 2009
Martin Pool has voted tweak.
Status is now: Conditionally approved
Comment:
This looks very tidy to me.
+ """Walk the revisions for a branch in merge sorted order.
Maybe add/copy a one-sentence definition of 'merge sorted'?
+
+ :return: an iterator over
+ (revision_id, depth, revno, end_of_merge) tuples.
I'd like to see some description of what end_of_merge is.
+ if stop_revision_id is not None and rev_id ==
stop_revision_id:
+ raise StopIteration
Is this different to just exiting the generator function? If so, maybe
explain why?
If you've addressed all John's comments then I think it's good to go.
--
Martin
For details, see:
http://bundlebuggy.aaronbentley.com/project/bzr/request/%3C497A3749.5050104%40internode.on.net%3E
Project: Bazaar
More information about the bazaar
mailing list