[jira] Updated: (WW-2282) Apply patch to Struts2 core module on performance issues reported by FindBugs

["Giovanni Azua Garcia (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/struts/browse/WW-2282?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Giovanni Azua Garcia updated WW-2282:
-------------------------------------

    Attachment: Struts2CorePerfPatch_ToRev590856.zip

Zip containing all files patched.

> 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.