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