RFR: 8344078: Remove security manager dependency in java.nio [v5]

Brian Burkhalter bpb at openjdk.org
Wed Dec 4 19:31:52 UTC 2024


On Fri, 15 Nov 2024 18:00:59 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:

>> src/java.base/share/classes/java/nio/file/Files.java line 1528:
>> 
>>> 1526:                 list.add(detector);
>>> 1527:             }
>>> 1528:             return list;
>> 
>> Possible refactor to use stream API:  (may be a bit too aggressive)
>> Suggestion:
>> 
>>              return ServiceLoader
>>                     .load(FileTypeDetector.class, ClassLoader.getSystemClassLoader())
>>                     .stream()
>>                     .map(p -> (FileTypeDetector)p).toList();
>
> Thanks for the suggestion. Perhaps in a later commit: I prefer to keep the initial changes close to the original.

Addressed in #22552.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22132#discussion_r1870167141


More information about the nio-dev mailing list