Code Review Request: 7160252: (prefs) NodeAddedEvent was not delivered when new node add when new Node

Kurchi Subhra Hazra kurchi.subhra.hazra at
Thu Jul 12 14:55:31 UTC 2012

I used a new workspace and missed adding it to mercurial. The test 
remains the same,
I'll push it after adding the test.

- Kurchi

On 7/12/12 12:16 AM, Alan Bateman wrote:
> On 12/07/2012 00:45, Kurchi Hazra wrote:
>> On 7/11/12 4:24 PM, Chris Hegarty wrote:
>>> On 12 Jul 2012, at 00:15, Kurchi 
>>> Hazra<kurchi.subhra.hazra at>  wrote:
>>>> Thanks for the review Alan. Updated webrev:
>>> Looks fine.
>>> Trivially, is there an opportunity to make any fields final since 
>>> initFields is replaced with a constructor?
>> Thanks for pointing that out. How about: 
> Looks fine except that the test case is missing from latest webrev. 
> Assuming that test/java/util/prefs/ hasn't 
> changed from the original webrev then I think you are all set to push.
> -Alan.

More information about the core-libs-dev mailing list