arturobernalg commented on a change in pull request #128:
URL: https://github.com/apache/commons-crypto/pull/128#discussion_r553459802
##########
File path: src/main/java/org/apache/commons/crypto/NativeCodeLoader.java
##########
@@ -171,7 +171,7 @@ private static File extractLibraryFile(final String
libFolderForCurrentOS, final
// Check whether the contents are properly copied from the resource
// folder
try (InputStream nativeInputStream =
NativeCodeLoader.class.getResourceAsStream(nativeLibraryFilePath)) {
- try (InputStream extractedLibIn = new
FileInputStream(extractedLibFile)) {
+ try (InputStream extractedLibIn =
Files.newInputStream(Paths.get(extractedLibFile.getPath()))) {
Review comment:
HI @garydgregory
https://pmd.github.io/latest/pmd_rules_java_performance.html#avoidfilestream
https://bugs.openjdk.java.net/browse/JDK-8080225
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
