Hi,

for improvement +1 for the implementation -1. You change a way how we used 
rwlock . Rwlock is to protect
log file list and not the output. Please keep rwlock in place then +1 from me.

Alex

Am 02.10.2013 um 17:52 schrieb Jacek Raczkiewicz <jacek.raczkiew...@gmail.com>:

> Hi, 
> 
> I have been working on imrpving application that uses a lot libraries from 
> kannel/gwlib.
> I noticed that logging function does format the string even if there is no 
> intention to save it to the log due to configured log level. 
> My improvement is that I moved the format() function inside the conditional 
> block which check for log level.
> This really make a difference when logged string is large (in our case it may 
> be as big as 1MB) and that is formatted for no reason.
> 
> Please include this fix into kannel codebase.
> 
> Regards 
> Jacek Raczkiewicz
> <logging.diff>


Reply via email to