[
https://issues.apache.org/jira/browse/WW-3578?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Lukasz Lenart closed WW-3578.
-----------------------------
Fix Version/s: (was: 6.1.0)
Resolution: Won't Fix
The Embedded JSP plugin is deprecated and will be removed with the next major
release
> replace function is called multiple times, which can be done in one run
> -----------------------------------------------------------------------
>
> Key: WW-3578
> URL: https://issues.apache.org/jira/browse/WW-3578
> Project: Struts 2
> Issue Type: Bug
> Components: Plugin - JasperReports
> Affects Versions: 2.2.1.1
> Reporter: Xiaoming Shi
> Priority: Major
>
> In the function "makeXmlJavaIdentifier"
> (./struts-2.2.1/src/plugins/embeddedjsp/src/main/java/org/apache/struts2/jasper/compiler/JspUtil.java
> line: 1059)
> The replace function is called 3 times, which can be done in one run with a
> for loop.
> StringBuilder sb = new StringBuilder(name.length());
> for(int i = 0; i < name.length(); i++)
> {
> char c = name.charAt(i);
> if (c == '-')
> sb.append("$1");
> else if (c == '.')
> sb.append("$2");
> else if (c == ':')
> sb.append("$3");
> else
> sb.append(c);
> }
> return sb.toString();
> The second method can be much faster.
> (Similar to the MySQL bug http://bugs.mysql.com/bug.php?id=45699)
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
