RFR: 8312444: Delete unused parameters and variables in SocketPermission [v3]

Korov duke at openjdk.org
Mon Mar 11 20:35:16 UTC 2024


On Tue, 5 Mar 2024 13:37:22 GMT, Korov <duke at openjdk.org> wrote:

>>> Seems good. If nobody disagrees such mixed patch, I will help you change the title and content of the issue.
>> 
>> Have changed just now! And please wait for a formal reviewer to review this path.
>
>> > Seems good. If nobody disagrees such mixed patch, I will help you change the title and content of the issue.
>> 
>> Have changed just now! And please wait for a formal reviewer to review this path.
> 
> Thanks so much!!!

> @Korov I did a round of testing and results came back green. You're good to go. The SecurityManager has been deprecated for removal, so maybe it would be better to stay away from SecurityManager related classes from now on unless there some strong reason to make a change.

Thanks for your friendly reminder.

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

PR Comment: https://git.openjdk.org/jdk/pull/18086#issuecomment-1989390863



More information about the security-dev mailing list