[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