[merge][rfc] log+ transport decorator
Robert Collins
robertc at robertcollins.net
Tue Sep 2 06:21:55 BST 2008
On Tue, 2008-09-02 at 15:08 +1000, Martin Pool wrote:
> On Tue, Sep 2, 2008 at 2:57 PM, Robert Collins
> <robertc at robertcollins.net> wrote:
> > This looks nice, I'm not clear about the log+ vs trace+ relationship
> > from the diff alone - can you expand on that please?
>
> trace captures them into an instance variable so that it can be
> inspected by a test that wants to eg make effort assertions about what
> IO is done. (This was existing code, which I think you added.) log+
> writes it to .bzr.log for ad-hoc debugging.
Yes, thats the _difference_. Whats the relationship? Aren't they very
similar? Is one a subclass, to avoid duplication, or are their needs so
different that this won't work? I know I wrote trace+ with the intent of
it being available for a log+ behaviour in the future (as the docstring
patch you have removes that comment I know you know that too :)).
-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: 189 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20080902/ddfe02f4/attachment-0001.pgp
More information about the bazaar
mailing list