Opened 13 years ago

#852 new enhancement

Add debug guards to all debug statements

Reported by: jesseeichar Owned by: jeichar
Priority: minor Milestone: v2.6.5
Component: General Version: v2.6.4
Keywords: Cc:

Description

In many of our debug logs we do things like:

Log.debug(Log.JEEVES, "message: "+Xml.getString(xml));

This can be very computationally expensive if xml is large. That is fine when in debug mode, unfortunately Xml.getString(xml) is done even when not in debug mode.

Sometime ago I added Log.isDebug(...) so a check can be made before doing an debug log. Last night I started a sweep through the code adding the guard to each debug statement.

Attachments (1)

isDebug.patch (363.3 KB ) - added by jesseeichar 13 years ago.

Download all attachments as: .zip

Change History (1)

by jesseeichar, 13 years ago

Attachment: isDebug.patch added
Note: See TracTickets for help on using tickets.