https://issues.apache.org/bugzilla/show_bug.cgi?id=49140

           Summary: Inefficient use of keySet() iterator in
                    StandardContext.java
           Product: Tomcat 7
           Version: trunk
          Platform: PC
        OS/Version: Windows XP
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Catalina
        AssignedTo: [email protected]
        ReportedBy: [email protected]


Created an attachment (id=25316)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=25316)
Change keySet() Iterator to values()

The loadOnStartup(Container[]) method of StandardContext loops on the keySet()
but actually only uses the key to get the values, so it would be more efficient
to use the values() Iterator.

According to the Javadoc, the keySet() and values() iterators generate the
entries in the same order (key ascending).

Findbugs detected another 6 such instances for which I can provide patches if
required.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to