Code Review for jdk

Prepared by:Steven R. Loomis on Fri Jun 22 12:56:23 PDT 2018
Workspace:/Users/srl/src/java/jdk
Summary of changes: 346 lines changed: 340 ins; 0 del; 6 mod; 3087 unchg
Patch of changes: jdk.patch
Bug id: JDK-8187100 : Wrong rendering of variation sequences
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.desktop/share/classes/sun/font/CMap.java

114 lines changed: 114 ins; 0 del; 0 mod; 1062 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.desktop/share/classes/sun/font/CharToGlyphMapper.java

14 lines changed: 14 ins; 0 del; 0 mod; 91 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.desktop/share/classes/sun/font/CompositeGlyphMapper.java

2 lines changed: 1 ins; 0 del; 1 mod; 272 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.desktop/share/classes/sun/font/Font2D.java

4 lines changed: 4 ins; 0 del; 0 mod; 562 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.desktop/share/classes/sun/font/TrueTypeGlyphMapper.java

40 lines changed: 39 ins; 0 del; 1 mod; 244 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.desktop/share/native/common/font/sunfontids.h

1 line changed: 1 ins; 0 del; 0 mod; 93 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.desktop/share/native/libfontmanager/hb-jdk-font.cc

12 lines changed: 8 ins; 0 del; 4 mod; 390 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.desktop/share/native/libfontmanager/sunFont.c

2 lines changed: 2 ins; 0 del; 0 mod; 373 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/java/awt/font/TextLayout/TestVS-expect.png

Unexpected Error occurred reading `diff -e /dev/null new/test/jdk/java/awt/font/TextLayout/TestVS-expect.png`: $?=0, err= 1

------ ------ ------ ------ --- New Patch Raw test/jdk/java/awt/font/TextLayout/TestVS.java

92 lines changed: 92 ins; 0 del; 0 mod; 0 unchg

------ ------ ------ ------ --- New Patch Raw test/jdk/java/awt/font/TextLayout/VariationSelectorTest.java

65 lines changed: 65 ins; 0 del; 0 mod; 0 unchg

This code review page was prepared using /Users/srl/bin/webrev.ksh (vers 25.17-hg+openjdk.java.net).