RFR (XS) 8210726: Fix up a few minor nits forgotten by JDK-8210665
JC Beyler
jcbeyler at google.com
Fri Sep 14 19:11:04 UTC 2018
Hi both,
Thanks for the reviews (@Chris: removed the space),
Jc
On Fri, Sep 14, 2018 at 10:54 AM Chris Plummer <chris.plummer at oracle.com>
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/
> Bug: https://bugs.openjdk.java.net/browse/JDK-8210726
>
> Thanks and sorry again,
> Jc
>
>
>
--
Thanks,
Jc
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20180914/bb6cd61d/attachment.html>
More information about the serviceability-dev
mailing list