Chris,

On May 7, 2015, at 12:45 PM, Chris Hegarty <[email protected]> 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

Reply via email to