On 10/08/2012 17:10, Chris Hegarty wrote:
Christian,

I finally got back to this.

Since init() is only ever called from the class initializer it is implicitly single threaded, so we only need to add the synchronized keyword to getSystemProxy().

I also added a test for completeness. Here is the final webrev that I intend to push ( listing 'Christian Schulte <c...@schulte.it>' as the contributor ).

http://cr.openjdk.java.net/~chegar/7188755/webrev.00/webrev/

-Chris.
The change looks good to me. The test is fine took although I assume if any of the threads through an exception then it will not cause the test to fail (but of course if they crash it will).

-Alan

Reply via email to