RFR [9] RandomFactory should be in the jdk.testlibrary package (original) (raw)
Brian Burkhalter brian.burkhalter at oracle.com
Thu May 7 20:19:43 UTC 2015
- Previous message: RFR [9] RandomFactory should be in the jdk.testlibrary package
- Next message: AbstractList etc. functionality as interfaces with default methods?
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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
- Previous message: RFR [9] RandomFactory should be in the jdk.testlibrary package
- Next message: AbstractList etc. functionality as interfaces with default methods?
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]