On Mon, 3 Aug 2026 11:17:07 GMT, ExE Boss <[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 eight additional 
>> commits since the last revision:
>> 
>>  - Merge branch 'master' of https://github.com/openjdk/jdk into 
>> feature/class-is-class-or-interface-1
>>  - More spec clarifications
>>  - Wording review from Joe
>>  - Update usage in Valhalla
>>  - Merge branch 'master' of https://github.com/openjdk/jdk into 
>> feature/class-is-class-or-interface-1
>>  - Simplify code
>>  - Merge branch 'master' of https://github.com/openjdk/jdk into 
>> feature/class-is-class-or-interface-1
>>  - Class.isClassOrInterface
>
> src/java.base/share/classes/java/lang/Class.java line 3831:
> 
>> 3829:                 isArray() ||
>> 3830:                 isPrimitive() ||
>> 3831:                 this == Void.TYPE) {
> 
> **Fun fact:** `Void.TYPE.isPrimitive()` already returns `true`, so this last 
> comparison would always return `false`.

Yep, don't know why the original code was checked in like this for annotations 
in Java 8.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32109#discussion_r3703873982

Reply via email to