Hi Amy, Not a review ...
On 8/05/2017 7:00 PM, Amy Lu wrote:
Please review this test-only change to move the test library jdk/test/java/util/stream/bootlib to jdk/test/lib/testlibrary/bootlib to make it formally easier for tests to depend on. There is no change to the functionality of the tests. This patch also removed NullArgsTestCase.java which is an abstract class designed to be extended reused with a data provider. It provides no test itself but be wrongly put in the directory of TestNG.dirs thus be run as a test. In fact it is not even used and can be removed. bug: https://bugs.openjdk.java.net/browse/JDK-8085814 https://bugs.openjdk.java.net/browse/JDK-8173414 webrev: http://cr.openjdk.java.net/~amlu/8085814/webrev.00/ (I did not update the year in the Copyright header for bootlib files, to
If the bootlib files were simply moved then no update to their copyright year is needed.
Cheers, David
make the review easier. I can do bulk update for those files on the header before push, if that is necessary.) Thanks, Amy
