On Sunday, April 12, 2015, Sanjoy Das <<a href="mailto:sanjoy@playingwithpointers.com">sanjoy@playingwithpointers.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> It's kind of nice to move code in a separate change from updating it, so<br>
> that it's easier to see what actually changed.<br>
<br>
Done.  The NFCI part is in D8995.</blockquote><div><br></div><div>Thanks.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> We don't usually repeat the function name in the doc comment for new<br>
> code.<br>
<br>
I went with it since that is the convention in the rest of the file.<br>
If it is okay with you, I'd prefer fixing this by submitting this<br>
change as is and fixing the whole ScalarEvolutionExpander.h file at<br>
once in an NFC commit. </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote><div><br></div><div>Sure.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> Also, I could not easily find this in<br>
CodingStandards.html -- should this clause be added there?<br>
<br>
-- Sanjoy<br>
</blockquote>