Hi Sergey,
There is a leading space added in the following files:
test/java/io/FileInputStream/OpsAfterClose.java
test/sun/util/resources/Locale/Bug6275682.java
The following file does not show 15 modified lines:
test/java/net/URLClassLoader/closetest/build.sh
Other looks fine.
SAM
On 01.11.2013 15:18, Sergey Bylokhov wrote:
Hello.
Please review the fix for jdk 8.
Most of tests in the sound area, and some tests in the client,
java.lang, security, jmx etc has incorrect copyright.
According to the http://openjdk.java.net/faq
"GPL v2 + the Classpath exception for the class libraries and those
parts of the virtual machine that expose public APIs"
But currently our tests mix gpl+cp and gpl or has no header at all.
Tests with "/nodynamiccopyright/", with other company's copyright, or
other gpl templates were not updated.
Also it would be good if in the faq we explicitly mention about
copyright of the tests.
Bug: https://bugs.openjdk.java.net/browse/JDK-8027696
Webrev can be found at:
http://cr.openjdk.java.net/~serb/8027696/webrev.00