RFR: 8288327: Executable.hasRealParameterData should not be volatile [v7]

Peter Levart plevart at openjdk.org
Fri Jul 1 10:38:43 UTC 2022


On Fri, 1 Jul 2022 07:27:59 GMT, Сергей Цыпанов <duke at openjdk.org> wrote:

>> src/java.base/share/classes/java/lang/reflect/Executable.java line 457:
>> 
>>> 455:     private transient @Stable ParameterData parameterData;
>>> 456: 
>>> 457:     record ParameterData(@Stable Parameter[] parameters, boolean isReal) {}
>> 
>> Record fields are implicitly "trusted final", which means they don't need @Stable annotation:
>> https://github.com/openjdk/jdk/blob/124c63c17c897404e3c5c3615d6727303e4f3d06/src/hotspot/share/ci/ciField.cpp#L240
>
> @plevart so should I remove it or keep as is?

Right, in that case, it should remain.

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

PR: https://git.openjdk.org/jdk/pull/9143


More information about the core-libs-dev mailing list