RFR: 8344197: SM cleanup in java.util.concurrent

[Doug Lea]

On Fri, 15 Nov 2024 10:16:00 GMT, Glavo <duke at openjdk.org> wrote:

>> src/java.base/share/classes/java/util/concurrent/Executors.java line 556:
>> 
>>> 554:      * A callable that runs under established access control settings.
>>> 555:      */
>>> 556:     private static final class PrivilegedCallable<T> implements Callable<T> {
>> 
>> I can't help thinking some larger cleanup or renaming will be needed here as PrivilegedXXX is misleading now.
>
> This class is currently meaningless, perhaps it should be carefully removed and the documentation for `priviledgedCallable` should be updated.

Deleting it would also mean removing or modifying a bunch of j.u./tck tests. Perhaps deprecation is enough.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22119#discussion_r1843788836