[MERGE] bzrlib.trace.debug_memory() helper

Martin Pool mbp at canonical.com
Thu Sep 11 03:43:08 BST 2008


On 9/11/08, Andrew Bennetts <andrew at canonical.com> wrote:
> Andrew Bennetts has voted approve.
>  Status is now: Approved
>  Comment:
>  I wonder if perhaps we should leave the "if 'memory' in debug.debug_flags"
> check to the caller of debug_memory, just like we do elsewhere.  I'm happy
> for this to get merged, though.

I'd rather we consistently did it before calling the debug routine,
because we know that if it's done too often the time to call a method
before doing the check will add up.

-- 
Martin <http://launchpad.net/~mbp/>



More information about the bazaar mailing list