<i18n dev> RFR: 8342682: Errors related to unused code on Windows after 8339120 in dt_shmem jdwp security and jpackage

Chris Plummer cjplummer at openjdk.org
Thu Oct 24 06:56:10 UTC 2024


On Thu, 24 Oct 2024 03:31:31 GMT, Julian Waters <jwaters at openjdk.org> wrote:

>> I'm not sure what you mean by "that one". It's the static one that should be removed. The local variables always hide the static, and there seems to be no reason for the value of memHandle to survive outside of the local scope it is used in.
>
> Oh, I was referring to the static global one. In that case I'll remove it (Or rather, comment it so it isn't lost to time and someone can bring it back if need be)

I would just remove the static. I don't see any reason for it to exist.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21616#discussion_r1814375764


More information about the i18n-dev mailing list