logging overhaul (please read)
Derek Atkins
warlord at MIT.EDU
Fri Feb 9 13:10:24 EST 2007
Quoting Josh Sled <jsled at asynchronous.org>:
[snip]
> The space-based indentation as per ENTER/LEAVE has been disabled.
> Before, the {in,de}dent calls were conditional on qof_log_check, but
> that no longer exists. I'm pretty sure it can fit back in by creating
> two new log levels... but I'd like to hear feedback first? I personally
> don't find it useful, but I think others do, and it's a pretty big
> regression.
I personally like this functionality and would miss it if it's gone
for good. It's EXTREMELY useful when you're trying to trace the
program to see when you get out of a particular call tree by
looking at the indentation level. It's most helpful when you have
a re-entrant function.
> - "log4glib_handler" optimization
> - don't alloc strings all the time
> - run the log-level test longest->shortest
I think I like the log-level test short->long, because it means
if I set e.g. "gnc" -> debug then I get ALL "gnc.*" messages,
even if gnc.foo.bar is set to "error"..
-derek
--
Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
Member, MIT Student Information Processing Board (SIPB)
URL: http://web.mit.edu/warlord/ PP-ASEL-IA N1NWH
warlord at MIT.EDU PGP key available
More information about the gnucash-devel
mailing list