On 28/11/2016 14:47, Chris Hegarty wrote:

:
2) jartool Main.java
   Maybe concealedPackages should have a comment about its use ( it is
   used in the Validator, and not by Main at all ).

Just on this one, I think this was introduced when Steve added the MR JAR validation, I agree it's ugly but we haven't changed it here. I'll add a comment as you suggest but it does some overall cleanup as part of finishing up the MR JAR work.

-Alan

Reply via email to