Re: [Patch] Improve logging function

Alexander Malysh <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Message-ID <[email protected]>
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 <[email protected]>:

> 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>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.