RFR (XS) 8210726: Fix up a few minor nits forgotten by JDK-8210665

serguei.spitsyn at oracle.com serguei.spitsyn at oracle.com
Mon Sep 17 07:10:52 UTC 2018


Hi Jc,

+1

Thanks,
Serguei


On 9/14/18 10:54 AM, Chris Plummer wrote:
> One minor issue. There's an extra space in the following line:
>
>  157             fields[i].fid = env-> GetStaticFieldID(cls, 
> fields[i].name, fields[i].sig);
>
> Chris
>
> On 9/14/18 9:49 AM, JC Beyler wrote:
>> Hi all,
>>
>> When doing the work for JDK-8210665, I got confused with the webrevs 
>> and forgot to add Serguei's nits for the patch. I apologize for that 
>> and here they are in a separate bug.
>>
>> Could I get a review for it?
>>
>> Webrev: http://cr.openjdk.java.net/~jcbeyler/8210726/webrev.00/ 
>> <http://cr.openjdk.java.net/%7Ejcbeyler/8210726/webrev.00/>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8210726
>>
>> Thanks and sorry again,
>> Jc
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20180917/4f82e72e/attachment.html>


More information about the serviceability-dev mailing list