RFR (M) 8212535: Remove spaces before/after () for vmTestbase/[a-s]*

JC Beyler jcbeyler at google.com
Fri Oct 19 20:49:22 UTC 2018


Hi all,

Sorry about the spam; forgot to add the subject :)

Here is the first of three webrevs to remove spaces around (); I also
removed any space after !.

When the change modified where future parameters should be indented, I
changed those too (such as
http://cr.openjdk.java.net/~jcbeyler/8212535/webrev.00/test/hotspot/jtreg/vmTestbase/nsk/jvmti/IterateOverObjectsReachableFromObject/iterobjreachobj002/iterobjreachobj002.cpp.udiff.html
)

Webrev: http://cr.openjdk.java.net/~jcbeyler/8212535/webrev.00/
Bug: https://bugs.openjdk.java.net/browse/JDK-8212535

Thanks!
Jc

On Fri, Oct 19, 2018 at 1:47 PM JC Beyler <jcbeyler at google.com> wrote:

> Hi all,
>
> Here is the first of three webrevs to remove spaces around (); I also
> removed any space after !.
>
> When the change modified where future parameters should be indented, I
> changed those too (such as
> http://cr.openjdk.java.net/~jcbeyler/8212535/webrev.00/test/hotspot/jtreg/vmTestbase/nsk/jvmti/IterateOverObjectsReachableFromObject/iterobjreachobj002/iterobjreachobj002.cpp.udiff.html
> )
>
> Webrev: https://bugs.openjdk.java.net/browse/JDK-8212535
> Bug: https://bugs.openjdk.java.net/browse/JDK-8212535
>
> Let me know what you think,
> Jc
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20181019/d49063ec/attachment.html>


More information about the serviceability-dev mailing list