Code review comment for lp:~statik/config-manager/specific-revisions

Revision history for this message
Robert Collins (lifeless) wrote :

On Sat, 2009-06-20 at 03:45 +0000, Elliot Murphy wrote:
>
> + else:
> + revision_id = br_from.last_revision()

here, revision_id=None would be correct and avoid lookups outside of
locks.

Otherwise good, I'll merge Monday.

-Rob

« Back to merge proposal