* The Roster implementation accessed the value of a Map entry, using a key that was retrieved from a keySet iterator. It is more efficient to use an iterator on the entrySet of the map, to avoid the Map.get(key) lookup. (thank you FindBugs);
* Applied Generics to a couple of methods.

git-svn-id: http://svn.igniterealtime.org/svn/repos/openfire/trunk@10835 b35dd754-fafc-0310-a699-88a17e54d16e