On Sun, 15 Oct 2023 23:45:05 GMT, Chen Liang <li...@openjdk.org> 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

Reply via email to