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
