RFR (L) 8208242: Add @requires to vmTestbase/gc/g1 tests

JC Beyler jcbeyler at google.com
Wed Aug 8 20:58:53 UTC 2018


Hi Thomas,

I still do need a sponsor :-)

If you have the time, here is the webrev with the metadata:
http://cr.openjdk.java.net/~jcbeyler/8208242/webrev.01/

Thanks again!
Jc


On Wed, Aug 8, 2018 at 12:39 AM Thomas Schatzl <thomas.schatzl at oracle.com>
wrote:

> Hi,
>
> On Thu, 2018-08-02 at 11:29 -0700, JC Beyler wrote:
> > Hi all,
> >
> > I added the fact that the test requires G1 and that unloading is not
> > disabled via 3 new @requires tags.
> >
> > Would someone review the test changes for me?
> > Webrev: http://cr.openjdk.java.net/~jcbeyler/8208242/webrev.00/
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8208242
> >
> > Thanks for your help,
> > Jc
> >
> > Ps: I merged the "add a requires for the unloading"  from https://bug
> > s.openjdk.java.net/browse/JDK-8208206 into JDK-8208242 and updated
> > the text so that I don't do another trivial webrev for all these
> > tests (hopefully)
>
>   looks good.
>
> I assume you still also need a sponsor ;) I can push.
>
> Thanks,
>   Thomas
>
>

-- 

Thanks,
Jc
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20180808/20bf014c/attachment.htm>


More information about the hotspot-gc-dev mailing list