RFR (M) 8211036 : Remove the NSK_STUB macros from vmTestbase for non jvmti

[JC Beyler]

Hi all,

Anybody on the hotspot-dev list have any comments or a LGTM?

Thanks!
Jc

On Tue, Sep 25, 2018 at 10:56 AM Alex Menkov <alexey.menkov at oracle.com>
wrote:

> +1
>
> --alex
>
> On 09/24/2018 11:40, Igor Ignatyev wrote:
> > (cc-ing hotspot-dev alias)
> >
> > Hi Jc,
> >
> > the fix looks good to me. don't forget to clean up nsk/share/jni/README
> at the end.
> >
> > Thanks,
> > -- Igor
> >
> >> On Sep 24, 2018, at 9:28 AM, JC Beyler <jcbeyler at google.com> wrote:
> >>
> >> Hi all,
> >>
> >> As the tests have become C++ tests, the NSK_CPP_STUBS are no longer
> needed. I did two awk scripts to remove them and will be rolling them out
> in 50 file max reviews to streamline the reviews for the reviewers.
> >>
> >> So here is the first which handles all the cases outside of the jvmti
> subfolder:
> >>
> >> Webrev: http://cr.openjdk.java.net/~jcbeyler/8211036/webrev.00/ <
> http://cr.openjdk.java.net/~jcbeyler/8211036/webrev.00/>
> >> Bug: https://bugs.openjdk.java.net/browse/JDK-8211036 <
> https://bugs.openjdk.java.net/browse/JDK-8211036>
> >>
> >> The bug contains information on the two scripts I used.
> >>
> >> Thanks,
> >> Jc
> >
>


-- 

Thanks,
Jc
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20180926/2bbc2678/attachment.html>