RFR: JDK-8080679: Include jline in JDK for Java and JavaScript REPLs

Alan Bateman Alan.Bateman at oracle.com
Thu Jun 25 16:38:08 UTC 2015

On 25/06/2015 17:25, Jan Lahoda wrote:
> Hello,
> Based on the feedback I've received so far, I've uploaded an updated 
> version of the patch:
> http://cr.openjdk.java.net/~jlahoda/8080679/webrev.01/full/
> Notable changes:
> -avoided the dependency on java.desktop and java.datatransfer
> -adjusted the native library build script as per Erik's recommendations
> -the module name is now changed to jdk.internal.le
> Any comments are welcome!
The new home and the dropping of the dependences on java.desktop & 
java.datatransfer look good.

Since there only 2 tests then it might be better to add 
jdk/internal/jline to the mixed bag that is jdk_other. That way you 
don't need to update update jprt.properties and other places where the 
groups are used.

The native method readKeyEvent seems to do a FindClass per key event. 
Maybe this is from the upstream project but I would think it would be 
more efficient to cache this in a global ref. It would also be more 
efficient if the INPUT_RECORD were just returned and avoid the upstream 
to create the object but that might be too much to change when you are 
trying to keep the code in sync with upstream.


More information about the nashorn-dev mailing list