Thanks Serguei, David, I uploaded webrev which includes @summary update and 4 indent:
http://cr.openjdk.java.net/~ysuenaga/JDK-8185164/webrev.05/ Could you review and be a sponsor for it? Yasumasa 2017-08-07 17:35 GMT+09:00 serguei.spit...@oracle.com <serguei.spit...@oracle.com>: > On 8/6/17 21:50, David Holmes wrote: >> >> Hi Yasumasa, >> >> On 7/08/2017 2:02 PM, Yasumasa Suenaga wrote: >>> >>> Hi David, >>> >>> I uploaded new webrev: >>> >>> http://cr.openjdk.java.net/~ysuenaga/JDK-8185164/webrev.04/ > > . . . >> >> >>>> Style query: do we do indent of 2 or 4 in hotspot C code tests ? >>> >>> >>> Fixed in C code. >> >> >> I was asking a question - I'm not sure if we use 2 or 4. Now looking at >> some other JVMTI tests they seem to use 4. I didn't do an exhaustive check >> to see if there is a predominant style. > > > The indent was always 4 even for the nsk.jvmti tests. > > > Thanks, > Serguei > >> >> Thanks, >> David >> >