<html><body bgcolor="#FFFFFF"><div><br><br>On Aug 3, 2011, at 12:11 AM, Alan Bateman <<a href="mailto:Alan.Bateman@oracle.com">Alan.Bateman@oracle.com</a>> wrote:<br><br></div><div></div><blockquote type="cite"><div><span>Xuelei Fan wrote:</span><br><blockquote type="cite"><span>:</span><br></blockquote><blockquote type="cite"><span>1. I noticed the copyright date of a few files are unchanged, please</span><br></blockquote><blockquote type="cite"><span>update them before you push the changes.</span><br></blockquote><blockquote type="cite"><span> </span><br></blockquote><span>This has come up a few times but I don't think it is strictly required. Kelly or one of the release engineers run a script over the forest periodically to fix up the date range. </span></div></blockquote>It's very good. I like the way that update the header in bunch.<div><br></div><div>Thanks,</div><div>Xuelei<br><div><br><blockquote type="cite"><div><span>The nice thing (in my view anyway) about not changing the headers is that it avoids clutter in the patch and webrev. It also makes it a bit easier to transport patches to other releases.</span><br><font class="Apple-style-span" color="#000000"><font class="Apple-style-span" color="#0023A3"><br></font></font></div></blockquote><blockquote type="cite"><div><span>-Alan</span><br><span></span><br></div></blockquote></div></div></body></html>