On Wed, 21 Jan 2026 16:02:05 GMT, Maurizio Cimadamore <[email protected]> 
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
>
> src/java.base/share/classes/java/lang/classfile/Signature.java line 295:
> 
>> 293: 
>> 294:         /**
>> 295:          * {@return a class or interface signature without an outer 
>> type}
> 
> Maybe we should say that, by default, the returned signature is NOT null 
> marked?

Same for other sig factories (like ArrayType, Typevar)

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

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

Reply via email to