<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Oct 22, 2014 at 5:32 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"><span class="">On Wed, Oct 22, 2014 at 8:13 PM, Richard Smith <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>> wrote:<br>
> On Wed, Oct 22, 2014 at 2:44 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>><br>
> wrote:<br>
>><br>
>> On Wed, Oct 22, 2014 at 5:21 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
>> > My only quandry is: what other things can the body be? If CompoundStmt<br>
>> > and<br>
>> > and CXXTryStmt are the only ones, then we can remove the conditional<br>
>> > entirely.<br>
>><br>
>> I think we can remove that conditional. IssueWarnings calls getBody()<br>
>> on the Decl passed in from PopFunctionScopeInfo, and that should<br>
>> always be something function-like, and I can't think of a situation<br>
>> where we'd get something other than a CompoundStmt or CXXTryStmt.<br>
>><br>
>> Richard, can you think of any situations that would be problematic?<br>
><br>
><br>
> No, the body of a function is always one of those two things.<br>
<br>
</span>Assuming I remove the isa<> tests, does the patch LGTY?</blockquote><div><br></div><div>Yes. </div></div></div></div>