On Fri, 15 Nov 2024 10:16:00 GMT, Glavo <d...@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
