<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Feb 15, 2013 at 4:47 PM, Eli Bendersky <span dir="ltr"><<a href="mailto:eliben@google.com" target="_blank">eliben@google.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 Fri, Feb 15, 2013 at 4:44 PM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>

><br>
> Comments?<br>
><br>
>><br>
>> -  if (!TypeName.empty()) {<br>
>> +  if (TypeName.empty()) {<br>
><br>
><br>
> Technically don't need the braces here, but honestly in this case I like<br>
> them :)<br>
><br>
<br>
</div>Dangling else ambiguity avoidance FTW ;-)<br>
<div class="im HOEnZb"><br></div></blockquote><div><br></div><div style>Aha. Excellent. Some comments would still be good ;)</div><div style><br></div><div style>-eric</div><div style><br></div><div> </div></div></div></div>