RFR [9] RandomFactory should be in the jdk.testlibrary package

Brian Burkhalter brian.burkhalter at oracle.com
Thu May 7 20:19:43 UTC 2015


Chris,

On May 7, 2015, at 12:45 PM, Chris Hegarty <chris.hegarty at oracle.com> wrote:

> When trying to use the recently added RandomFactory test library utility, I noticed that it is missing a package declaration. The pattern to date has been for these test library utilities to be declared in the jdk.testlibrary package, and imported by tests requiring them. This is a simple patch that does just that.
> 
> I think this was a minor oversight, as opposed to a deliberate decision to put RandomFactory in the unnamed package.

Indeed it was an oversight on my part having been the first time I work with the jdk.testlibrary package.

Your changes look fine to me assuming that the affected tests all still succeed.

Thanks,

Brian


More information about the core-libs-dev mailing list