RFR: 8035618 : Four api/org_omg/CORBA TCK tests fail under plugin only

Seán Coffey sean.coffey at oracle.com
Thu Feb 27 09:38:58 UTC 2014


On 27/02/2014 07:45, Alan Bateman wrote:
> On 26/02/2014 21:47, Seán Coffey wrote:
>> Good points Alan. Changes uploaded here :
>> http://cr.openjdk.java.net/~coffeys/webrev.8035618.v2/webrev/
> This looks much better. Are you sure that pmLock is needed? Is there 
> any reason not to synchronize on the AppContext when adding the 
> PresentationManager key?
I was conscious of possible contention coming on the AppContext object 
if syncing on it. I figured a static lock should work (given that we had 
zero synchronization in the old code) - Apologies for not including your 
name in final push. It's been pointed out to me that I should include 
all reviewers. I thought I could only include reviewers once I had an 'ok'.
>
> Are you planning to propose a fix for jdk9/dev too? It would be good 
> to eliminate the direct dependency on sun.awt.AppContext while doing 
> that.
Yes - I'll get the direct AppContext reference removed for JDK 9 fix via 
JavaAWTAccess but will need to re-examine the code to see if we can 
avoid sun.awt code.

regards,
Sean.

>
> -Alan




More information about the core-libs-dev mailing list