Hi Igor,
I think it would be better to postpone this; we currently have at least two different snapshots of the API in various JDK repo - the one you are looking at has been added by Paul in order to test condy; Valhalla has another (slightly tweaked) version to deal with value type support.

Also, the version of the library in jdk/jdk is missing several fixes and improvements added over the last year, which constitutes another reason not to move the API into a more central place, as doing so will likely give a false sense of confidence that the API/impl is mature enough to start building upon it.

That said, I'll register this as a note of positive interest in the API - which means that Brian, Robert and I should start thinking about how to consolidate all these different snapshot, and working on the API in a more transparent way.

I hope this makes sense.

Cheers
Maurizio


On 19/09/18 22:37, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8210925/webrev.00/index.html
14 lines changed: 0 ins; 0 del; 14 mod;
Hi all,

could you please review this small and trivial patch which moves bytecode 
testlibrary to top-level testlibrary directory?

webrev: http://cr.openjdk.java.net/~iignatyev//8210925/webrev.00/index.html
JBS: https://bugs.openjdk.java.net/browse/JDK-8210925
testing: java/lang/invoke/condy tests

Thanks,
-- Igor

Reply via email to