<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jun 20, 2013 at 4:26 PM, Ye, Mei <span dir="ltr"><<a href="mailto:Mei.Ye@amd.com" target="_blank">Mei.Ye@amd.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">





<div lang="EN-US" link="blue" vlink="purple">
<div>
<p class=""><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">Hi Sean<u></u><u></u></span></p>
<p class=""><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u> <u></u></span></p>
<p class=""><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">See the attached new patch, which fixes all issues brought up by you.<u></u><u></u></span></p>
<p class=""><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">I ran “clang-format” on whole files which makes more diffs.  Hopefully this is OK with you.</span></p></div></div></blockquote>
<div><br></div><div style>Thanks for using clang-format, but unfortunately it's not OK to have unrelated formatting changes in a patch. Please run clang-format over just the added or modified code. (you may want to try clang-format-diff.py; see <<a href="http://clang.llvm.org/docs/ClangFormat.html#script-for-patch-reformatting">http://clang.llvm.org/docs/ClangFormat.html#script-for-patch-reformatting</a>>)</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><div>
<p class=""><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u><u></u></span></p>
<p class=""><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)">It will be nice if the check-in process can automatically run “clang-format”.
<u></u><u></u></span></p>
<p class=""><span style="font-size:11pt;font-family:Calibri,sans-serif;color:rgb(31,73,125)"><u></u> </span></p></div></div></blockquote><div><br></div><div style>Yeah. Someday we will hopefully have that set up!</div><div style>
<br></div><div style>-- Sean Silva </div></div></div></div>