RFR: 8353487: JShell LocalExecutionControl should allow decorating the execution task [v2]
Archie Cobbs
acobbs at openjdk.org
Mon May 19 15:25:55 UTC 2025
On Mon, 19 May 2025 12:30:21 GMT, xxDark <duke at openjdk.org> wrote:
> that exposes accessible `java.lang.reflect.Method` to user code, but it was not possible to get back the invocation result. With this PR, it is now possible.
Good point. I believe my [original patch](https://github.com/openjdk/jdk/compare/master...68cccbf18ab1dea51e3e673523e77a19cc221b82) which adds a `protected Runnable decorateExecution(Runnable)` method instead did not have this issue, so we could just revert to that approach.
Or there is the other approach (previously mentioned) which adds `executionStart()` and `executionFinish()` methods and should also be safe.
@lahodaj what's your opinion here?
Thanks.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/24398#issuecomment-2891433102
More information about the kulla-dev
mailing list