hg: jdk8/tl/jdk: 7164636: (prefs) Cleanup src/macosx/classes/java/util/prefs

2012-05-15 Thread kurchi . subhra . hazra
Changeset: 9a18e318f95a Author:khazra Date: 2012-05-15 11:51 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/9a18e318f95a 7164636: (prefs) Cleanup src/macosx/classes/java/util/prefs Summary: Remove rawtype usages and other code cleanup Reviewed-by: chegar, briangoetz ! src/ma

Re: Code Review Request: 7164636: (prefs) Cleanup src/macosx/classes/java/util/prefs

2012-05-15 Thread Chris Hegarty
On 14/05/12 21:00, Kurchi Hazra wrote: Thanks for the detailed explanation. I have removed the default initializations: http://cr.openjdk.java.net/~khazra/7164636/webrev.01/ This looks fine to me. -Chris. - Kurchi On 5/14/2012 10:07 AM, Chris Hegarty wrote: I always thought that removing

hg: jdk8/tl/jdk: 7164191: properties.putAll API may fail with ConcurrentModifcationException on multi-thread scenario

2012-05-15 Thread dmitriy . samersoff
Changeset: df33f5f750ec Author:dsamersoff Date: 2012-05-15 16:46 +0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/df33f5f750ec 7164191: properties.putAll API may fail with ConcurrentModifcationException on multi-thread scenario Reviewed-by: dholmes, sla Contributed-by: Deven Y