On Thu, 7 Oct 2021 14:21:27 GMT, Sean Coffey <coff...@openjdk.org> wrote:

>> src/java.base/share/classes/sun/security/util/ManifestEntryVerifier.java 
>> line 66:
>> 
>>> 64: 
>>> 65:     private String name = null;
>>> 66:     private String manifestFileName;
>> 
>> Make this final and add a comment that it will never be null.
>
> Thanks for the review - Made final and added comment. Also decided to make 
> the "Manifest man" variable final here also while here.

Great, the more final there are, the less I need to look back and forth while 
reading the code.

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

PR: https://git.openjdk.java.net/jdk/pull/5841

Reply via email to