open (original) (raw)

Code Review for open

Prepared by: dholmes on Tue May 22 05:29:04 EDT 2018
Workspace: /export/users/dh198349/valhalla/repos/jdk-jep181/open
Compare against version: -2
Summary of changes: 2563 lines changed: 2541 ins; 5 del; 17 mod; 4846 unchg
Patch of changes: <open.patch>
Legend: Modified file Deleted file New file

[Cdiffs](src/java.base/share/classes/java/lang/Class.java.cdiff.html) [Udiffs](src/java.base/share/classes/java/lang/Class.java.udiff.html) [Sdiffs](src/java.base/share/classes/java/lang/Class.java.sdiff.html) [Frames](src/java.base/share/classes/java/lang/Class.java.frames.html) [Old](src/java.base/share/classes/java/lang/Class.java-.html) [New](src/java.base/share/classes/java/lang/Class.java.html) [Patch](src/java.base/share/classes/java/lang/Class.java.patch) [Raw](raw%5Ffiles/new/src/java.base/share/classes/java/lang/Class.java) src/java.base/share/classes/java/lang/Class.java

rev 50076 : [mq]: jep181-rev2

8 lines changed: 2 ins; 0 del; 6 mod; 3997 unchg

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/HostOfMemberMalformedHost.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/HostOfMemberMalformedHost.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/HostOfMemberMalformedHost.jcod) test/jdk/java/lang/reflect/Nestmates/HostOfMemberMalformedHost.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/HostOfMemberMissingHost.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/HostOfMemberMissingHost.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/HostOfMemberMissingHost.jcod) test/jdk/java/lang/reflect/Nestmates/HostOfMemberMissingHost.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/HostOfMemberNoHost.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/HostOfMemberNoHost.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/HostOfMemberNoHost.jcod) test/jdk/java/lang/reflect/Nestmates/HostOfMemberNoHost.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/HostOfMemberNotInstanceHost.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/HostOfMemberNotInstanceHost.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/HostOfMemberNotInstanceHost.jcod) test/jdk/java/lang/reflect/Nestmates/HostOfMemberNotInstanceHost.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/HostOfMemberNotOurHost.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/HostOfMemberNotOurHost.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/HostOfMemberNotOurHost.jcod) test/jdk/java/lang/reflect/Nestmates/HostOfMemberNotOurHost.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/HostWithDuplicateMembers.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/HostWithDuplicateMembers.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/HostWithDuplicateMembers.jcod) test/jdk/java/lang/reflect/Nestmates/HostWithDuplicateMembers.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/HostWithSelfMember.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/HostWithSelfMember.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/HostWithSelfMember.jcod) test/jdk/java/lang/reflect/Nestmates/HostWithSelfMember.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/Hosts.java.html) [Patch](test/jdk/java/lang/reflect/Nestmates/Hosts.java.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/Hosts.java) test/jdk/java/lang/reflect/Nestmates/Hosts.java

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/InvalidNestHost.java.html) [Patch](test/jdk/java/lang/reflect/Nestmates/InvalidNestHost.java.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/InvalidNestHost.java) test/jdk/java/lang/reflect/Nestmates/InvalidNestHost.java

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/MalformedHost.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/MalformedHost.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/MalformedHost.jcod) test/jdk/java/lang/reflect/Nestmates/MalformedHost.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/MemberMalformedHost.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/MemberMalformedHost.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/MemberMalformedHost.jcod) test/jdk/java/lang/reflect/Nestmates/MemberMalformedHost.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/MemberMissingHost.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/MemberMissingHost.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/MemberMissingHost.jcod) test/jdk/java/lang/reflect/Nestmates/MemberMissingHost.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/MemberNoHost.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/MemberNoHost.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/MemberNoHost.jcod) test/jdk/java/lang/reflect/Nestmates/MemberNoHost.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/MemberNotInstanceHost.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/MemberNotInstanceHost.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/MemberNotInstanceHost.jcod) test/jdk/java/lang/reflect/Nestmates/MemberNotInstanceHost.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/MemberNotOurHost.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/MemberNotOurHost.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/MemberNotOurHost.jcod) test/jdk/java/lang/reflect/Nestmates/MemberNotOurHost.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/PackagedNestHost.java.html) [Patch](test/jdk/java/lang/reflect/Nestmates/PackagedNestHost.java.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/PackagedNestHost.java) test/jdk/java/lang/reflect/Nestmates/PackagedNestHost.java

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/PackagedNestHost.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/PackagedNestHost.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/PackagedNestHost.jcod) test/jdk/java/lang/reflect/Nestmates/PackagedNestHost.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/PackagedNestHost2.java.html) [Patch](test/jdk/java/lang/reflect/Nestmates/PackagedNestHost2.java.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/PackagedNestHost2.java) test/jdk/java/lang/reflect/Nestmates/PackagedNestHost2.java

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/PackagedNestHost2Member.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/PackagedNestHost2Member.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/PackagedNestHost2Member.jcod) test/jdk/java/lang/reflect/Nestmates/PackagedNestHost2Member.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/PackagedNestHostMember.jcod.html) [Patch](test/jdk/java/lang/reflect/Nestmates/PackagedNestHostMember.jcod.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/PackagedNestHostMember.jcod) test/jdk/java/lang/reflect/Nestmates/PackagedNestHostMember.jcod

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/SampleNest.java.html) [Patch](test/jdk/java/lang/reflect/Nestmates/SampleNest.java.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/SampleNest.java) test/jdk/java/lang/reflect/Nestmates/SampleNest.java

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/TestReflectionAPI.java.html) [Patch](test/jdk/java/lang/reflect/Nestmates/TestReflectionAPI.java.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/TestReflectionAPI.java) test/jdk/java/lang/reflect/Nestmates/TestReflectionAPI.java

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/TestSecurityManagerChecks.java.html) [Patch](test/jdk/java/lang/reflect/Nestmates/TestSecurityManagerChecks.java.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/TestSecurityManagerChecks.java) test/jdk/java/lang/reflect/Nestmates/TestSecurityManagerChecks.java

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/empty.policy.html) [Patch](test/jdk/java/lang/reflect/Nestmates/empty.policy.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/empty.policy) test/jdk/java/lang/reflect/Nestmates/empty.policy

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/testPkg/Host.java.html) [Patch](test/jdk/java/lang/reflect/Nestmates/testPkg/Host.java.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/testPkg/Host.java) test/jdk/java/lang/reflect/Nestmates/testPkg/Host.java

rev 50076 : [mq]: jep181-rev2

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

------ ------ ------ ------ ---[New](test/jdk/java/lang/reflect/Nestmates/testPkg/Singleton.java.html) [Patch](test/jdk/java/lang/reflect/Nestmates/testPkg/Singleton.java.patch) [Raw](raw%5Ffiles/new/test/jdk/java/lang/reflect/Nestmates/testPkg/Singleton.java) test/jdk/java/lang/reflect/Nestmates/testPkg/Singleton.java

rev 50076 : [mq]: jep181-rev2

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

[Cdiffs](test/jdk/jdk/lambda/vm/InterfaceAccessFlagsTest.java.cdiff.html) [Udiffs](test/jdk/jdk/lambda/vm/InterfaceAccessFlagsTest.java.udiff.html) [Sdiffs](test/jdk/jdk/lambda/vm/InterfaceAccessFlagsTest.java.sdiff.html) [Frames](test/jdk/jdk/lambda/vm/InterfaceAccessFlagsTest.java.frames.html) [Old](test/jdk/jdk/lambda/vm/InterfaceAccessFlagsTest.java-.html) [New](test/jdk/jdk/lambda/vm/InterfaceAccessFlagsTest.java.html) [Patch](test/jdk/jdk/lambda/vm/InterfaceAccessFlagsTest.java.patch) [Raw](raw%5Ffiles/new/test/jdk/jdk/lambda/vm/InterfaceAccessFlagsTest.java) test/jdk/jdk/lambda/vm/InterfaceAccessFlagsTest.java

rev 50076 : [mq]: jep181-rev2

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

[Cdiffs](src/java.base/share/classes/jdk/internal/reflect/Reflection.java.cdiff.html) [Udiffs](src/java.base/share/classes/jdk/internal/reflect/Reflection.java.udiff.html) [Sdiffs](src/java.base/share/classes/jdk/internal/reflect/Reflection.java.sdiff.html) [Frames](src/java.base/share/classes/jdk/internal/reflect/Reflection.java.frames.html) [Old](src/java.base/share/classes/jdk/internal/reflect/Reflection.java-.html) [New](src/java.base/share/classes/jdk/internal/reflect/Reflection.java.html) [Patch](src/java.base/share/classes/jdk/internal/reflect/Reflection.java.patch) [Raw](raw%5Ffiles/new/src/java.base/share/classes/jdk/internal/reflect/Reflection.java) src/java.base/share/classes/jdk/internal/reflect/Reflection.java

rev 50076 : [mq]: jep181-rev2

7 lines changed: 0 ins; 1 del; 6 mod; 366 unchg

[Cdiffs](src/java.base/share/classes/sun/invoke/util/VerifyAccess.java.cdiff.html) [Udiffs](src/java.base/share/classes/sun/invoke/util/VerifyAccess.java.udiff.html) [Sdiffs](src/java.base/share/classes/sun/invoke/util/VerifyAccess.java.sdiff.html) [Frames](src/java.base/share/classes/sun/invoke/util/VerifyAccess.java.frames.html) [Old](src/java.base/share/classes/sun/invoke/util/VerifyAccess.java-.html) [New](src/java.base/share/classes/sun/invoke/util/VerifyAccess.java.html) [Patch](src/java.base/share/classes/sun/invoke/util/VerifyAccess.java.patch) [Raw](raw%5Ffiles/new/src/java.base/share/classes/sun/invoke/util/VerifyAccess.java) src/java.base/share/classes/sun/invoke/util/VerifyAccess.java

6 lines changed: 0 ins; 3 del; 3 mod; 394 unchg


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