[foreign-memaccess+abi] RFR: Change local variable from Optional to String

Jorn Vernee jvernee at openjdk.org
Thu Oct 12 14:21:27 UTC 2023


On Tue, 10 Oct 2023 12:43:14 GMT, Per Minborg <pminborg at openjdk.org> wrote:

> This PR suggests removing the storage of `Optional` instance variables and instead adhere to the recommendations of Optional usage. This will also reduce memory usage slightly.

These changes seem fairly superfluous. I think I'd slightly prefer keeping things as they are.

-------------

PR Review: https://git.openjdk.org/panama-foreign/pull/907#pullrequestreview-1674276899


More information about the panama-dev mailing list