<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jan 15, 2014 at 2:46 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Wed, Jan 15, 2014 at 5:40 PM, Delesley Hutchins <<a href="mailto:delesley@google.com">delesley@google.com</a>> wrote:<br>

>> Why is this code now commented out?<br>
><br>
> It's looked "fishy" to me ;-) and it didn't break any tests when I<br>
> removed it, but I left it there in case I ever discover why it was<br>
> there.<br>
<br>
</div>Can you put a FIXME with an explanation in then?<br></blockquote><div><br></div><div>Just delete it. If you remember what it's for, get it from svn. Let's not have uncommented code in the codebase.</div><div>
 </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
><br>
>> I think you have to check whether the function is static or not as well.<br>
><br>
> Good point.  Thanks!<br>
><br>
>> These can be hoisted into the enclosing if.<br>
><br>
> Will do.<br>
><br>
>> Quite a few style guideline mishaps in here (extra newlines, curly<br>
>> braces when they should be elided, else clauses on their own line,<br>
>> etc). It might make sense to run clang-format over the file at some<br>
>> point.<br>
><br>
> Maybe, but we're not supposed to change SVN blame...<br>
<br>
</div>I've heard that you can run it over just your patch contents (but have<br>
yet to try this).<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div></div>