Re: SocketPermission's implies() interesting behavior

2011-07-12 Thread Chris Hegarty



On 07/ 5/11 10:10 AM, Charles Lee wrote:

On 04/08/2011 04:47 PM, Chris Hegarty wrote:

On 04/ 7/11 04:53 PM, Neil Richards wrote:

On Wed, 2011-03-16 at 10:43 +, Neil Richards wrote:

On 11 March 2011 14:39, Chris Hegarty wrote:

As Michael (cc'ed) mentioned in an earlier mail, he is going to be
making
some significant changes in this area in the next week or two. He will
include this change with the other one, so their impact can be
considered
together, if that is ok?


That's fine by me, so long as the changeset that addresses this
problem is linked back to this conversation (so that one can track the
progress of the fix for this problem into the component, then master
repository).


Can you tell me if the changes headlined above have yet been made to the
code (and point me to the URL of the change) ?


These changes have not yet been integrated. I believe Michael will be
pushing them soon.

-Chris.



Thanks,
Neil


Hi Chris,

I have tested this issue with latest openjdk7 build (b147). The
"blabla"[1] test case still fails. Is it ok if I am going to give a
patch according the openjdk 8 code base?


I apologize for this. By the time the other changes were ready for 
SocketPermission we were past the lock down period for JDK7. I'll 
proceed with getting the changes we agreed upon [1] into JDK8.


-Chris.

[1] http://cr.openjdk.java.net/~chegar/7021280/webrev.01/webrev/



[1] http://bugs.sun.com/view_bug.do?bug_id=7021280



hg: jdk8/tl/jdk: 7058828: test/java/util/concurrent/Phaser/Arrive.java fails intermittently

2011-07-12 Thread chris . hegarty
Changeset: 549b7c3f0bdc
Author:dl
Date:  2011-07-12 15:23 +0100
URL:   http://hg.openjdk.java.net/jdk8/tl/jdk/rev/549b7c3f0bdc

7058828: test/java/util/concurrent/Phaser/Arrive.java fails intermittently
Reviewed-by: chegar

! test/java/util/concurrent/Phaser/Arrive.java



hg: jdk8/tl/jdk: 7022407: Spinning CPU in LocaleObjectCache.get()

2011-07-12 Thread naoto . sato
Changeset: 42fe05e54e69
Author:naoto
Date:  2011-07-12 10:28 -0700
URL:   http://hg.openjdk.java.net/jdk8/tl/jdk/rev/42fe05e54e69

7022407: Spinning CPU in LocaleObjectCache.get()
Reviewed-by: okutsu

! src/share/classes/sun/util/locale/LocaleObjectCache.java



Re: SocketPermission's implies() interesting behavior

2011-07-12 Thread Charles Lee

On 07/12/2011 07:03 PM, Chris Hegarty wrote:



On 07/ 5/11 10:10 AM, Charles Lee wrote:

On 04/08/2011 04:47 PM, Chris Hegarty wrote:

On 04/ 7/11 04:53 PM, Neil Richards wrote:

On Wed, 2011-03-16 at 10:43 +, Neil Richards wrote:
On 11 March 2011 14:39, Chris Hegarty 
wrote:

As Michael (cc'ed) mentioned in an earlier mail, he is going to be
making
some significant changes in this area in the next week or two. He 
will

include this change with the other one, so their impact can be
considered
together, if that is ok?


That's fine by me, so long as the changeset that addresses this
problem is linked back to this conversation (so that one can track 
the

progress of the fix for this problem into the component, then master
repository).


Can you tell me if the changes headlined above have yet been made 
to the

code (and point me to the URL of the change) ?


These changes have not yet been integrated. I believe Michael will be
pushing them soon.

-Chris.



Thanks,
Neil


Hi Chris,

I have tested this issue with latest openjdk7 build (b147). The
"blabla"[1] test case still fails. Is it ok if I am going to give a
patch according the openjdk 8 code base?


I apologize for this. By the time the other changes were ready for 
SocketPermission we were past the lock down period for JDK7. I'll 
proceed with getting the changes we agreed upon [1] into JDK8.


-Chris.

[1] http://cr.openjdk.java.net/~chegar/7021280/webrev.01/webrev/



[1] http://bugs.sun.com/view_bug.do?bug_id=7021280


No problem. Thank you very much, Chris.

--
Yours Charles