[WW-4124] Wasted work in ValidateVisitor.checkXmlAttributes()

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.3.14.3
• Fix Version/s: 2.3.16
• Component/s: Plugin - Embedded JSP
• Labels: patch performance

Environment

any

• Flags: Patch, Important
• Assignee: Lukasz Lenart
• Reporter: Adrian Nistor
• Created: 21/Jun/13 19:14
• Updated: 08/Dec/13 17:21
• Resolved: 12/Jul/13 07:09

Description

The problem appears in version 2.3.14.3 and in revision 1495522. I
attached a one-line patch that fixes it. This problem and the
attached patch are similar to the previously fixed WW-4115 and its
patch.

In method "ValidateVisitor.checkXmlAttributes", the loop over "nodes"
should break immediately after "elExpression" and
"deferredValueIsLiteral" are set to "true". All the iterations after
"elExpression" and "deferredValueIsLiteral" are set to "true" do not
perform any useful work, at best they just set "elExpression" and
"deferredValueIsLiteral" again to "true".