On Wed, 22 Oct 2025 04:56:01 GMT, Serguei Spitsyn <[email protected]> wrote:
>> test/hotspot/jtreg/serviceability/jvmti/events/FieldAccess/FieldsEventsFromJNI/libFieldsEventsFromJNI.cpp >> line 167: >> >>> (failed to retrieve contents of file, check the PR for context) >> Need to rename these as well: `s/FieldsEventsFromJNI/FieldEventsFromJNI/g`. > >> jvmtiFieldEventsFromJNI/ >> jvmtiFieldEventsFromJNI/jvmtiFieldEventsFromJNI.java >> jvmtiFieldEventsFromJNI/libJvmtiFieldEventsFromJNI.jcpp > > It seems the file names are still not right and do not follow the above > patterns: > - FieldsEventsFromJNI/FieldsEventsFromJNI.java > - FieldsEventsFromJNI/libJvmtiFieldEventsFromJNI.cpp > > To follow the above this needs to be applied to folder and Java file names: > `s/FieldsEventsFromJNI/FieldEventsFromJNI/g` Renamved tio FieldsEventsFromJNI/FieldsEventsFromJNI.java FieldsEventsFromJNI/libFieldsEventsFromJNI.cpp the 'libJvmti...." should be just "lib..." ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27584#discussion_r2450444580
