RFR(S) : 8178695: jdk.test.lib.artifacts.ArtifactContainer has to be public

Christian Tornqvist christian.tornqvist at oracle.com
Thu Apr 13 00:40:29 UTC 2017


Hi Igor,

This looks good, thanks for fixing this :)

Thanks,
Christian

> On Apr 12, 2017, at 8:15 PM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
> 
> http://cr.openjdk.java.net/~iignatyev//8178695/webrev.00/index.html
>> 38 lines changed: 33 ins; 5 del; 0 mod; 
> 
> Hi all,
> 
> could you please review this small fix for the testlibrary?
> 
> @ArtifactContainer is created to support repeating @Artifact annotations, but it is package private class, which causes a compilation error if one uses more than one @Artifact. the fix make j.t.l.a.ArtifactContainer publicly accessible.
> 
> webrev: http://cr.openjdk.java.net/~iignatyev//8178695/webrev.00/index.html
> JBS: https://bugs.openjdk.java.net/browse/JDK-8178695
> 
> Thanks,
> -- Igor
> 
> 



More information about the hotspot-dev mailing list