RFR(S) : 8180805 : move RandomFactory to the top level testlibrary

Brian Burkhalter brian.burkhalter at oracle.com
Wed May 31 14:50:21 UTC 2017


Hi Igor,

On May 31, 2017, at 7:23 AM, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:

> thank you for your review.

You’re welcome.

> regarding '@Deprecated(since=“9”, forRemoval=true)', since the only users of testlibrary classes are our own tests and we do not need to inform anyone or wait for a new release to remove them, I don't think it makes much sense to have such attributes.

I think it’s fine: I was just pointing out a possibility.

> PS sorry, by some reason the email unnoticeably stuck in my outbox, and I have already pushed the patch. if you have any further comments, I will address them by a separate fix.

No, it’s good.

Thanks,

Brian


More information about the core-libs-dev mailing list