AWT Dev hg: jdk8/awt/jdk: 7080700: numAppContexts is accessed in non-threadsafe way in sun.awt.AppContext

2012-04-16 Thread anthony . petrov
Changeset: 7b0ed75abef4 Author:anthony Date: 2012-04-16 18:36 +0400 URL: http://hg.openjdk.java.net/jdk8/awt/jdk/rev/7b0ed75abef4 7080700: numAppContexts is accessed in non-threadsafe way in sun.awt.AppContext Summary: Replace a volatile integer with an AtomicInteger for

Re: AWT Dev numAppContexts in AppContext modified in not-thread-safe way.

2012-04-16 Thread Anthony Petrov
On 04/13/12 23:13, Clemens Eisserer wrote: I'm sorry for such a long delay. I'll push the fix next week. Thanks a lot :-) Here it is: http://hg.openjdk.java.net/jdk8/awt/jdk/rev/7b0ed75abef4 I forgot to add the Contributed-by: line to the changeset and didn't substitute the author of the

Re: AWT Dev [8] Request for review: 7080109 Dialog.show() lacks doPrivileged() to access system event queue

2012-04-16 Thread Sergey Bylokhov
Hi, Artem. Thanks for view. Scope was changed. Here is an updated version: http://cr.openjdk.java.net/~serb/7080109/webrev.01/ 05.04.2012 14:46, Artem Ananiev wrote: Hi, Sergey, to limit the scope of the added doPrivileged() block, I would rewrite the code this way: secondaryLoop =

AWT Dev [8] Request for review for 7161109: [macosx] JCK AWT interactive test DnDTextDropTest fails on MacOS

2012-04-16 Thread Alexander Zuev
Hello, please review my fix for CR 7161109: [macosx] JCK AWT interactive test DnDTextDropTest fails on MacOS The fix is just a forward-port of fix for CR 7124262 which was not synced up into jdk8 workspace. Bug description: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7161109

Re: AWT Dev [8] Request for review for 7161109: [macosx] JCK AWT interactive test DnDTextDropTest fails on MacOS

2012-04-16 Thread Sergey Bylokhov
Hi, Alexander. Fix looks good. 16.04.2012 22:12, Alexander Zuev wrote Hello, please review my fix for CR 7161109: [macosx] JCK AWT interactive test DnDTextDropTest fails on MacOS The fix is just a forward-port of fix for CR 7124262 which was not synced up into jdk8 workspace. Bug