On Tue, 10 May 2022 19:24:24 GMT, Mark Powers <d...@openjdk.java.net> wrote:
>> src/java.base/share/classes/java/lang/reflect/AccessibleObject.java line 777: >> >>> 775: if (!printStackPropertiesSet && VM.initLevel() >= 1) { >>> 776: printStackWhenAccessFails = GetBooleanAction. >>> 777: >>> privilegedGetProperty("sun.reflect.debugModuleAccessChecks"); >> >> Running with `-Dsun.reflect.debugModuleAccessChecks` should set >> printStackWhenAccessFails to true, not false. > > You are right. The old way maps the null string to true, and the new way maps > it to false. I did not notice that. At this point, I see no value in making > the "true".equals and "false".equals changes. Too much can break. I'll > reverse these changes. This change still needs to be reversed. ------------- PR: https://git.openjdk.java.net/jdk/pull/8559