On Wed, 25 Jun 2025 18:06:45 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:
>> I copied this code for another test in the Valhalla repo and thought it
>> would be a good utility function. It might be better written using the
>> Classfile API.
>> Tested with test.
>
> Coleen Phillimore has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Use ClassFile API to replace the class name.
test/lib/RedefineClassHelper.java line 95:
> 93: ClassDesc newClassDesc = ClassDesc.of(newClassName);
> 94: byte[] newBytes = ClassFile.of().build(newClassDesc,
> 95: classBuilder -> {
Can replace this lambda with `classModel::forEach` method reference.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25857#discussion_r2167390696
