10 Oct
2018
10 Oct
'18
9:03 p.m.
On 10/10/18 1:41 PM, Alan Bateman wrote:
On 10/10/2018 21:38, Mandy Chung wrote:
:
This assertion is not strictly necessary and test/jdk/java/lang/invoke/DefineClassTest.java verifies these properties of the resulting Class. I propose to remove this assertion. An alternative is to just drop the PD check from the assertion, the check that defining class loader and package should never fail. Either is okay with me.
If the loader or package is not the one expected, it's a serious bug and also the DefineClassTest test will catch it. I opt to take this out as the assertion does not seem adding much value. thanks Mandy