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