On Jun 19, 2013, at 11:53 AM, Remi Forax <fo...@univ-mlv.fr> wrote: >> This is another little oddity in Map.forEach: >> >> try { >> k = entry.getKey(); >> v = entry.getValue(); >> } catch(IllegalStateException ise) { >> throw new ConcurrentModificationException(ise); >> } >> >> I would presume the entries from CconcurrentMap.entrySet would not throw ISEs > > Does it worth to override forEach in ConcurrentMap for that ? >
Dunno, i am struggling to work up a strong opinion :-) It seems wrong for ConcurrentMap impls but i just cannot see why any such impls would throw ISEs. Paul.