<Sound Dev> RFR: 8266459: Implement JEP 411: Deprecate the Security Manager for Removal [v3]

Alan Bateman alanb at openjdk.java.net
Sat May 22 07:00:03 UTC 2021

On Fri, 21 May 2021 18:00:13 GMT, Phil Race <prr at openjdk.org> wrote:

> Are you suggesting that the patch doesn't need testing as it is ? It should be the same either way.
> It is very straight forward to run the automated tests across all platforms these days.
> I get the impression that no one is guaranteeing to do this straight after integration.
> It sounds like it is up for deferral if time runs out.
> The amount of follow-on work that I am hearing about here, and may be for tests, does not make it sound
> like this JEP is nearly as done as first presented.
> If there was some expectation that groups responsible for an area would get involved with this
> issue which I am assured was already known about, then why was it not raised before and made
> part of the plan ?

Sprinkling SuppressWarnings should be very low risk. Refactoring code to have the scope of the SW be as small as possible may be a bit more risky, esp. in areas where one doesn't know the code or the tests that exercise it. The tests may be good but it's not clear to me that we want to force Max to do significant refactoring in this PR.  PR 4138 has been created as the follow-on PR so I think we should help get that reviewed so that it can be integrated soon after this PR.


PR: https://git.openjdk.java.net/jdk/pull/4073

More information about the sound-dev mailing list