Code review comment for lp:~luoyonggang/bzr-svn/parse_revnum

Jelmer Vernooij (jelmer) wrote :

> Er... OK.

Layout implementations don't usually have a reference to the repository. Passing in the revnum *and* the repository means that other people who implement custom layouts will be able to use revision-specific data. It also means we'll only have to change the existing API only once (I have no idea if and how many custom bzr plugins there are out there that add custom layout implementations).

« Back to merge proposal