RFR: https://bugs.openjdk.java.net/browse/JDK-8232785

Frederic Parain frederic.parain at oracle.com
Tue Oct 22 13:59:49 UTC 2019


Thank you for the review.

Fre


> On Oct 22, 2019, at 08:49, David Simms <david.simms at oracle.com> wrote:
> 
> 
> Looks good, thank you Frederic !
> 
> On 22/10/19 2:40 PM, Frederic Parain wrote:
>> Greetings,
>> 
>> Please review this patch adding unit tests for inline types
>> support in jdb. The goal is to have high level unit tests
>> to track regressions at the debugger level. More refined
>> unit tests for each layer (JDI, JDWP, JVMTI, JNI) will be
>> added later once their respective specifications have been
>> updated with inline types support.
>> 
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8232785
>> 
>> Webrev:
>> http://cr.openjdk.java.net/~fparain/jdb_tests/webrev.00/index.html
>> 
>> Note: the patch also contains a 1 line fix in primitive array detection.
>> 
>> Thank you,
>> 
>> Fred
>> 
> 




More information about the valhalla-dev mailing list