<AWT Dev> Reviewer needed - awt/FontClass/CreateFont/DeleteFont test correction

Dr Andrew John Hughes ahughes at redhat.com
Thu Jan 13 10:05:18 PST 2011


On 09:56 Thu 13 Jan     , Joe Darcy wrote:
> On 1/13/2011 9:12 AM, Dr Andrew John Hughes wrote:
> > On 11:50 Thu 13 Jan     , Coleen Phillimore wrote:
> >>>> We're getting close to the stable hs20 branch.  I don't know how or when
> >>>> it's imported to OpenJDK6 though so I shouldn't have tried to answer the
> >>>> question.
> >>>>
> >>> I usually do the work to import new versions of HotSpot into OpenJDK6 and Joe
> >>> approves it, as with all OpenJDK6 patches.  I've never really understood why
> >>> we need to have both a stable branch AND an OpenJDK6 HotSpot tree, given they
> >>> end up being essentially the same, but that's the way it is.
> >>>
> >>> We're about to make a new release of OpenJDK6 (b21) with HotSpot 19.  We've
> >>> obviously don't want to import an entire new HotSpot at this juncture, but will
> >>> probably do so for b22 once the HotSpot tree becomes available.
> >>>
> >>>   From what you say, my understanding is that 6938627 causes a regression from previous
> >>> HotSpot behaviour and your patch, 7009828, fixes that regression.  As OpenJDK6 has
> >>> 6938627, would it not make sense to backport 7009828 as well to fix the regression?
> >>> If not, what am I missing here?  The patch doesn't seem hugely intrusive, and it reverts
> >>> back to using the previous expected behaviour of /tmp over java.io.tmpdir.
> >>>
> >> I didn't know all of this.
> > I finally realised this, hence the explanation :-)
> >
> >> Yes, I think you want to backport this
> >> change.
> > Thanks.  I now just need the thumbs up from Joe as well.  Joe?
> 
> I've taken a look at the patch and I approve 7009828 to be backported to 
> OpenJDK 6 b21.
> 

Done: http://hg.openjdk.java.net/jdk6/jdk6/hotspot/rev/2c8e1acf0433

> Thanks,
> 
> -Joe
> 

Thanks,
-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Support Free Java!
Contribute to GNU Classpath and IcedTea
http://www.gnu.org/software/classpath
http://icedtea.classpath.org
PGP Key: 94EFD9D8 (http://subkeys.pgp.net)
Fingerprint = F8EF F1EA 401E 2E60 15FA  7927 142C 2591 94EF D9D8



More information about the awt-dev mailing list