Code Review for jdk

Prepared by:Steven R. Loomis on Mon Jun 18 09:59:04 PDT 2018
Workspace:/Users/srl/src/java/jdk
Summary of changes: 808 lines changed: 799 ins; 0 del; 9 mod; 3778 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

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

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

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

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

168 lines changed: 168 ins; 0 del; 0 mod; 273 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/FontRunIterator.java

33 lines changed: 32 ins; 0 del; 1 mod; 170 unchg

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

3 lines changed: 2 ins; 0 del; 1 mod; 522 unchg

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

200 lines changed: 196 ins; 0 del; 4 mod; 241 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

43 lines changed: 40 ins; 0 del; 3 mod; 391 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

64 lines changed: 64 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).