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

[Jan Lahoda]

Hi Alan,

Thanks for the comments! A question inline.

On 25.6.2015 18:38, Alan Bateman wrote:
>
>
> 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

I will work on this.

> 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

I am sorry, but I don't follow - I don't have much experience with 
native programing for JDK. Are there some more details on what I should do?

> trying to keep the code in sync with upstream.

This part of the code is not from the upstream, so I can change it as 
needed.

Thanks!
     Jan

>
> -Alan
>
>
>