On Sun, 15 Oct 2023 23:45:05 GMT, Chen Liang <[email protected]> wrote:
>> Erik Gahlin has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Updates
>
> src/jdk.jfr/share/classes/jdk/jfr/internal/EventInstrumentation.java line 204:
>
>> 202: String typeDescriptor = classDesc.descriptorString();
>> 203: for (ClassElement ce : classModel.elements()) {
>> 204: if (ce instanceof RuntimeVisibleAnnotationsAttribute rvaa) {
>
> You can use
> `classModel.findAttribute(Attributes.RUNTIME_VISIBLE_ANNOTATIONS)` instead of
> looping.
I could not get it to work with findAttribute. No annotations were found.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16195#discussion_r1360107669
