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