[MERGE] Merge directive format 2, Bundle format 4
Aaron Bentley
aaron.bentley at utoronto.ca
Sat Jun 30 15:32:49 BST 2007
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Vincent Ladeuil wrote:
> Not a complete review, just a few remarks.
>
>>>>>> "aaron" == Aaron Bentley <aaron.bentley at utoronto.ca> writes:
>
> <snip/>
>
> aaron> === added file 'doc/developers/bundle-format4.txt'
>
> Great ! Can you add a pointer to this file as comments in the relevant files ?
Okay.
> aaron> === modified file 'bzrlib/errors.py'
>
> We are nearly consistent in errors.py in defining exceptions
> *without* a final 'Error' except for base classes to derive from.
>
> Also, we are a bit less consistent in using a sentence explaining
> the error as a format without duplicating the error name itself.
Okay. These are actually critiques of the container code, which is
going through a separate review, but I'll apply them there.
Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFGhmmR0F+nu1YWqI0RAjsyAJwOp/yvDM5evEkboYJg+zdq9VNEBQCfVYa/
n1OJE1Nip+SKESw64kHGhI0=
=8t/8
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list