<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