23 Jan
2008
23 Jan
'08
3:32 p.m.
http://bugzilla.qos.ch/show_bug.cgi?id=104 ------- Comment #3 from joern@huxhorn.de 2008-01-23 16:32 ------- Looks fine. I just think that the "if (isDebugEnabled())" in line 414 isn't necessary if a "if (isTraceEnabled())" isn't necessary in line 390. I thought that this was probably a performance optimization since debug and trace is called more often than info, warn and error... I wasn't sure so I just changed it to isTrace instead of isDebug. Am I supposed to close this bug? I don't have a testcase but reviewed the changes. -- Configure bugmail: http://bugzilla.qos.ch/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee.