RFR 8173845: JShell API: not patch compatible
Jan Lahoda
jan.lahoda at oracle.com
Fri Feb 3 14:23:25 UTC 2017
Overall, seems OK to me. Regarding the removal of final, I wonder: I
would expect that after the JShell is closed, the whole JShell instance
would be released (so that it would be GCed eventually). Who is keeping
the instance(s)?
Thanks,
Jan
On 3.2.2017 08:13, Robert Field wrote:
> Please review....
>
> Bug:
>
> https://bugs.openjdk.java.net/browse/JDK-8173845
>
> Webrev:
>
> http://cr.openjdk.java.net/~rfield/8173845v0.webrev/
>
> Note: the additional test was the straw that broke the camels back -- I
> could not get the tests to run without running out of memory, so, for
> them to pass I had to add more memory clean-up.
>
> Thanks,
> Robert
>
More information about the kulla-dev
mailing list