<AWT Dev> numAppContexts in AppContext modified in not-thread-safe way.

Clemens Eisserer linuxhippy at gmail.com
Thu Aug 18 15:02:39 PDT 2011


Hi Mario,

It looks good to me, and I don't expect any issues after this patch.
> The only thing is that you should check the settings for the editor, I
> think you still use a mix of tabs and spaces :)
>

Hmm, I should really check those eclipse settings ^^


> Also,
>
> public final static AppContext getAppContext()
>
> Doesn't seem to be thread safe by nature, I don't know if this is part
> of the issue you found, so maybe you should consider to synchronize
> this as well.
>

Could be I got this wrong ... but isn't the idea behind getAppContext() that
it can be called  from any thread, and get the AppContext(if there is any)
back?


- Clemens
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20110819/c570894b/attachment.html 


More information about the awt-dev mailing list