Classfile API CodeElement.Kind cleanup

Brian Goetz brian.goetz at oracle.com
Thu Oct 13 17:20:35 UTC 2022


Overall this change looks fine.  I would rather keep the Opcode values 
for the pseudos, though, because I would rather not use null as a 
sentinel (see the constructors for, eg, ExceptionCatchImpl, which passes 
null to its super-ctor.)  Even if this is the only place these are used; 
these nulls are an error waiting to happen.  (This seems like a small 
bit to roll back.)






On 10/12/2022 2:57 AM, Adam Sotona wrote:
>
> It seems that there are some issues delivering my replies, so 
> forwarding directly:
>
> I’ve prepared CodeElement.Kind cleanup pull request:
>
> https://github.com/openjdk/jdk-sandbox/pull/41
>
>   * CodeElement.Kind moved to Opcode
>   * CodeElement::opcode and CodeElement::sizeInBytes moved down to
>     Instruction
>   * removed all pseudo Opcodes and Kinds
>
> Please review.
>
> Thanks,
>
> Adam
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/classfile-api-dev/attachments/20221013/5ccd8021/attachment-0001.htm>


More information about the classfile-api-dev mailing list