[
https://issues.apache.org/struts/browse/WW-2282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_42492
]
Giovanni Azua Garcia commented on WW-2282:
------------------------------------------
btw I checked all tests pass before/after the patch.
regards,
Giovanni
> Apply patch to Struts2 core module on performance issues reported by FindBugs
> ------------------------------------------------------------------------------
>
> Key: WW-2282
> URL: https://issues.apache.org/struts/browse/WW-2282
> Project: Struts 2
> Issue Type: Improvement
> Components: Core Actions
> Affects Versions: 2.1.0
> Reporter: Giovanni Azua Garcia
> Assignee: Ted Husted
> Priority: Minor
> Attachments: Struts2CorePerfPatch_ToRev590856.diff,
> Struts2CorePerfPatch_ToRev590856.zip
>
>
> Post to DEV list:
> Recently I was following some threads in the Struts users list about
> performance issues in S2 and was curious to look at Struts code. Running
> findBugs evidences many small improvements that would overall and cheaply
> improve S2 performance e.g.
> - In many places it is continuously creating large number of small objects by
> using new rather than valueOf.
> http://findbugs.sourceforge.net/bugDescriptions.html#DM_NUMBER_CTOR
> - FindBugs spotted several places using non static inner classes
> unnecessarily.
> - Also a few places use inneficiently keySet iterators on maps rather than
> using entrySet.
> - Calling toString on String types.
> Good idea probably adding findbugs to the pom reporting.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
