On Mon, 7 Jul 2025 12:05:36 GMT, Chen Liang <li...@openjdk.org> wrote:

>> src/java.base/share/classes/java/util/concurrent/atomic/AtomicIntegerFieldUpdater.java
>>  line 405:
>> 
>>> 403:                 throw new IllegalArgumentException("Must be volatile 
>>> type");
>>> 404: 
>>> 405:             if (Modifier.isStatic(modifiers))
>> 
>> Can we break out the Modifier tests in the three updater classes to a common 
>> method?
>
> Should we include that in this patch?

Created https://bugs.openjdk.org/browse/JDK-8364544 to track that.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25945#discussion_r2248393574

Reply via email to