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


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.

Reply via email to