On 5/9/2013 12:09 PM, Dmitry Samersoff wrote:
David,

test/java/lang/CharSequence/DefaultTest.java

has a "Classpath exception" but shouldn't.

Good catch.

Scripts just check for correct format of the header, but not whether the file should have GPL or GPL+classpath.

I've updated the webrev with a corrected GPL header

        http://cr.openjdk.java.net/~katleman/8014289/webrev.jdk.02

The other 3 files are unchanged.

            Thanks
                    Dave
,
 On 2013-05-09 22:38, David Katleman wrote:
Below are corrections to GPL headers found in JDK8 b89:

     http://cr.openjdk.java.net/~katleman/8014289/webrev.jdk.01/

Just 4 files this go around, all are simple punctuation changes, anyone
can review.

As a veteran of many such header reviews, reviewing the patch link at
the top of the webrev is often easier than each individual file.

         Thanks
                 Dave


Reply via email to