<Swing Dev> [OpenJDK 2D-Dev] [PATCH] SwingUtilities2.isLocalDisplay()

[Roman Kennke]

I need to repost my original patch for two reasons: 1. it doesn't apply
cleanly (only with some fuzz), 2. it also has this init-loop problem.
Find attached the correct version. I'd be happy to see it committed
ASAP.

/Roman

Am Mittwoch, den 10.09.2008, 21:04 +0200 schrieb Roman Kennke:
> Hi Dmitri and Oleg,
> 
> > > I understand than you do not want to add questionable code in 2D,
> > > but I do not like the idea to make this by adding questionable code
> > > to Swing. So before falliwing back to hack I'd suggest to thinkabout
> > 
> >    I'm not sure what you mean by "adding questionable code
> >    to Swing" - the code was already there. Now instead of
> >    directly calling sun internal API it will call a better
> >    defined internal API.
> 
> Yeah, I agree.
> 
> > > possible API, otherwise we have a good chance to keep this hack
> > > in Swing code forever :(
> > 
> >    Swing is part of the platform, and will always have to
> >    use some APIs in the implementation which are not available
> >    to external developers.
> > 
> >    I don't see a problem with that. In this particular case
> >    I really don't see any benefit in exposing this very
> >    platform-specific API to the developers.
> 
> Yeah. It looks like most (all?) Sun-specific code is in SwingUtilities2
> anyway, so this is more or less the porting interface for Swing for
> people who really need to port OpenJDK Swing to a different JDK. If it
> is a goal to support that, it might make sense to find all remaining
> uses of Sun-specific code in Swing and refactor that to also use
> SwingUtilities2. Or even better, create a separate interface and factory
> for implementations of that interface that would be provided by the
> developer who ports Swing. But that seems a little over the top right
> now. I also suggest to get the original patch through.
> 
> /Roman
-- 
http://kennke.org/blog/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch.txt
Type: text/x-patch
Size: 4368 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20080910/be736a0a/patch.txt>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Dies ist ein digital signierter Nachrichtenteil
URL: <http://mail.openjdk.java.net/pipermail/swing-dev/attachments/20080910/be736a0a/signature.asc>