Why is this so written?

[gaohoward]

In JBM 14 SimpleConnectionManager.cleanupForSessionID((String jmsSessionID),

 .....
 ConnectionEndpoint conn = null;

 for (Map.Entry<String, ConnectionEndpoint> entry: endpoints.entrySet())
 {
 if (entry.getKey().equals(jmsSessionID))
 {
 conn = entry.getValue();

 break;
 }
 }
 ........

Why don't use

 .....
 ConnectionEndpoint conn = null;

 conn = endpoints.get(jmsSessionID);
 ........

instead of the for loop?

[clebert.suconic]

This was done during https://jira.jboss.org/jira/browse/JBMESSAGING-1262:

We had a couple of other similar methods that used to do loops for other reasons. That method was probably a modification of those methods and that's probably the only reason. So I would say you could use the .get instead of the loop on that case.