RFR: 8345141: Remove uses of SecurityManager in ShellFolder related classes [v2]

Alexander Zvegintsev azvegint at openjdk.org
Mon Dec 2 00:47:53 UTC 2024


On Sat, 30 Nov 2024 03:45:14 GMT, Phil Race <prr at openjdk.org> wrote:

>> src/java.desktop/windows/classes/sun/awt/shell/Win32ShellFolderManager2.java line 167:
>> 
>>> 165: 
>>> 166:     static Win32ShellFolder2 getDesktop() {
>>> 167:         if (desktop == null) {
>> 
>> `import java.util.stream.Stream;` unused
>
> Doesn't seem related to this change. I'll fix it here since a simple build should prove it is safe, but refactoring and non-SM clean up is not a goal in these fixes.

In this case, it is related because you have removed its use:

`- private static File[] checkFiles(Stream<File> filesStream, @SuppressWarnings("removal") SecurityManager sm) {`

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22423#discussion_r1865077865


More information about the client-libs-dev mailing list