> On Mar 15, 2017, at 11:25 AM, Claes Redestad <claes.redes...@oracle.com> > wrote: > > > > On 03/15/2017 05:36 PM, Mandy Chung wrote: >> I would prefer to separate the performance issue from this fix and keep this >> fix simple. There may be more to tease out for performance regression that >> I defer to Sherman to discuss with you. > > Ok: > > http://cr.openjdk.java.net/~redestad/8176709/jdk.03/ > > Also fixed an additional correctness issue, as we discussed offline, in that > we should only catch and swallow NoSuchFileException (when manifest is > missing), and rethrow any other IOException.
Looks good. You can take out the unused import before the push. Thanks for separating the performance fix from this patch. Mandy