On Wed, 17 Jun 2026 13:50:28 GMT, Alan Bateman <[email protected]> wrote:
>> I gave it a try, and with this patch `IMPL_LOOKUP` can create var handles >> that can write to `static final` fields, whereas with the current `lworld` >> code it fails with an `UnsupportedOperationException`. This seems like an >> unintended change. > > I have to admit that when I initially saw the parameter to makeFieldHandle > named "trusted" that I assumed it meant a trusted-final when it actually > means produced from IMPL_LOOKUP. > This seems like an unintended change. I intentionally made that change to secure against modifying strict or trusted finals even with IMPL_LOOKUP. I will roll it back if you think it's too strict. ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/2550#discussion_r3428795762
