<i18n dev> RFR: 8345325: SM cleanup of GetPropertyAction in java.base [v3]

Roger Riggs rriggs at openjdk.org
Mon Dec 2 22:54:42 UTC 2024


On Mon, 2 Dec 2024 22:51:55 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Remove sun/security/action/GetPropertyAction.java and all uses.
>> 
>> Dependent on PR#22418
>
> Roger Riggs has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 15 commits:
> 
>  - Merge branch 'master' into 8345325-sm-remove-getpropertyaction
>  - Remove unused import of Properties
>  - Update copyrights
>    Remove jdk/sun/security/action/Generify test
>  - 8345325: SM cleanup of GetPropertyAction in java.base
>    Remove use of GetPropertyAction
>    Remove sun/security/action/GetPropertyAction.java
>  - Remove PutAllAction and GetBooleanAction classes as there are no more dependencies.
>    Move GetPropertyAction.privilegedGetTimeoutProp and privilegedGetBooleanProp
>    methods to sun.security.util.SecurityProperties and rename them.
>  - Replace remaining calls to SecurityProperties.privilegedGetOverridable()
>    with SecurityProperties.getOverridableProperty() and remove
>    privilegedGetOverridable().
>  - Merge
>  - Remove doPrivileged calls.
>  - Remove dependency on SecurityConstants.ALL_PERMISSION from java.lang.Class.
>  - Remove permission text from comments in Provider.java.
>  - ... and 5 more: https://git.openjdk.org/jdk/compare/3a3bcd53...4c855033

The merge of master after integrating 8344397 looks to have resolved conflicts and only the intended changes remain.
I'll run a CI build to confirm tier1-3.

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

PR Comment: https://git.openjdk.org/jdk/pull/22497#issuecomment-2513145496


More information about the i18n-dev mailing list