[Merge] Slow socket
Robert Collins
robertc at robertcollins.net
Wed Aug 16 03:24:55 BST 2006
On Tue, 2006-08-15 at 18:37 -0500, John Arbash Meinel wrote:
>
> I know what Robert said, though I disagree with him.
> To me 'from bzrlib import' is (bzrlib, '') not (bzrlib, import). While
> and the other is (bzrlib, branch).
> Now, your original form was still wrong, because the last line is
> (bzrlib, tests, TestUtil). So *I* think it needs to go inbetween:
>
> from bzrlib import (
> + bzrdir,
> plugin,
> )
> +import bzrlib.branch
> from bzrlib.tests.TestUtil import TestLoader
I wasn't very clear.
from bzrlib import is (bzrlib, '') to me too.
The hunk I was looking at appeared to have a bzrlib.i??? section in it
IIRC.
The only other comment I've got is that <3 letter variable names
are /really/ hard for people to read.
wt sort of sneaks through, but even it is not desirable.
Rob
--
GPG key available at: <http://www.robertcollins.net/keys.txt>.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 191 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060816/c480e287/attachment.pgp
More information about the bazaar
mailing list