RFR(XS): 8210925 : move bytecode testlibrary to top-level testlibrary

Maurizio Cimadamore maurizio.cimadamore at oracle.com
Fri Sep 21 20:29:50 UTC 2018

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.


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

More information about the core-libs-dev mailing list