Hi all, Anybody else want to review this? We can close the book on spaces before/after () once this one goes in :) Jc
On Mon, Oct 22, 2018 at 1:37 PM Alex Menkov <alexey.men...@oracle.com> wrote: > LGTM. > > --alex > > On 10/22/2018 11:29, JC Beyler wrote: > > Hi all, > > > > Here is the next webrev (second out of 3) to remove the spaces > > after/before () from vmTestbase. It is straightforward and I generated > > the webrev with white-space changes showing up of course: > > > > Webrev: http://cr.openjdk.java.net/~jcbeyler/8212770/webrev.00/ > > <http://cr.openjdk.java.net/%7Ejcbeyler/8212770/webrev.00/> > > Bug: https://bugs.openjdk.java.net/browse/JDK-8212770 > > > > Could I please get some reviews? > > > > Thanks, > > Jc > -- Thanks, Jc