Re: [OpenJDK 2D-Dev] [PATCH] SwingUtilities2.isLocalDisplay()

2008-09-10 Thread Dmitri Trembovetski
Yep, I see. I didn't look at the file itself, only at the patch, and got confused. Thanks, Dmitri Roman Kennke wrote: Hi Dmitri, 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 attac

Re: [OpenJDK 2D-Dev] [PATCH] SwingUtilities2.isLocalDisplay()

2008-09-10 Thread Roman Kennke
Hi Dmitri, > > 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. > >I have a question about the fix: > --- a/src/so

Re: [OpenJDK 2D-Dev] [PATCH] SwingUtilities2.isLocalDisplay()

2008-09-10 Thread Mario Torre
Il giorno mer, 10/09/2008 alle 15.45 -0700, Dmitri Trembovetski ha scritto: > --- a/src/solaris/native/sun/awt/fontpath.c Thu Aug 07 09:42:31 2008 -0700 > +++ b/src/solaris/native/sun/awt/fontpath.c Wed Sep 10 23:52:15 2008 +0200 > @@ -156,7 +156,7 @@ > > isLocal = JNU_CallStaticMethodB

Re: [OpenJDK 2D-Dev] [PATCH] SwingUtilities2.isLocalDisplay()

2008-09-10 Thread Dmitri Trembovetski
Roman Kennke wrote: 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. I have a question about the fix: --- a/src/solaris/nati

Re: [OpenJDK 2D-Dev] [PATCH] SwingUtilities2.isLocalDisplay()

2008-09-10 Thread 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

Re: [OpenJDK 2D-Dev] [PATCH] SwingUtilities2.isLocalDisplay()

2008-09-10 Thread 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

Re: [OpenJDK 2D-Dev] [PATCH] SwingUtilities2.isLocalDisplay()

2008-09-10 Thread Dmitri Trembovetski
Hi Oleg, Oleg Sukhodolsky wrote: 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 c

hg: jdk7/swing/jdk: 6587742: filling half of a JSlider's track is no longer optional

2008-09-10 Thread pavel . porvatov
Changeset: 32fb1f4f40b8 Author:rupashka Date: 2008-09-10 19:16 +0400 URL: http://hg.openjdk.java.net/jdk7/swing/jdk/rev/32fb1f4f40b8 6587742: filling half of a JSlider's track is no longer optional Summary: now OceanTheme uses the JSlider.isFilled property like other themes Reviewed

Re: [OpenJDK 2D-Dev] [PATCH] SwingUtilities2.isLocalDisplay()

2008-09-10 Thread Martin Buchholz
It's very hard to get these kinds of UI issues right. I once tried to configure my xemacs to have less eye candy when talking to a "slow" X server by actually timing some operation that requires a round trip. Note that a method like isLocalDisplay would have the wrong semantics for me, because I