[MERGE] init remote branches

Aaron Bentley aaron.bentley at utoronto.ca
Sun Jul 2 22:37:02 BST 2006


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Wouter van Heyst wrote:
> -        transport = get_transport(location)
> +        transport = bzrlib.transport.get_transport(location)

One style nitpick: I think it's nicer to importexceeding 80 chars, or
particular names than to import the whole package.  I prefer
get_transport to bzrlib.transport.get_transport.

- - It's much shorter, so we don't wind up having to contort our code to
avoid exceeding 80 chars.
- - It declares what we're using at the top, which can be useful to know.
 (For example, if someone removed get_transport from bzrlib.transport,
we'd get an ImportError from importing builtins, instead of when we ran
init.)

Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2.2 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFEqDx+0F+nu1YWqI0RAo9VAJ0W0/1dqLnyhSIGjph2OvcGoDaDyACfb6kn
t2Vt/LAf5xky9HyzTQUBttk=
=bgxf
-----END PGP SIGNATURE-----




More information about the bazaar mailing list