RFR: JDK-8247932: JShell crashes when typing text block
Robert Field
robert.field at oracle.com
Fri Jun 19 18:00:13 UTC 2020
Looks good.
Nit: copyright on ConsoleIOContext needs update (no need for re-review).
-Robert
On 2020-06-19 10:28, Jan Lahoda wrote:
> Hi,
>
> In JDK 15, we've improved JShell to automatically indent multi-line
> snippets while typing them. To detect the proper indent, it is
> necessary to detect '{' and '}'. The javac's lexer has been used for
> this detection. But there is a bug in that - the javac's Log is not
> properly configured for that:
> a) the Log's source is not configured (which leads to the exception
> from the bug)
> b) the Log will print any errors detected while detecting the indent
> to the System.err
>
> These are obviously undesirable, so the proposed patch sets a Log's
> source, and and configures Log to ignore all diagnostics reported.
>
> Proposed webrev:
> http://cr.openjdk.java.net/~jlahoda/8247932/webrev.00/
>
> JBS:
> https://bugs.openjdk.java.net/browse/JDK-8247932
>
> Does this look sensible?
>
> Thanks,
> Jan
More information about the kulla-dev
mailing list