Code Review for open

Prepared by:rriggs on Thu May 9 10:56:18 EDT 2019
Workspace:/ws/valhalla-dev/open
Compare against version:-2
Branch:lworld
Summary of changes: 45 lines changed: 4 ins; 5 del; 36 mod; 8047 unchg
Patch of changes: open.patch
Bug id: JDK-8223350] [lworld : Use inline classes instead of value classes
Legend: Modified file
Deleted file
New file

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/java/lang/Class.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
5 lines changed: 0 ins; 0 del; 5 mod; 4207 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/java/lang/reflect/Constructor.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
2 lines changed: 0 ins; 0 del; 2 mod; 674 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw src/java.base/share/classes/java/lang/reflect/Field.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
11 lines changed: 4 ins; 5 del; 2 mod; 1237 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/java/lang/invoke/VarHandles/Value.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
1 line changed: 0 ins; 0 del; 1 mod; 59 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/valhalla/valuetypes/ArrayElementVarHandleTest.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
1 line changed: 0 ins; 0 del; 1 mod; 130 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/valhalla/valuetypes/Line.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
1 line changed: 0 ins; 0 del; 1 mod; 50 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/valhalla/valuetypes/MethodHandleTest.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
6 lines changed: 0 ins; 0 del; 6 mod; 299 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/valhalla/valuetypes/MixedValues.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
1 line changed: 0 ins; 0 del; 1 mod; 41 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/valhalla/valuetypes/NonFlattenValue.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
1 line changed: 0 ins; 0 del; 1 mod; 51 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/valhalla/valuetypes/ObjectMethods.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
3 lines changed: 0 ins; 0 del; 3 mod; 190 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/valhalla/valuetypes/Point.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
1 line changed: 0 ins; 0 del; 1 mod; 37 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/valhalla/valuetypes/QTypeDescriptorTest.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
2 lines changed: 0 ins; 0 del; 2 mod; 220 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/valhalla/valuetypes/Reflection.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
3 lines changed: 0 ins; 0 del; 3 mod; 207 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/valhalla/valuetypes/SubstitutabilityTest.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
1 line changed: 0 ins; 0 del; 1 mod; 112 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/valhalla/valuetypes/Value.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
3 lines changed: 0 ins; 0 del; 3 mod; 182 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/valhalla/valuetypes/ValueArray.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
2 lines changed: 0 ins; 0 del; 2 mod; 151 unchg

Cdiffs Udiffs Sdiffs Frames Old New Patch Raw test/jdk/valhalla/valuetypes/ValueBootstrapMethods.java

rev 55117 : 8223350: [lworld] Use inline classes instead of value classes
1 line changed: 0 ins; 0 del; 1 mod; 200 unchg

This code review page was prepared using /home/rriggs/bin/webrev.ksh (vers 25.16-hg+openjdk.java.net).