Re: [aur-dev] [PATCH] Add a logging function and logging fixins through the code

On 02/07/2009, at 1:31 AM, Sebastian Nowicki wrote:

Might be easier to just put the "if(LOGGING) {}" into the logit() function itself, assuming this is the desired behavior for all instances of logging.

I feel like an idiot for not putting it there in the first place but now that I think about it... if logging was turned off we would be doing 2 hits to the DB for almost every single action needlessly if the check was inside the function. So maybe it's not worth the simplicity?