Re: RFR: 8344330: Remove AccessController.doPrivileged() from jdk.charsets module [v2]

2024-11-18 Thread Naoto Sato
On Fri, 15 Nov 2024 20:17:59 GMT, Naoto Sato wrote: >> Removing a `AccessController.doPrivileged()` call from jdk.charsets module >> after JEP486 integration. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: > > fixed import Tha

Re: RFR: 8344330: Remove AccessController.doPrivileged() from jdk.charsets module [v2]

2024-11-18 Thread Roger Riggs
On Fri, 15 Nov 2024 20:17:59 GMT, Naoto Sato wrote: >> Removing a `AccessController.doPrivileged()` call from jdk.charsets module >> after JEP486 integration. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: > > fixed import Mar

Re: RFR: 8344330: Remove AccessController.doPrivileged() from jdk.charsets module

2024-11-15 Thread Lance Andersen
On Fri, 15 Nov 2024 19:26:15 GMT, Naoto Sato wrote: > Removing a `AccessController.doPrivileged()` call from jdk.charsets module > after JEP486 integration. Marked as reviewed by lancea (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/22162#pullrequestreview-2439458103

Re: RFR: 8344330: Remove AccessController.doPrivileged() from jdk.charsets module [v2]

2024-11-15 Thread Roger Riggs
On Fri, 15 Nov 2024 20:15:10 GMT, Naoto Sato wrote: >> Removing a `AccessController.doPrivileged()` call from jdk.charsets module >> after JEP486 integration. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: > > fixed import loo

Re: RFR: 8344330: Remove AccessController.doPrivileged() from jdk.charsets module [v2]

2024-11-15 Thread Justin Lu
On Fri, 15 Nov 2024 20:17:59 GMT, Naoto Sato wrote: >> Removing a `AccessController.doPrivileged()` call from jdk.charsets module >> after JEP486 integration. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: > > fixed import Mar

Re: RFR: 8344330: Remove AccessController.doPrivileged() from jdk.charsets module [v2]

2024-11-15 Thread Lance Andersen
On Fri, 15 Nov 2024 20:17:59 GMT, Naoto Sato wrote: >> Removing a `AccessController.doPrivileged()` call from jdk.charsets module >> after JEP486 integration. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: > > fixed import Mar

Re: RFR: 8344330: Remove AccessController.doPrivileged() from jdk.charsets module [v2]

2024-11-15 Thread Brian Burkhalter
On Fri, 15 Nov 2024 20:17:59 GMT, Naoto Sato wrote: >> Removing a `AccessController.doPrivileged()` call from jdk.charsets module >> after JEP486 integration. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: > > fixed import Upd

Re: RFR: 8344330: Remove AccessController.doPrivileged() from jdk.charsets module [v2]

2024-11-15 Thread Naoto Sato
> Removing a `AccessController.doPrivileged()` call from jdk.charsets module > after JEP486 integration. Naoto Sato has updated the pull request incrementally with one additional commit since the last revision: fixed import - Changes: - all: https://git.openjdk.org/jdk/pull/22

Re: RFR: 8344330: Remove AccessController.doPrivileged() from jdk.charsets module

2024-11-15 Thread Brian Burkhalter
On Fri, 15 Nov 2024 19:26:15 GMT, Naoto Sato wrote: > Removing a `AccessController.doPrivileged()` call from jdk.charsets module > after JEP486 integration. Looks fine. - Marked as reviewed by bpb (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/22162#pullrequestreview-243

RFR: 8344330: Remove AccessController.doPrivileged() from jdk.charsets module

2024-11-15 Thread Naoto Sato
Removing a `AccessController.doPrivileged()` call from jdk.charsets module after JEP486 integration. - Commit messages: - initial commit Changes: https://git.openjdk.org/jdk/pull/22162/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=22162&range=00 Issue: https://bugs.ope