RFR: Classfile signature bang support [v2]

Chen Liang liach at openjdk.org
Wed Jan 21 20:27:46 UTC 2026


On Wed, 21 Jan 2026 16:08:56 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> Chen Liang has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>> 
>>  - Review, various cleanup
>>  - Merge branch 'bworld' of https://github.com/openjdk/valhalla into feature/cf-bang-sig
>>  - Classfile signature bang support
>
> test/jdk/jdk/classfile/SignaturesTest.java line 411:
> 
>> 409:                 !Ljava/lang/Object;
>> 410:                 Ljava/lang/Object;!Ljava/lang/Runnable;
>> 411:                 <>Ljava/lang/Object;
> 
> This test line seems repeated? Or did you want to add a `!` here?

Replaced with `<T:!Ljava/lang/Number;>Ljava/lang/Object;` which tries to represent `<T extends Number!>`, currently illegal

-------------

PR Review Comment: https://git.openjdk.org/valhalla/pull/1933#discussion_r2714210432


More information about the valhalla-dev mailing list