RFR: JEP 359-Records: hotspot runtime and serviceability code

[Harold Seigel]

Hi Sergei,

Thanks for looking at the change.

I'll fix line 7789 in jvmti.xml before the change gets pushed.

Harold

On 10/23/2019 3:46 AM, serguei.spitsyn at oracle.com wrote:
> Hi Vicente and Harold,
>
> I'm still reviewing this, it looks pretty good so far.
>
> One quick comment about a pre-existed issue (not your fault).
> The jvmti.xml has an update to add the Record attribute into the list 
> of attributes that must not be changed:
> 7789 The retransformation must not change the <code>NestHost</code>,
> 7790 <code>NestMembers</code>, or <code>Record</code> attributes.
> The line 7789 has to say:
> 7789 The redefinition must not change the <code>NestHost</code>,
> Could you, please, fix this?
>
> Thanks,
> Serguei
>
>
> On 10/18/19 11:44, Vicente Romero wrote:
>> Hi,
>>
>> Please review the hotspot runtime and serviceability code for JEP 359 
>> (Records).
>>
>> Thanks in advance for the feedback,
>> Vicente
>>
>> PS, Thanks to Harold for the development
>>
>>
>> [1] 
>> http://cr.openjdk.java.net/~vromero/records.review/hotspot_runtime/webrev.00/
>