Hello, Henry.
AquaComboBoxRendererInternal : why ? extends Object and not just ??
AquaComboBoxUI:427, 510, 530 - you can replace JComboBoxObject with
JComboBox? and avoid a @SupressWarning annotation.
AquaFileChooserUI.createDirectoryComboBoxRenderer actually returns a
ListCellRendererFile,
Thanks, Petr.
I worked out AquaFileChooserUI and updated the webrev to adapt comments
from both you and Joe.
The updated webrev is at,
http://cr.openjdk.java.net/~henryjen/jdk9/8044862/1/webrev/
Cheers,
Henry
On 07/09/2014 01:07 AM, Petr Pchelko wrote:
Hello, Henry.
Hello, Henry.
The new version of the fix looks good to me.
With best regards. Petr.
On Jul 10, 2014, at 4:51 AM, Henry Jen henry@oracle.com wrote:
Thanks, Petr.
I worked out AquaFileChooserUI and updated the webrev to adapt comments from
both you and Joe.
The updated webrev is
Looks fine to me too; thanks,
-Joe
On 07/09/2014 09:13 PM, Petr Pchelko wrote:
Hello, Henry.
The new version of the fix looks good to me.
With best regards. Petr.
On Jul 10, 2014, at 4:51 AM, Henry Jen henry@oracle.com wrote:
Thanks, Petr.
I worked out AquaFileChooserUI and updated
Hi,
Would someone from swing team care to take a look at this?
The change of src/macosx/classes/sun/font/CFontConfiguration.java will
be dropped as it had been fixed in JDK-8048980.
Cheers,
Henry
On 06/27/2014 03:00 PM, Joe Darcy wrote:
Hi Henry,
Your changes look good to me (I assume
Hi,
Please review a webrev for fixing rawtypes and unchecked lint warning
for macosx specific java code, this webrev is depending on
JDK-8043548[1] where some of the override methods are defined,
Also as the src/macosx/classes/apple/launcher/JavaAppLauncher.java to be
removed from
Hi Henry,
Your changes look good to me (I assume there was an issue using
wildcards in AquaComboBoxButton.java).
Thanks,
-Joe
On 06/27/2014 02:45 PM, Henry Jen wrote:
Hi,
Please review a webrev for fixing rawtypes and unchecked lint warning
for macosx specific java code, this webrev is
On June 27, 2014 at 3:00:13 PM, Joe Darcy
(joe.da...@oracle.com(mailto:joe.da...@oracle.com)) wrote:
Hi Henry,
Your changes look good to me (I assume there was an issue using
wildcards in AquaComboBoxButton.java).
Cannot make it to work. The problematic code is in line 169,
169