Re: [PATCH] Log all transactions

From: Kinkie <gkinkie_at_gmail.com>
Date: Thu, 31 Mar 2011 22:36:56 +0200

> I left the "if (true)" guard to prevent indentation changes to the
> pretty long logRequest method, making the patch more difficult to
> understand. I will remove that otherwise useless guard during commit
> unless you think it should stay.

+1, with the suggestion of possibly making the debugs() statement have
a lower priority (that is, make it harder to print).

-- 
    /kinkie
Received on Thu Mar 31 2011 - 20:37:02 MDT

This archive was generated by hypermail 2.2.0 : Fri Apr 01 2011 - 12:00:05 MDT