On Wed, 8 Feb 2023 11:00:14 GMT, Adam Sotona <[email protected]> wrote:
>> src/java.base/share/classes/jdk/internal/classfile/attribute/StackMapTableAttribute.java
>> line 68:
>>
>>> 66: * A simple stack value.
>>> 67: */
>>> 68: public enum SimpleVerificationTypeInfo implements
>>> VerificationTypeInfo {
>>
>> I note that in this class we have made the decision to model all the innards
>> (XYZInfo) as nested classes - while in all the other cases XYZInfo are
>> toplevel types. Moving forward, we should pick something consistent.
>
> Every case has been considered individually, evaluated on use cases and pros
> and cons have been weighted. Unified approach across the whole API would be
> nice, however not so simple and not the highest priority.
I had the same observation as Maurizio.
-------------
PR: https://git.openjdk.org/jdk/pull/10982
