[VELOCITY-731] Velocity 1.6 performance is degraded by introduced toString() calls

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.6, 1.6.1, 1.6.2
• Fix Version/s: 1.6.x, 1.7
• Component/s: Engine
• Environment: Apache Velocity 1.6.2
• Assignee: Unassigned
• Reporter: Jörgen Rydenius
• Created: 19/Aug/09 16:46
• Updated: 07/Jan/18 15:33
• Resolved: 21/Aug/09 15:32

Description

As part of VELOCITY-531, r686428 introduced frequent calls to toString() just to check if toString() != null. If toString() implementations are slightly complex, and very frequently executed, these calls will degrade the site performance. Are they at all necessary? What kind of object would use null as a return value for toString()? Is it possible to remove these toString() calls from the velocity code again?