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>