kratos0718 commented on PR #166:
URL: https://github.com/apache/maven-shared-jar/pull/166#issuecomment-5855105121

   Done, the whole constructor is one try/finally now, including the JarFile 
open. The file is opened into a local and the field is only assigned once 
construction succeeds, so finally closes it on any failure without calling 
closeQuietly() on a field that may never have been set.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to