RFR : 8016446 : (m) Add override forEach/replaceAll to HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap (original) (raw)
Paul Sandoz paul.sandoz at oracle.com
Wed Jun 19 10:49:12 UTC 2013
- Previous message: RFR : 8016446 : (m) Add override forEach/replaceAll to HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap
- Next message: RFR : 8016446 : (m) Add override forEach/replaceAll to HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On Jun 19, 2013, at 11:53 AM, Remi Forax <forax at 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.
- Previous message: RFR : 8016446 : (m) Add override forEach/replaceAll to HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap
- Next message: RFR : 8016446 : (m) Add override forEach/replaceAll to HashMap, Hashtable, IdentityHashMap, WeakHashMap, TreeMap
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]