Re: RFR: 8344331: SM cleanup in java.scripting [v2]

2024-11-18 Thread Justin Lu
On Sat, 16 Nov 2024 00:03:23 GMT, Justin Lu wrote: >> Please review this PR which removes the lone `AccessController.doPrivileged` >> call from the _java.scripting_ module after JEP486 integration. > > Justin Lu has updated the pull request incrementally with one additional > commit since the l

Re: RFR: 8344331: SM cleanup in java.scripting

2024-11-15 Thread Naoto Sato
On Fri, 15 Nov 2024 23:23:39 GMT, Justin Lu wrote: > Please review this PR which removes the lone `AccessController.doPrivileged` > call from the _java.scripting_ module after JEP486 integration. Marked as reviewed by naoto (Reviewer). src/java.scripting/share/classes/javax/script/ScriptEngine

Re: RFR: 8344331: SM cleanup in java.scripting [v2]

2024-11-15 Thread Naoto Sato
On Sat, 16 Nov 2024 00:03:23 GMT, Justin Lu wrote: >> Please review this PR which removes the lone `AccessController.doPrivileged` >> call from the _java.scripting_ module after JEP486 integration. > > Justin Lu has updated the pull request incrementally with one additional > commit since the l

Re: RFR: 8344331: SM cleanup in java.scripting [v2]

2024-11-15 Thread Lance Andersen
On Sat, 16 Nov 2024 00:03:23 GMT, Justin Lu wrote: >> Please review this PR which removes the lone `AccessController.doPrivileged` >> call from the _java.scripting_ module after JEP486 integration. > > Justin Lu has updated the pull request incrementally with one additional > commit since the l

Re: RFR: 8344331: SM cleanup in java.scripting [v2]

2024-11-15 Thread Justin Lu
> Please review this PR which removes the lone `AccessController.doPrivileged` > call from the _java.scripting_ module after JEP486 integration. Justin Lu has updated the pull request incrementally with one additional commit since the last revision: remove local var 'sl' - Chang