Re: [OpenJDK 2D-Dev] [14] RFR JDK-8223558: Java apps do not support Myanmar fonts
The code changes look fine, but nothing you are doing here makes Myanmar "supported". It may work reasonably well, but it does not get added to the list of supported locales - for Oracle JDK anyway. Also I don't know what input method support might be required for this besides what you are doing here on the display side. -phil. On 10/3/19 10:28 AM, Alexey Ivanov wrote: Hi Phil, Thank you for your review. Please see my answers inline: On 03/10/2019 17:31, Philip Race wrote: 1) This is an RFE, not a bug. I have updated the CR type to RFE. 2) Does this font exist on Windows 7 ? No, it does not. This font was added in Windows 8. https://docs.microsoft.com/en-us/typography/font-list/myanmar-text The tests check if the required font is available; if not, an message is printed and the test exists. 3) This cannot be backported since the older JDK releases will not support it. No, it cannot be backported to 8u because ICU does not support Myanmar. It can be brought to 11 updates. Regards, Alexey -phil. On 10/3/19, 8:57 AM, Alexey Ivanov wrote: Any volunteers to review? On 25/09/2019 20:38, Alexey Ivanov wrote: Hello, Please review the fix for JDK 14: bug: https://bugs.openjdk.java.net/browse/JDK-8223558 webrev: http://cr.openjdk.java.net/~aivanov/8223558/webrev.00/ Problem description: Java renders Myanmar script incorrectly. Some glyphs are not combined, which makes text unreadable. For example, this glyph ကြ is made of two characters U+1000 and U+103C. Java renders two separate glyphs. Workaround: Enable ligatures or kerning on the font to get the correct rendering. The fix: Enables complex layout for base Myanmar characters (U+1000-U+109F). Provides fallback font on Windows. On Linux, fontconfig handles the fallback. I ran awt/font tests, no new failures found.
Re: [OpenJDK 2D-Dev] [14] RFR JDK-8223558: Java apps do not support Myanmar fonts
Hi Phil, Thank you for your review. Please see my answers inline: On 03/10/2019 17:31, Philip Race wrote: 1) This is an RFE, not a bug. I have updated the CR type to RFE. 2) Does this font exist on Windows 7 ? No, it does not. This font was added in Windows 8. https://docs.microsoft.com/en-us/typography/font-list/myanmar-text The tests check if the required font is available; if not, an message is printed and the test exists. 3) This cannot be backported since the older JDK releases will not support it. No, it cannot be backported to 8u because ICU does not support Myanmar. It can be brought to 11 updates. Regards, Alexey -phil. On 10/3/19, 8:57 AM, Alexey Ivanov wrote: Any volunteers to review? On 25/09/2019 20:38, Alexey Ivanov wrote: Hello, Please review the fix for JDK 14: bug: https://bugs.openjdk.java.net/browse/JDK-8223558 webrev: http://cr.openjdk.java.net/~aivanov/8223558/webrev.00/ Problem description: Java renders Myanmar script incorrectly. Some glyphs are not combined, which makes text unreadable. For example, this glyph ကြ is made of two characters U+1000 and U+103C. Java renders two separate glyphs. Workaround: Enable ligatures or kerning on the font to get the correct rendering. The fix: Enables complex layout for base Myanmar characters (U+1000-U+109F). Provides fallback font on Windows. On Linux, fontconfig handles the fallback. I ran awt/font tests, no new failures found.
Re: [OpenJDK 2D-Dev] [14] RFR JDK-8223558: Java apps do not support Myanmar fonts
1) This is an RFE, not a bug. 2) Does this font exist on Windows 7 ? 3) This cannot be backported since the older JDK releases will not support it. -phil. On 10/3/19, 8:57 AM, Alexey Ivanov wrote: Any volunteers to review? On 25/09/2019 20:38, Alexey Ivanov wrote: Hello, Please review the fix for JDK 14: bug: https://bugs.openjdk.java.net/browse/JDK-8223558 webrev: http://cr.openjdk.java.net/~aivanov/8223558/webrev.00/ Problem description: Java renders Myanmar script incorrectly. Some glyphs are not combined, which makes text unreadable. For example, this glyph ကြ is made of two characters U+1000 and U+103C. Java renders two separate glyphs. Workaround: Enable ligatures or kerning on the font to get the correct rendering. The fix: Enables complex layout for base Myanmar characters (U+1000-U+109F). Provides fallback font on Windows. On Linux, fontconfig handles the fallback. I ran awt/font tests, no new failures found.
Re: [OpenJDK 2D-Dev] [14] RFR JDK-8223558: Java apps do not support Myanmar fonts
Any volunteers to review? On 25/09/2019 20:38, Alexey Ivanov wrote: Hello, Please review the fix for JDK 14: bug: https://bugs.openjdk.java.net/browse/JDK-8223558 webrev: http://cr.openjdk.java.net/~aivanov/8223558/webrev.00/ Problem description: Java renders Myanmar script incorrectly. Some glyphs are not combined, which makes text unreadable. For example, this glyph ကြ is made of two characters U+1000 and U+103C. Java renders two separate glyphs. Workaround: Enable ligatures or kerning on the font to get the correct rendering. The fix: Enables complex layout for base Myanmar characters (U+1000-U+109F). Provides fallback font on Windows. On Linux, fontconfig handles the fallback. I ran awt/font tests, no new failures found. -- Regards, Alexey
[OpenJDK 2D-Dev] [14] RFR JDK-8223558: Java apps do not support Myanmar fonts
Hello, Please review the fix for JDK 14: bug: https://bugs.openjdk.java.net/browse/JDK-8223558 webrev: http://cr.openjdk.java.net/~aivanov/8223558/webrev.00/ Problem description: Java renders Myanmar script incorrectly. Some glyphs are not combined, which makes text unreadable. For example, this glyph ကြ is made of two characters U+1000 and U+103C. Java renders two separate glyphs. Workaround: Enable ligatures or kerning on the font to get the correct rendering. The fix: Enables complex layout for base Myanmar characters (U+1000-U+109F). Provides fallback font on Windows. On Linux, fontconfig handles the fallback. I ran awt/font tests, no new failures found. -- Regards, Alexey