<AWT Dev> [PATCH] ComponentAccessor

Oleg Sukhodolsky son.two at gmail.com
Fri Jan 16 06:38:06 PST 2009


Hi Roman,

I'd synchronized setting/getting of access field.  And (perhaps) I'd only allow
set this field once.

Regards, Oleg.

On Fri, Jan 16, 2009 at 5:14 PM, Roman Kennke <roman at kennke.org> wrote:
> Today I found the aweful (sorry) ComponentAccessor. I quickly rewrote it
> using what I call the 'friends for Java' pattern (described here:
>
> http://kennke.org/blog/2009/01/08/friends-for-java/
>
> ).
>
> This has a couple of advantages:
>
> - no reflection (I don't like reflection)
> - compile-time safety
> - better IDE support (i.e. I was trying to trace
> ComponentPeer.getGraphicsConfiguration() back today, and only got to
> Window.resetGC(), because this one is only called via reflection in
> ComponentAccessor).
> - I'd guess that performance is slighly better, but that probably
> doesn't matter much in these cases.
>
> Do you think that should go into OpenJDK?
>
> Webrev: http://kennke.org/~roman/componentaccess/webrev/
>
> Cheers, Roman
>
> --
> http://kennke.org/blog/
>



More information about the awt-dev mailing list