Looks good.
Jennifer
On 7/22/2016 1:02 PM, Phil Race wrote:
https://bugs.openjdk.java.net/browse/JDK-8162429
http://cr.openjdk.java.net/~prr/8162429/
Clean up some now obsolete font code.
-phil.
Looks good.
Jennifer
On 05/05/2016 06:33 AM, Sergey Bylokhov wrote:
Looks fine.
On 29.04.16 22:35, Phil Race wrote:
Bug : https://bugs.openjdk.java.net/browse/JDK-8155682
Webrev: http://cr.openjdk.java.net/~prr/8155682/
None of these flags have been referenced since (I think) the current
D3D
Hi Prasanta,
It looks to me that we missed to check (flags & PD_NOSELECTION) in
setNativeAttributes that's why we are setting SunPageSelection attribute
when we shouldn't. I think that is where we should put the fix.
Jennifer
On 04/15/2016 03:34 AM, prasanta sadhukhan wrote:
Hi Phil,
On 4
Looks good.
Jennifer
On 04/04/2016 03:33 PM, Phil Race wrote:
https://bugs.openjdk.java.net/browse/JDK-8134986
When converted from a HashSet to a HashMap contains() should have
changed to containsKey(). It does not lead to duplicate fonts because
it is caught later on but it might lead to waste
Vote: Yes
- Jennifer
On 3/17/2016 3:49 PM, Philip Race wrote:
I hereby nominate Alexander Scherbatiy to membership in the 2D group.
Alexander Scherbatiy is a current member of the Swing group
and has contributed almost 200 changesets to OpenJDK :-
http://hg.openjdk.java.net/jdk9/dev/jdk/log?re
Hi Prasanta,
Looks good.
Jennifer
On 03/08/2016 02:26 AM, prasanta sadhukhan wrote:
Hi Phil,
It's my mistake. There's no need to copy as we are not passing the
supportedDocFlavors outside the class. I was influenced by the lines
before this where copyFlavors was copied from supportedDocFlavo
Looks good.
Jennifer
On 12/16/2015 01:12 PM, Phil Race wrote:
http://cr.openjdk.java.net/~prr/8145584/
https://bugs.openjdk.java.net/browse/JDK-8145584
The bug is that the code that tries to measure where the bounds
of the rendering is not applying the same rounding of the rendering
position a
Looks good.
Jennifer
On 12/07/2015 10:03 AM, Phil Race wrote:
Fixed : http://cr.openjdk.java.net/~prr/8137106.1/
-phil.
On 12/07/2015 09:21 AM, Philip Race wrote:
Yeah .. that should have been true. I'll upload a fixed patch.
-phil.
On 12/7/15, 8:14 AM, Sergey Bylokhov wrote:
Hi, Phil.
It
Vote: Yes
- Jennifer
On 12/07/2015 12:29 PM, Phil Race wrote:
I hereby nominate Sergey Bylokhov to membership in the 2D group.
Sergey has been a de facto member of the group for a long time and
has contributed numerous substantial fixes and provided many code reviews.
Votes are due by 21st De
Looks good. Approved.
Jennifer
On 11/13/2015 2:26 PM, Philip Race wrote:
I have managed to create a regression test that I think I can trust on
OS X
Updated webrev :- http://cr.openjdk.java.net/~prr/7162125.1
-phil.
On 11/12/15, 9:48 PM, Philip Race wrote:
Bug: https://bugs.openjdk.java.ne
Looks good. Approved.
Jennifer
- Original Message -
From: prasanta.sadhuk...@oracle.com
To: philip.r...@oracle.com
Cc: 2d-dev@openjdk.java.net, rajeev.cham...@oracle.com
Sent: Friday, November 13, 2015 3:36:15 AM GMT -08:00 US/Canada Pacific
Subject: Re: [OpenJDK 2D-Dev] [9] RFR JD
Approved.
- Jennifer
On 10/20/2015 01:47 PM, Phil Race wrote:
On 10/20/2015 12:32 PM, Jennifer Godinez wrote:
Hi Phil,
I assume you've run all the manual regression tests on text?
I would not say all. But I have run the ones that relate
to text size/position/transform.
The copyright
Hi Phil,
I assume you've run all the manual regression tests on text? The
copyright year in the regression test needs updating. Otherwise, looks
good.
Jennifer
On 10/16/2015 04:16 PM, Phil Race wrote:
https://bugs.openjdk.java.net/browse/JDK-8132890
http://cr.openjdk.java.net/~prr/8132890
Looks good.
Jennifer
On 09/23/2015 02:18 PM, Phil Race wrote:
https://bugs.openjdk.java.net/browse/JDK-8137059
http://cr.openjdk.java.net/~prr/8137059/
This is a follow-up to https://bugs.openjdk.java.net/browse/JDK-8136783
This new change cleans up the demos and one additional JDK source file
Looks good. Don't we need regression test though?
- Jennifer
On 09/23/2015 07:43 AM, Phil Race wrote:
Any takers ?
-phil.
On 9/18/15 1:00 PM, Phil Race wrote:
https://bugs.openjdk.java.net/browse/JDK-8133309
http://cr.openjdk.java.net/~prr/8133309/
Add Segoe UI Symbol (available on Vista and
Hi Phil,
Looks good.
Jennifer
On 8/12/15 2:01 PM, Phil Race wrote:
Hi all,
Still looking for any reviewers for this 7 day old review ...
-phil.
On 8/5/15 12:15 PM, Phil Race wrote:
https://bugs.openjdk.java.net/browse/JDK-8132850
http://cr.openjdk.java.net/~prr/8132850/
The bug is an Arra
Looks good.
Jennifer
On 04/29/2015 10:43 AM, Phil Race wrote:
https://bugs.openjdk.java.net/browse/JDK-8079067
http://cr.openjdk.java.net/~prr/8079067/
Just deleting unused code.
-phil.
Looks good.
- Jennifer
On 04/17/2015 12:27 PM, Phil Race wrote:
http://cr.openjdk.java.net/~prr/8076979/
https://bugs.openjdk.java.net/browse/JDK-8076979
-phil.
Looks good.
Jennifer
On 11/06/2014 10:50 AM, Phil Race wrote:
Any takers ?
-phil.
On 10/30/2014 11:14 AM, Phil Race wrote:
https://bugs.openjdk.java.net/browse/JDK-8062163
http://cr.openjdk.java.net/~prr/8062163/
Fixes a test that currently fails because of a JDK9 b33 change
in the implement
Hi Phil,
The change looks fine.
Jennifer
On 10/27/2014 02:49 PM, Phil Race wrote:
I can add othervm but each VM invocation slows down the
overall testing time and I'd like to think this only will be run on
versions that do not have the bug.
I added the custom time out code so that I was not re
Hi Phil,
Looks good.
Jennifer
On 10/17/2014 2:19 PM, Phil Race wrote:
http://cr.openjdk.java.net/~prr/8061392/
https://bugs.openjdk.java.net/browse/JDK-8061392
A check for null was missing
This was partly to blame for a JavaFX printing bug.
-phil.
Looks good.
- Jennifer
On 10/16/2014 01:32 PM, Phil Race wrote:
http://cr.openjdk.java.net/~prr/8061267/
https://bugs.openjdk.java.net/browse/JDK-8061267
An apparently old issue that the specified page ranges don't get
displayed
because we forgot to set the flag bit ...
-phil.
Looks good.
- Jennifer
On 10/9/2014 12:09 PM, Phil Race wrote:
Apparently jigsaw finds it inconvenient that the service provider
class names
are different for Windows vs the rest of the platforms, so this fix
basically
just renames both classes to PrintServiceLookupProvider.
I did jprt build
Looks good.
- Jennifer
On 10/3/2014 12:22 PM, Phil Race wrote:
http://cr.openjdk.java.net/~prr/8059219/
https://bugs.openjdk.java.net/browse/JDK-8059219
Straightforward fix for a very old bug ..
-phil.
Everything looks good.
Jennifer
On 09/03/2014 02:54 PM, Phil Race wrote:
No actual change. Perhaps there was white space ..
-phil.
On 9/3/2014 2:41 PM, Jennifer Godinez wrote:
Hi Phil,
Is there any change to the file cmsgats.c? This sdiff link, shows blank.
http://cr.openjdk.java.net/~prr
Hi Phil,
Is there any change to the file cmsgats.c? This sdiff link, shows blank.
http://cr.openjdk.java.net/~prr/8056122/src/java.desktop/share/native/liblcms/cmscgats.c.sdiff.html
Jennifer
On 08/26/2014 03:26 PM, Phil Race wrote:
Andrew (and at least one other brave soul), please review :-
Looks good.
Jennifer
On 7/11/2014 11:24 AM, Phil Race wrote:
http://cr.openjdk.java.net/~prr/8048583.1/
updated version to fix the incorrect usage of w and h instead of dw
and dh
spotted by Andrew.
-phil.
On 06/27/2014 01:40 PM, Phil Race wrote:
Bug : https://bugs.openjdk.java.net/browse/J
Looks good.
- Jennifer
On 7/10/2014 2:49 PM, Phil Race wrote:
Jennifer,
Please review/check http://cr.openjdk.java.net/~prr/7067052.8u/
This is a 100% identical backport of your JDK 9 fix
https://bugs.openjdk.java.net/browse/JDK-7067052
http://hg.openjdk.java.net/jdk9/client/jdk/rev/327e816d4
Looks good. Approved.
- Jennifer
On 06/27/2014 01:40 PM, Phil Race wrote:
Bug : https://bugs.openjdk.java.net/browse/JDK-8048583
Fix : http://cr.openjdk.java.net/~prr/8048583/
As described in the bug the basic problem is that we associate
a printer defined media with an inappropriate "standar
Looks good.
- Jennifer
On 6/3/14 11:42 AM, Joe Darcy wrote:
On 06/03/2014 11:34 AM, Phil Race wrote:
Both of these came in via an IBM contributed fix :-
http://mail.openjdk.java.net/pipermail/2d-dev/2011-October/002206.html
He [it appears] wanted to be very sure the throw was executed but it
s
Looks good.
Jennifer
On 05/23/2014 02:50 PM, Phil Race wrote:
http://cr.openjdk.java.net/~prr/8035301/
https://bugs.openjdk.java.net/browse/JDK-8035301
Builds cleanly. Client tests run.
-phil.
Checked. They are identical.
- Jennifer
On 5/22/2014 2:45 PM, Phil Race wrote:
On 5/22/14 2:39 PM, Phil Race wrote:
JDK 9 changeset :
http://hg.openjdk.java.net/jdk9/client/jdk/rev/006f772bbc7e
Sorry copy paste problems that should have been
http://hg.openjdk.java.net/jdk9/client/jdk/rev/be33
Looks good.
- Jennifer
On 5/19/2014 9:35 AM, Joe Darcy wrote:
*ping*
Any other comments?
Thanks,
-Joe
On 05/15/2014 06:06 PM, Joe Darcy wrote:
Hi Henry,
On 05/15/2014 03:46 PM, Henry Jen wrote:
On 05/15/2014 12:07 PM, Joe Darcy wrote:
Hello,
Please review these change to fix
JDK-8
Looks good.
Jennifer
On 4/4/2014 12:01 PM, Phil Race wrote:
Jennifer & Sergey,
Please check the identical JDK8u20 backport of the fix for the
JNI corrrectness fix in WPrinterJob.cpp that you previously approved
for JDK 9
Bug link: https://bugs.openjdk.java.net/browse/JDK- 8035569
Synopsis: [p
Approved.
Jennifer
On 4/4/2014 11:29 AM, Phil Race wrote:
Jennifer & Andrew,
Please check the identical JDK8u20 backport of the fix for the
(theoretical) memory
leak in awt_Font.c that you approved for JDK9
Bug link: https://bugs.openjdk.java.net/browse/JDK-8031095
Synopsis: [Parfait] warning
Looks good.
- Jennifer
On 04/03/2014 09:13 AM, Sergey Bylokhov wrote:
Hi, Phil.
looks good.
On 4/2/14 1:34 AM, Phil Race wrote:
Sergey & Jennifer,
Please review 3 mostly windows related JNI correctness backports to
jdk8u20 that you previously approved for JDK 9
Bug link: https://bugs.openjdk
Looks good.
- Jennifer
On 03/29/2014 03:00 PM, Phil Race wrote:
Sergey & Jennifer,
Please review/check 6 MacOSX backports of JNI correctness fixes of
changes
you previously reviewed& approved for JDK 9.
They are all one per-file and exact copies of what was pushed into
jdk9/client.
I have t
Looks good.
- Jennifer
On 3/28/2014 2:12 PM, Phil Race wrote:
Please sanity check this backport ..
Code : http://cr.openjdk.java.net/~prr/8029008.8u20/
Bug https://bugs.openjdk.java.net/browse/JDK-8029008
JDK 9 changeset :-
http://hg.openjdk.java.net/jdk9/client/jdk/rev/f0f8960831c1
Note th
Approved.
- Jennifer
On 03/28/2014 11:30 AM, Phil Race wrote:
Jennifer,
Can you pls review this fix for
https://bugs.openjdk.java.net/browse/JDK-8037506
[javadoc] broken link in java.awt.geom.Line2D.java
The suggested fix came from the pubs team.
-phil.
hg diff src/share/classes/java/awt/g
Looks good.
- Jennifer
On 02/11/2014 12:36 PM, Phil Race wrote:
The closed source library dcpr (ductus) needs to reference a JNU function
So an update is needed to a build file :
This needs a quick review as I (ahem) pushed the closed part but
not this ... not of relevance to open source users
Looks good. Approved.
- Jennifer
On 2/9/2014 9:12 PM, Joe Darcy wrote:
Hello,
Please review the proposed fix for
JDK-8034050: Fix serial lint warnings in javax.print
http://cr.openjdk.java.net/~darcy/8034050.0/
I checked that the added serialVerionUID values match the serialver
com
Approved.
Jennifer
On 02/06/2014 02:31 PM, Phil Race wrote:
http://cr.openjdk.java.net/~prr/8032693/
This is a fix for https://bugs.openjdk.java.net/browse/JDK-8032693
It was introduced by the JDK 8 b115 fix for
https://bugs.openjdk.java.net/browse/JDK-8022536
The root of the issue is that
Changeset: 723bcc68738b
Author:jgodinez
Date: 2013-11-26 10:24 -0800
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/723bcc68738b
8028584: sun.net.www.protocol.file.FileURLConnection cannot be cast to
java.net.HttpURLConnection
Reviewed-by: bae, prr
! src/solaris/classes/sun/print
http://cr.openjdk.java.net/~jgodinez/8028584/webrev.00/
ClassCastException occurs when device uri is a file. Added a check for
file and additional safeguard against cast exception.
Jennifer
Changeset: 2a3e21fe9d0d
Author:jgodinez
Date: 2013-10-21 13:18 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/2a3e21fe9d0d
8026951: Fix for 8025988 breaks jdk build on windows
Reviewed-by: prr, jchen
! src/share/classes/sun/print/RasterPrinterJob.java
Changeset: bcf8e9a59968
Author:jgodinez
Date: 2013-10-18 15:30 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/bcf8e9a59968
8025988: [macosx] Attribute settings don't work for JobAttributes range
8025990: [macosx] Attribute settings don't work for JobAttributes
setOrientation
Changeset: c9c945cea665
Author:jgodinez
Date: 2013-10-15 14:24 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/c9c945cea665
8015586: [macosx] Test closed/java/awt/print/PrinterJob/PrintToDir.java fails
on MacOSX
Reviewed-by: prr, jchen
! src/macosx/classes/sun/lwawt/macosx/C
Changeset: 0df5cda89a50
Author:jchen
Date: 2013-10-15 14:16 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/0df5cda89a50
8025429: [parfait] warnings from b107 for sun.java2d.cmm: JNI exception pending
Reviewed-by: prr, bae
! src/share/native/sun/java2d/cmm/lcms/LCMS.c
Changeset: b164c8eb1295
Author:jgodinez
Date: 2013-10-14 09:15 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/b164c8eb1295
8022536: closed/javax/print/TextFlavorTest.java fails
Reviewed-by: prr, jchen
! src/solaris/classes/sun/print/CUPSPrinter.java
! src/solaris/classes/sun
Changeset: 8fd757f31470
Author:jchen
Date: 2013-10-04 16:17 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/8fd757f31470
8025940: Windows build fails after the fix for 8025280
Reviewed-by: prr, jgodinez
! src/share/native/sun/java2d/loops/MaskBlit.c
Changeset: 0ed939dc4230
Author:jchen
Date: 2013-10-03 13:49 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/0ed939dc4230
8025294: [parfait] JNI-related warnings from b107 for
jdk.src.solaris.native.sun.java2d.x11
Reviewed-by: prr, jgodinez
! src/solaris/native/sun/java2d/x11
Changeset: d37594b689ce
Author:jchen
Date: 2013-10-03 13:41 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/d37594b689ce
8025664: [parfait] warnings from b62 for jdk.src.share.native.sun.font
Reviewed-by: prr, jgodinez
! src/share/native/sun/font/freetypeScaler.c
Changeset: 3c1b13ad0677
Author:jchen
Date: 2013-10-03 13:35 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/3c1b13ad0677
8025309: [parfait] JNI-related warnings from b107 for
jdk.src.share.native.sun.java2d.pipe
Reviewed-by: prr, jgodinez
! src/share/native/sun/java2d/pipe/B
Changeset: e88d39b110dd
Author:jchen
Date: 2013-10-03 13:26 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/e88d39b110dd
8025480: [parfait] "JNI exception pending" warnings from b107 for
jdk.src.share.native.sun.java2d
Reviewed-by: prr, jgodinez
! src/share/native/sun/java2d
Changeset: 2f11a00279ec
Author:jchen
Date: 2013-10-03 13:16 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/2f11a00279ec
8025280: [parfait] warnings from b107 for
jdk.src.share.native.sun.java2d.loops: JNI exception pending, JNI critical
region violation
Reviewed-by: prr, jg
Approved.
Jennifer
On 8/30/2013 4:51 AM, Andrew Brygin wrote:
Hello,
could you please review a fix for CR 8022632?
Bug: http://bugs.sun.com/view_bug.do?bug_id=8022632
Webrev: http://cr.openjdk.java.net/~bae/8022632/webrev.00/
Suggested fix just makes WBMP spi canDecodeInput() method
more reli
Approved.
Jennifer
On 10/2/2013 10:56 AM, Phil Race wrote:
This fix : http://cr.openjdk.java.net/~prr/8025837/
undoes some extraneous changes here :
http://cr.openjdk.java.net/~ceisserer/8007386/webrev.04/
they apparently crept into that updated final webrev as
they were not in http://cr.openj
Changeset: 8a041011b6e6
Author:jgodinez
Date: 2013-09-27 13:04 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/8a041011b6e6
6870661: Setting a custom PrintService on a PrinterJob leads to a
PrinterException
Reviewed-by: prr, jgodinez
Contributed-by: patr...@reini.net
! src/w
all tests and helper classes have been moved to "
test/java/awt/print/PrinterJob/CustomPrintService"
Cheers Patrick
On 09/26/2013 10:41 PM, Jennifer Godinez wrote:
Hi Patrick,
See inline comments.
On 9/26/13 12:20 PM, Patrick Reinhart wrote:
1) What would be the correct subdirectory for
shown.
Unfortunately I did not found a manual test for a non applet test.
In that case, you can just use an automated test.
Jennifer
Cheers Patrick
On 09/26/2013 07:19 PM, Jennifer Godinez wrote:
Hi Patrick,
Fix looks fine but can you send me a new webrev with the corrected
typo and move the
Hi Patrick,
Fix looks fine but can you send me a new webrev with the corrected typo
and move the regression tests to a new subdirectory? Also, the manual
test is reporting failure for "cancel" which I think should not be the
case. You may look at other manual tests on how they do this. On t
Hi Patrick,
Sorry I've been busy with other things. I will review it this week.
Jennifer
On 09/23/2013 01:15 PM, Patrick Reinhart wrote:
Hi Jennifer,
Did you had the time to look into it jet?
Cheers Patrick
Am 16.09.13 22:19, schrieb Patrick Reinhart:
Hi Phil,
Thanks for the quick respo
Changeset: 7da90f645a63
Author:jgodinez
Date: 2013-08-30 09:10 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/7da90f645a63
8017469: [macosx] Printing problem using ja and zh_CN locales
Reviewed-by: prr, jchen
! src/macosx/native/sun/awt/CTextPipe.m
Changeset: 0c950b2be7ab
Author:jgodinez
Date: 2013-08-19 11:21 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/0c950b2be7ab
8022241: [macosx] [PIT] lookupPrintServices() returns one too long array
Reviewed-by: prr, jchen
! src/solaris/classes/sun/print/UnixPrintServiceLookup.
Hi Dan,
Fix looks good.
Jennifer
On 8/12/2013 1:48 PM, Jennifer Godinez wrote:
Hi Dan,
I will run some tests and let you know the results.
Thank you.
Jennifer
On 8/8/2013 1:14 PM, Dan Xu wrote:
Hi Phil and Jennifer,
Thanks for your suggestions!
I have updated the changes at
http
Changeset: 2722f4000b65
Author:jgodinez
Date: 2013-08-15 11:56 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/2722f4000b65
8023045: [MacOSX] PrinterIOException when printing a JComponent
Reviewed-by: bae, jchen
! src/share/classes/sun/print/PSPrinterJob.java
Hi Dan,
I will run some tests and let you know the results.
Thank you.
Jennifer
On 8/8/2013 1:14 PM, Dan Xu wrote:
Hi Phil and Jennifer,
Thanks for your suggestions!
I have updated the changes at
http://cr.openjdk.java.net/~dxu/8017109/webrev.00/. Please help review
and sponsor it.
-Dan
Changeset: c3b91dc2504a
Author:jgodinez
Date: 2013-08-06 14:22 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/c3b91dc2504a
8021583: test/javax/print/autosense/PrintAutoSenseData.java throwing NPE
Reviewed-by: jchen, prr
! src/solaris/classes/sun/print/UnixPrintJob.java
! src
Changeset: 046025f78ea8
Author:jgodinez
Date: 2013-07-30 13:01 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/046025f78ea8
8021835: Fix for 8016343 will not compile on Windows.
Reviewed-by: jchen, prr
! src/share/classes/sun/print/PSPrinterJob.java
Changeset: 56c6f9a9653d
Author:jgodinez
Date: 2013-07-26 15:25 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/56c6f9a9653d
8016343: [macosx] Print job goes to default printer regardless of chosen printer
Reviewed-by: jchen, prr
! src/share/classes/sun/print/PSPrinterJob.java
Changeset: 467a0c21790b
Author:jgodinez
Date: 2013-07-26 15:08 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/467a0c21790b
8020208: NullPointerException at sun.print.Win32PrintService.getMediaPrintables
Reviewed-by: jchen, prr
! src/windows/classes/sun/print/Win32PrintServic
Changeset: c2e27e7a42ae
Author:jchen
Date: 2013-07-24 13:05 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/c2e27e7a42ae
8005126: [parfait] #418 - #428 XRBackendNative.c Integer overflow
Reviewed-by: prr, vadim
! src/solaris/native/sun/java2d/x11/XRBackendNative.c
Changeset: db2e3a686cf3
Author:jchen
Date: 2013-07-24 12:40 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/db2e3a686cf3
8011709: [parfait] False positive: memory leak in
jdk/src/share/native/sun/font/layout/CanonShaping.cpp
Reviewed-by: jgodinez, prr
! src/share/native/sun/
Changeset: dead66347eca
Author:jgodinez
Date: 2013-07-10 11:49 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/dead66347eca
8016737: After clicking on "Print UNCOLLATED" button, the print out come in
order 'Page 1', 'Page 2', 'Page 1'
Reviewed-by: jchen, prr
! src/solaris/cl
Just confirming if Johny's subscription is already working so I am
resending this again.
- Jennifer
On 7/3/2013 11:02 AM, Jennifer Godinez wrote:
Hi Phil, Johny,
Please review fix for 8016737: After clicking on "Print UNCOLLATED"
button, the print out come in order &#
Johny's reply below:
Original Message
Subject:Re: [OpenJDK 2D-Dev] [7u40] Please review 8016737
Date: Fri, 5 Jul 2013 11:44:04 -0700
From: Johnny Chen
To: Jennifer Godinez
CC: Johnny Chen , Phil Race
, "2d-dev@openjdk.java.net >>
Hi Johny,
Please review.
Thanks.
- Jennifer
On 7/3/2013 12:43 PM, Phil Race wrote:
Approved.
-phil.
On 7/3/2013 11:02 AM, Jennifer Godinez wrote:
Hi Phil, Johny,
Please review fix for 8016737: After clicking on "Print UNCOLLATED"
button, the print out come in order 'P
Hi Phil, Johny,
Please review fix for 8016737: After clicking on "Print UNCOLLATED"
button, the print out come in order 'Page 1', 'Page 2', 'Page 1'
Bug: http://bugs.sun.com/view_bug.do?bug_id=8016737
Webrev: http://cr.openjdk.java.net/~jgodinez/8016737/webrev.00/
Thank you.
Jennifer
Changeset: 75844b444879
Author:jchen
Date: 2013-07-03 10:22 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/75844b444879
8014497: [parfait] Potential null pointer dereference in
jdk/src/share/native/sun/java2d/cmm/lcms/cmsgamma.c
Reviewed-by: bae, prr
! src/share/native/sun/
r one media size: 1
media size name registered in JDK, whose media size matches with this
media size, 1 new media size name constructed from a string returned
by the driver. Testing in my local environment with all earlier
mentioned printers showed that cases of presence of such pairs of
media
Fix is pushed in both 8 and 7u40.
Jennifer
On 6/6/2013 1:18 PM, Phil Race wrote:
Hello,
The fix looks fine since appears to be functionally equivalent to
what I suggested in whatever was my last email.
In terms of back porting I think we literally have until Tuesday
to get it into 7u40.
Jenn
Changeset: 1431488fb0f9
Author:jgodinez
Date: 2013-06-07 10:26 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/1431488fb0f9
8013810: PrintServiceLookup.lookupPrintServices() does not return consistent
result
Reviewed-by: prr, jgodinez
Contributed-by: patr...@reini.net
! src/
Fix looks good.
Jennifer
On 06/01/2013 01:02 AM, Patrick Reinhart wrote:
Hi Phil,
As I'm now back from my holidays I changed the fix according your
suggestions:
http://reinharts.dyndns.org/8013810/v3/webrev
Can you take a look at it? I also restructured my webrev web folders
for all older
Hi Phil and Andrew,
Please review the 7u backport fix of
8009199: Printed text become garbage on Mac OSX
File changed is identical in 8.
Bug:
http://bugs.sun.com/view_bug.do?bug_id=8009199
Webrev:
http://cr.openjdk.java.net/~jgodinez/8009199/webrev/
Thank you.
Jennifer
Hi Phil and Andrew,
Please review the 7u backport fix of
7183520: [macosx]Unable to print out the defined page for
2D_PrintingTiger/JTablePrintPageRangesTest.
There's a minor difference compared to 8 as 7154047, a subset of 7183520
was already fixed in 7u so the code which has the same effect
Hi Phil and Andrew,
Please review JDK 7 backport fix for
7158350: [macosx] Strange results of SwingUIText printing
Bug:
http://bugs.sun.com/view_bug.do?bug_id=7158350
Webrev:
http://cr.openjdk.java.net/~jgodinez/7158350/webrev.0/
Thank you.
- Jennifer
Yes, backport to 7. The webrev shows 8 repo but file is identical in 7.
Jennifer
On 5/29/2013 1:54 PM, Phil Race wrote:
I think you mean to back port this from 8 to 7u ? Yes, looks good.
-phil.
On 5/29/2013 1:44 PM, Jennifer Godinez wrote:
Hi Phil and Johny,
Please review backport fix for
Hi Phil and Johny,
Please review backport fix for 8012381 : [macosx] : Collation selection
ignored when printing on MacOSX
Bug:
http://bugs.sun.com/view_bug.do?bug_id=8012381
Webrev:
http://cr.openjdk.java.net/~jgodinez/8012381/
Thank you.
- Jennifer
Changeset: 7e2a887a069e
Author:jgodinez
Date: 2013-05-29 09:46 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/7e2a887a069e
8012381: [macosx]Unable to print out the defined page for
2D_PrintingTiger/JTablePrintPageRangesTest
Reviewed-by: jchen, prr
! src/solaris/classes/sun/
Changeset: f4ad2fa22474
Author:jgodinez
Date: 2013-05-29 09:18 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/f4ad2fa22474
7183520: [macosx]Unable to print out the defined page for
2D_PrintingTiger/JTablePrintPageRangesTest.
Reviewed-by: bae, prr
! src/macosx/classes/sun/lw
Looks good.
-Jennifer
On 5/23/2013 2:26 PM, Phil Race wrote:
Jennifer & Andrew,
http://cr.openjdk.java.net/~prr/8008535/
Simple fix for a printing bug dating back to JDK 7 build 08 !
-phil.
Changeset: 0208f5f12dc3
Author:jchen
Date: 2013-05-23 12:16 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/0208f5f12dc3
8012629: java.lang.UnsatisfiedLinkError exception throw by getAllFonts() on
MacOSX
Reviewed-by: bae, prr
! make/sun/awt/FILES_c_unix.gmk
! make/sun/awt/FI
Changeset: 93de1ab38793
Author:jchen
Date: 2013-05-17 10:04 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/93de1ab38793
8003444: Fix potential NULL pointer dereference
Reviewed-by: jgodinez, prr
! src/share/native/sun/java2d/cmm/lcms/cmscgats.c
! src/share/native/sun/java2d/
Changeset: 23f7ff502a89
Author:jgodinez
Date: 2013-05-07 09:32 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/23f7ff502a89
8011069: Printing: NullPointerException since jdk8 b82 showing native Page
Setup Dialog.
Reviewed-by: bae, prr
! src/macosx/classes/sun/lwawt/macosx/CP
:39 PM, Patrick Reinhart wrote:
Hi Jennifer,
Had you got already the time to confirm the problem?
Cheers Patrick
Am 23.04.2013 um 18:56 schrieb Jennifer Godinez :
Hi Phil,
I haven't confirmed this. I was about to ask for the program but now I see it
is included in the webrev. I'
Changeset: f4aa34a7a44d
Author:jchen
Date: 2013-04-29 10:02 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/f4aa34a7a44d
8005302: [findbugs] public methods return internal arrays; may be private
Reviewed-by: bae, prr
! src/share/classes/sun/java2d/pipe/AAShapePipe.java
Changeset: b0c41789f500
Author:jgodinez
Date: 2013-04-25 10:14 -0700
URL: http://hg.openjdk.java.net/jdk8/2d/jdk/rev/b0c41789f500
8009199: Printed text become garbage on Mac OSX
Reviewed-by: bae, prr
! src/macosx/native/sun/awt/CTextPipe.m
Hi Phil,
I haven't confirmed this. I was about to ask for the program but now I
see it is included in the webrev. I'll let you know.
Jennifer
On 4/23/2013 9:27 AM, Phil Race wrote:
That's a Java incident number, not a JDK bug number.
JDK numbers begin with "JDK-8", incident numbers begin w
with the fix? Do you use the test application provided on the
bug's record page? Perhaps, you are using a "fastdebug" build of JDK
8, if it is so, then can this kind of build be considered as a
reliable for verification of fixes?
Thank you,
Anton
On 4/19/2013 1:21 AM, Jennife
, tid=9484
# guarantee(result == EXCEPTION_CONTINUE_EXECUTION) failed: Unexpected
result f
rom topLevelExceptionFilter
#
On 4/17/2013 9:49 AM, Jennifer Godinez wrote:
Thanks Anton. I will test your fix and let you know.
Jennifer
On 4/17/2013 5:53 AM, Anton Litvinov wrote:
Hello Jennifer,
Than
1 - 100 of 253 matches
Mail list logo
