Looks good to me.
Naoto
On 7/3/18 1:40 AM, Dipak Kumar wrote:
Just a gentle reminder. Requesting to review the changes for 8u-backport.
Thanks,
Dipak
-----Original Message-----
From:Dipak Kumar
Sent:28 June 2018 14:43
To:2d-...@openjdk.java.net; i18n-dev@openjdk.java.net; Philip Race
<philip.r...@oracle.com>; Naoto Sato <naoto.s...@oracle.com>
Subject:<i18n dev> [8u-backport] Review Request: 8202696 Remove
exclusion range for phonetic chars in windows fontconfig.properties
Hi,
Please review below patch for 8u-backport:
Main JBS bug:_https://bugs.openjdk.java.net/browse/JDK-8202696_
Webrev for
8u-backport:_http://cr.openjdk.java.net/~dkumar/8202696/8u-backport/webrev.00/_
JDK-11 changeset
-_http://hg.openjdk.java.net/jdk/client/rev/b9456394d24f_<http://hg.openjdk.java.net/jdk/client/rev/b9456394d24f>
Patch from JDK-11 doesn't apply cleanly because of file name and file
path differences.
Moreover there are some additional ranges added to font exclusion in
JDK-11 config file.
JPRT and font related Jtreg tests are fine.
Thanks,
Dipak