RFR: 8224243: Make AccessibleObject a sealed class [v2]

[Joe Darcy]

On 5/24/2021 2:05 AM, Kasper Nielsen wrote:
> On Fri, 21 May 2021 02:42:50 GMT, Joe Darcy <darcy at openjdk.org> wrote:
>
>>> Conceptually, AccessbileObject is a sealed class with a protected constructor stating
>>>
>>>      Constructor: only used by the Java Virtual Machine.
>>>
>>> With the language now supporting sealed classes, the AccessbileObject should be marked as sealed.
>>>
>>> Executable and Field are the subclasses of AccessbileObject in the JDK; as Executable has subclasses, it is marked as non-sealed.
>>>
>>> Please also review the corresponding CSR:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8224243
>> Joe Darcy has updated the pull request incrementally with one additional commit since the last revision:
>>
>>    Update in response to review feedback.
> Missed the part about only Executable being made sealed. But if Executable is sealed, setAccessible(boolean) could still be moved from Method/Constructor down to Executable.

Perhaps; the caller sensitive nature of the methods may preclude or 
complicate that. I don't plan to add such method hoisting to this PR.

Thanks,

-Joe