<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#ffffff" text="#000000">
    <br>
    The changes look good to me.<br>
    <br>
    -Xiomara<br>
    <br>
    <br>
    On 12/28/2010 1:37 PM, Kelly O'Hair wrote:
    <blockquote
      cite="mid:89E84FB8-5DD0-4845-9CBD-CAC368F50160@oracle.com"
      type="cite"><span class="Apple-style-span" style="font-family:
        monospace;">Need reviewer: OpenJDK7 2010 copyright year changes</span><span
        class="Apple-style-span" style="font-family: monospace;"><br>
      </span><span class="Apple-style-span" style="font-family:
        monospace;"><br>
      </span><span class="Apple-style-span" style="font-family:
        monospace;">Any sources changed in 2010 need to have their
        copyright year adjusted.</span><span class="Apple-style-span"
        style="font-family: monospace;"><br>
      </span><span class="Apple-style-span" style="font-family:
        monospace;"><br>
      </span><span class="Apple-style-span" style="font-family:
        monospace;">Script used is in the top repository at
        make/scripts/update_copyright_year.sh.</span>
      <div><span class="Apple-style-span" style="font-family:
          monospace;"></span><span class="Apple-style-span"
          style="font-family: monospace;">It finds all changesets
          created in 2010 and verifies that any sources with</span></div>
      <div><span class="Apple-style-span" style="font-family:
          monospace;">copyrights mention 2010 adding it if necessary.</span></div>
      <div><span class="Apple-style-span" style="font-family:
          monospace;"><br>
        </span><span class="Apple-style-span" style="font-family:
          monospace;">6962318: Update copyright year</span><span
          class="Apple-style-span" style="font-family: monospace;"><br>
        </span><span class="Apple-style-span" style="font-family:
          monospace;"><a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Eohair/openjdk7/jdk7-build-copyright/webrev/">http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-build-copyright/webrev/</a></span>
        <div><span class="Apple-style-span" style="font-family:
            monospace;"><br>
          </span></div>
        <div><span class="Apple-style-span" style="font-family:
            monospace;">The webrev tool created some false 'file adds',
            not sure whether this is a webrev</span></div>
        <div><span class="Apple-style-span" style="font-family:
            monospace;">bug or a /bin/sh problem on Solaris, but these
            changes only impact the Copyright line.</span></div>
        <div><span class="Apple-style-span" style="font-family:
            monospace;"><br>
          </span><span class="Apple-style-span" style="font-family:
            monospace;">Easiest to just view the 'hg diff -U 0' results
            for each repository impacted:</span></div>
        <div><font class="Apple-style-span" face="monospace">  root:    
             <a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Eohair/openjdk7/jdk7-build-copyright/diffs.jdk7-build-copyright.txt">http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-build-copyright/diffs.jdk7-build-copyright.txt</a><br>
          </font>
          <div><span class="Apple-style-span" style="font-family:
              monospace;">  corba:     <a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Eohair/openjdk7/jdk7-build-copyright/diffs.corba.txt">http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-build-copyright/diffs.corba.txt</a></span></div>
          <div><font class="Apple-style-span" face="monospace">  jaxp:  
                 <a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Eohair/openjdk7/jdk7-build-copyright/diffs.jaxp.txt">http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-build-copyright/diffs.jaxp.txt</a></font></div>
          <div><font class="Apple-style-span" face="monospace">  jaxws:
                  <a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Eohair/openjdk7/jdk7-build-copyright/diffs.jaxws.txt">http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-build-copyright/diffs.jaxws.txt</a></font></div>
          <div><font class="Apple-style-span" face="monospace">  langtools:
              <a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Eohair/openjdk7/jdk7-build-copyright/diffs.langtools.txt">http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-build-copyright/diffs.langtools.txt</a></font></div>
          <div><span class="Apple-style-span" style="font-family:
              monospace;">
              <div style="font-family: Times;"><font
                  class="Apple-style-span" face="monospace">  jdk:      
                  <a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Eohair/openjdk7/jdk7-build-copyright/diffs.jdk.txt">http://cr.openjdk.java.net/~ohair/openjdk7/jdk7-build-copyright/diffs.jdk.txt</a></font></div>
              <div><font class="Apple-style-span" face="monospace"><br>
                </font></div>
            </span><span class="Apple-style-span" style="font-family:
              monospace;">This does not include hotspot. I'll be asking
              the hotspot team to deal with that separately.</span></div>
          <div><span class="Apple-style-span" style="font-family:
              monospace;"><br>
            </span></div>
          <div><span class="Apple-style-span" style="font-family:
              monospace;"></span><span class="Apple-style-span"
              style="font-family: monospace;">I know this is a mindless
              and tedious review. Sorry.<br>
            </span><span class="Apple-style-span" style="font-family:
              monospace;"><br>
            </span><span class="Apple-style-span" style="font-family:
              monospace;">-kto</span></div>
        </div>
      </div>
    </blockquote>
  </body>
</html>