On Fri, 28 Jul 2023 19:23:26 GMT, Mark Powers <mpow...@openjdk.org> wrote:
>> `tmp` will not be null. `GetIntegerAction.privilegedGetProperty()` will >> return the default value when jdk.jar.maxSignatureFileSize=null > > You're right. Didn't look close enough. Might make sense to declare `tmp` as an `int` to avoid future confusion and save a few bytecode on repeated implicit unboxing every time `tmp` is used. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/15072#discussion_r1278027326